svn commit: r365367 - /incubator/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/util/MessageIdList.java

[ch...@apache.org]

Author: chirino
Date: Mon Jan  2 08:05:31 2006
New Revision: 365367

URL: http://svn.apache.org/viewcvs?rev=365367&view=rev
Log:
The SimpleDispatchPolicyTest and RoundRobinDispatchPolicyTest were failing on my single proc powerbook.  
The parent of the MessageIdList was being called before the clild had processed the message which
cause the test to not wait till the childeren had processed the messages so tests would fail 
due to short counts.

Modified:
    incubator/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/util/MessageIdList.java

Modified: incubator/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/util/MessageIdList.java
URL: http://svn.apache.org/viewcvs/incubator/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/util/MessageIdList.java?rev=365367&r1=365366&r2=365367&view=diff
==============================================================================
--- incubator/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/util/MessageIdList.java (original)
+++ incubator/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/util/MessageIdList.java Mon Jan  2 08:05:31 2006
@@ -90,9 +90,6 @@
     }
 
     public void onMessage(Message message) {
-        if (parent != null) {
-            parent.onMessage(message);
-        }
         String id=null;
         try {
             id = message.getJMSMessageID();
@@ -105,6 +102,9 @@
             }
         } catch (JMSException e) {
             e.printStackTrace();
+        }
+        if (parent != null) {
+            parent.onMessage(message);
         }
     }