[Jboss-cvs] JBoss Messaging SVN: r1214 - in trunk/tests/src/org/jboss/test/messaging/jms: . server/destination

[jboss-cvs-commits at lists.jboss.org]

Author: timfox
Date: 2006-08-08 09:10:56 -0400 (Tue, 08 Aug 2006)
New Revision: 1214

Modified:
   trunk/tests/src/org/jboss/test/messaging/jms/AcknowledgmentTest.java
   trunk/tests/src/org/jboss/test/messaging/jms/server/destination/TopicManagementTest.java
Log:
Interim race fix



Modified: trunk/tests/src/org/jboss/test/messaging/jms/AcknowledgmentTest.java
===================================================================
--- trunk/tests/src/org/jboss/test/messaging/jms/AcknowledgmentTest.java	2006-08-08 11:47:48 UTC (rev 1213)
+++ trunk/tests/src/org/jboss/test/messaging/jms/AcknowledgmentTest.java	2006-08-08 13:10:56 UTC (rev 1214)
@@ -1192,6 +1192,8 @@
    
    private boolean assertRemainingMessages(int expected) throws Exception
    {
+      //Need to pause since delivery may still be in progress
+      Thread.sleep(1000);
       ObjectName destObjectName = 
          new ObjectName("jboss.messaging.destination:service=Queue,name=Queue");
       Integer messageCount = (Integer)ServerManagement.getAttribute(destObjectName, "MessageCount");      

Modified: trunk/tests/src/org/jboss/test/messaging/jms/server/destination/TopicManagementTest.java
===================================================================
--- trunk/tests/src/org/jboss/test/messaging/jms/server/destination/TopicManagementTest.java	2006-08-08 11:47:48 UTC (rev 1213)
+++ trunk/tests/src/org/jboss/test/messaging/jms/server/destination/TopicManagementTest.java	2006-08-08 13:10:56 UTC (rev 1214)
@@ -571,6 +571,9 @@
          
          // Remove all messages from the topic
          
+         //Need to pause since delivery may still be in progress
+         Thread.sleep(2000);
+         
          ServerManagement.invoke(destObjectName, "removeAllMessages", null, null);
    
          // Try to receive messages from the two subscriptions, should be null