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

[chirino]

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);
         }
     }