Fix shutting down a never-started ServerImpl

core/src/main/java/io/grpc/ServerImpl.java

@@ -132,13 +132,21 @@ public final class ServerImpl extends Server {
    * Initiates an orderly shutdown in which preexisting calls continue but new calls are rejected.
    */
   public ServerImpl shutdown() {
+    boolean shutdownTransportServer;
     synchronized (lock) {
       if (shutdown) {
         return this;
       }
       shutdown = true;
+      shutdownTransportServer = started;
+      if (!shutdownTransportServer) {
+        transportServerTerminated = true;
+        checkForTermination();
       }
+    }
+    if (shutdownTransportServer) {
       transportServer.shutdown();
+    }
     SharedResourceHolder.release(TIMER_SERVICE, timeoutService);
     return this;
   }

core/src/test/java/io/grpc/ServerImplTest.java

@@ -115,6 +115,20 @@ public class ServerImplTest {
     assertTrue(server.isTerminated());
   }
 
+  @Test
+  public void stopImmediate() {
+    transportServer = new SimpleServer() {
+      @Override
+      public void shutdown() {
+        throw new AssertionError("Should not be called, because wasn't started");
+      }
+    };
+    ServerImpl server = new ServerImpl(executor, registry, transportServer);
+    server.shutdown();
+    assertTrue(server.isShutdown());
+    assertTrue(server.isTerminated());
+  }
+
   @Test
   public void startStopImmediateWithChildTransport() throws IOException {
     ServerImpl server = new ServerImpl(executor, registry, transportServer);