Use a more reasonable timeout

michaelklishin · michaelklishin · commit 593a6340f104 · 2016-04-13T17:16:35.000+03:00
diff --git a/src/test/java/com/rabbitmq/client/test/functional/ConnectionRecovery.java b/src/test/java/com/rabbitmq/client/test/functional/ConnectionRecovery.java
@@ -750,9 +750,9 @@ private ConnectionFactory buildConnectionFactoryWithRecoveryEnabled(boolean disa
     }
 
     private static void wait(CountDownLatch latch) throws InterruptedException {
-        // Very very generous amount of time to wait, just make sure we never
-        // hang forever
-        assertTrue(latch.await(1800, TimeUnit.SECONDS));
+        // we want to wait for recovery to complete for a reasonable amount of time
+        // but still make recovery failures easy to notice in development environments
+        assertTrue(latch.await(90, TimeUnit.SECONDS));
     }
 
     private void waitForConfirms(Channel ch) throws InterruptedException, TimeoutException {

Original file line number	Diff line number	Diff line change
`@@ -750,9 +750,9 @@ private ConnectionFactory buildConnectionFactoryWithRecoveryEnabled(boolean disa`
`750`	`750`	`}`
`751`	`751`
`752`	`752`	`private static void wait(CountDownLatch latch) throws InterruptedException {`
`753`		`- // Very very generous amount of time to wait, just make sure we never`
`754`		`- // hang forever`
`755`		`- assertTrue(latch.await(1800, TimeUnit.SECONDS));`
	`753`	`+ // we want to wait for recovery to complete for a reasonable amount of time`
	`754`	`+ // but still make recovery failures easy to notice in development environments`
	`755`	`+ assertTrue(latch.await(90, TimeUnit.SECONDS));`
`756`	`756`	`}`
`757`	`757`
`758`	`758`	`private void waitForConfirms(Channel ch) throws InterruptedException, TimeoutException {`