You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by gt...@apache.org on 2010/07/28 17:30:22 UTC

svn commit: r980095 - /activemq/trunk/activemq-core/src/test/java/org/apache/activemq/usecases/NoDuplicateOnTopicNetworkTest.java

Author: gtully
Date: Wed Jul 28 15:30:22 2010
New Revision: 980095

URL: http://svn.apache.org/viewvc?rev=980095&view=rev
Log:
resolve intermittent hudson failure with test, may not always produce dups so verify that case

Modified:
    activemq/trunk/activemq-core/src/test/java/org/apache/activemq/usecases/NoDuplicateOnTopicNetworkTest.java

Modified: activemq/trunk/activemq-core/src/test/java/org/apache/activemq/usecases/NoDuplicateOnTopicNetworkTest.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/usecases/NoDuplicateOnTopicNetworkTest.java?rev=980095&r1=980094&r2=980095&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/test/java/org/apache/activemq/usecases/NoDuplicateOnTopicNetworkTest.java (original)
+++ activemq/trunk/activemq-core/src/test/java/org/apache/activemq/usecases/NoDuplicateOnTopicNetworkTest.java Wed Jul 28 15:30:22 2010
@@ -211,7 +211,11 @@ public class NoDuplicateOnTopicNetworkTe
             assertEquals("got all required messages: " + map.size(), consumer
                     .getNumMessages(), map.size());
         } else {
-            assertTrue("we got some duplicates", duplicateCount > 0);
+            assertTrue("we can get some duplicates: " + duplicateCount, duplicateCount >= 0);
+            if (duplicateCount == 0) {
+               assertEquals("got all required messages: " + map.size(), consumer
+                    .getNumMessages(), map.size()); 
+            }
         }
     }