[jboss-cvs] JBoss Messaging SVN: r7334 - trunk/tests/src/org/jboss/messaging/tests/integration/cluster/failover.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Mon Jun 15 08:35:11 EDT 2009


Author: timfox
Date: 2009-06-15 08:35:11 -0400 (Mon, 15 Jun 2009)
New Revision: 7334

Modified:
   trunk/tests/src/org/jboss/messaging/tests/integration/cluster/failover/ReplicateConnectionFailureTest.java
Log:
tweak to test

Modified: trunk/tests/src/org/jboss/messaging/tests/integration/cluster/failover/ReplicateConnectionFailureTest.java
===================================================================
--- trunk/tests/src/org/jboss/messaging/tests/integration/cluster/failover/ReplicateConnectionFailureTest.java	2009-06-15 12:09:53 UTC (rev 7333)
+++ trunk/tests/src/org/jboss/messaging/tests/integration/cluster/failover/ReplicateConnectionFailureTest.java	2009-06-15 12:35:11 UTC (rev 7334)
@@ -109,8 +109,17 @@
 
       ((ConnectionManagerImpl)sf1.getConnectionManagers()[0]).cancelPingerForConnectionID(conn1.getID());
 
-      Thread.sleep(3 * clientFailureCheckPeriod);
+      for (int i = 0; i < 1000; i++)
+      {
+         // a few tries to avoid a possible race caused by GCs or similar issues
+         if (liveServer.getRemotingService().getConnections().isEmpty())
+         {
+            break;
+         }
 
+         Thread.sleep(10);
+      }
+
       assertEquals(0, liveServer.getRemotingService().getConnections().size());
 
       assertEquals(1, backupServer.getRemotingService().getConnections().size());




More information about the jboss-cvs-commits mailing list