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 2009/07/23 11:00:40 UTC

svn commit: r796983 - /activemq/trunk/activemq-core/src/test/java/org/apache/activemq/broker/BrokerTest.java

Author: gtully
Date: Thu Jul 23 09:00:40 2009
New Revision: 796983

URL: http://svn.apache.org/viewvc?rev=796983&view=rev
Log:
improve assert information to help diagnose intermittent failure

Modified:
    activemq/trunk/activemq-core/src/test/java/org/apache/activemq/broker/BrokerTest.java

Modified: activemq/trunk/activemq-core/src/test/java/org/apache/activemq/broker/BrokerTest.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/broker/BrokerTest.java?rev=796983&r1=796982&r2=796983&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/test/java/org/apache/activemq/broker/BrokerTest.java (original)
+++ activemq/trunk/activemq-core/src/test/java/org/apache/activemq/broker/BrokerTest.java Thu Jul 23 09:00:40 2009
@@ -1187,7 +1187,8 @@
         // Send another message, connection1 should not get the message.
         connection2.request(createMessage(producerInfo2, destination, deliveryMode));
 
-        assertNull(connection1.getDispatchQueue().poll(MAX_NULL_WAIT, TimeUnit.MILLISECONDS));
+        Object msg = connection1.getDispatchQueue().poll(MAX_NULL_WAIT, TimeUnit.MILLISECONDS);
+        assertNull("no message received from connection1 after session close", msg);
     }
 
     public void initCombosForTestConsumerClose() {