pr comment

pongad · pongad · commit 91f3a3317c05 · 2017-03-31T14:41:14.000+11:00
diff --git a/google-cloud-logging/src/test/java/com/google/cloud/logging/LoggingHandlerTest.java b/google-cloud-logging/src/test/java/com/google/cloud/logging/LoggingHandlerTest.java
@@ -19,6 +19,7 @@
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.assertFalse;
 
 import com.google.api.gax.core.ApiFutures;
 import com.google.api.gax.core.SettableApiFuture;
@@ -408,12 +409,13 @@ public void run() {
     flushWaiter.start();
 
     // flushWaiter should be waiting for mockRpc to complete.
-    flushWaiter.join(100);
+    flushWaiter.join(1000);
     assertTrue(flushWaiter.isAlive());
 
     // With the RPC completed, flush should return, and the thread should terminate.
     mockRpc.set(null);
-    flushWaiter.join();
+    flushWaiter.join(1000);
+    assertFalse(flushWaiter.isAlive());
   }
 
   @Test
