You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ta...@apache.org on 2012/06/11 21:43:52 UTC

svn commit: r1348983 - /activemq/trunk/activemq-core/src/test/java/org/apache/activemq/transport/vm/VMTransportThreadSafeTest.java

Author: tabish
Date: Mon Jun 11 19:43:52 2012
New Revision: 1348983

URL: http://svn.apache.org/viewvc?rev=1348983&view=rev
Log:
Test can fail on slower machines, give it a bit more time and improve the assert output. 

Modified:
    activemq/trunk/activemq-core/src/test/java/org/apache/activemq/transport/vm/VMTransportThreadSafeTest.java

Modified: activemq/trunk/activemq-core/src/test/java/org/apache/activemq/transport/vm/VMTransportThreadSafeTest.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/transport/vm/VMTransportThreadSafeTest.java?rev=1348983&r1=1348982&r2=1348983&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/test/java/org/apache/activemq/transport/vm/VMTransportThreadSafeTest.java (original)
+++ activemq/trunk/activemq-core/src/test/java/org/apache/activemq/transport/vm/VMTransportThreadSafeTest.java Mon Jun 11 19:43:52 2012
@@ -487,7 +487,7 @@ public class VMTransportThreadSafeTest {
             @Override
             public void run() {
                 try {
-                    Thread.sleep(500);
+                    Thread.sleep(1000);
                     remote.stop();
                 } catch (Exception e) {
                 }
@@ -498,7 +498,7 @@ public class VMTransportThreadSafeTest {
 
         t.start();
 
-        assertTrue(Wait.waitFor(new Wait.Condition() {
+        assertTrue("Remote should receive: " + expect + ", commands but got: " + remoteReceived.size(), Wait.waitFor(new Wait.Condition() {
             @Override
             public boolean isSatisified() throws Exception {
                 return remoteReceived.size() >= expect;