Checkstyle

Author: rmichela

grpc-contrib/src/test/java/com/salesforce/grpc/contrib/ServersTest.java

@@ -34,7 +34,7 @@ public void shutdownGracefullyThrowsIfMaxWaitTimeInMillisIsZero() {
 
         assertThatThrownBy(() -> Servers.shutdownGracefully(server, 0))
                 .isInstanceOf(IllegalArgumentException.class)
-                .hasMessageContaining("maxWaitTimeInMillis");
+                .hasMessageContaining("timeout must be greater than 0");
     }
 
     @Test
@@ -44,7 +44,7 @@ public void shutdownGracefullyThrowsIfMaxWaitTimeInMillisIsLessThanZero() {
 
         assertThatThrownBy(() -> Servers.shutdownGracefully(server, maxWaitTimeInMillis))
                 .isInstanceOf(IllegalArgumentException.class)
-                .hasMessageContaining("maxWaitTimeInMillis");
+                .hasMessageContaining("timeout must be greater than 0");
     }
 
     @Test

grpc-testing-contrib/src/main/java/com/salesforce/grpc/testing/contrib/GrpcContextRule.java

@@ -13,8 +13,6 @@
 import org.junit.runner.Description;
 import org.junit.runners.model.Statement;
 
-import java.util.logging.Logger;
-
 /**
  * {@code GrpcContextRule} is a JUnit {@link TestRule} that forcibly resets the gRPC
  * {@link Context} to {@link Context#ROOT} between every unit test.