[jboss-cvs] JBoss Messaging SVN: r3786 - trunk/tests/src/org/jboss/test/messaging/jms/server.

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

Author: jmesnil
Date: 2008-02-25 08:45:12 -0500 (Mon, 25 Feb 2008)
New Revision: 3786

Modified:
   trunk/tests/src/org/jboss/test/messaging/jms/server/JMSServerManagerTest.java
Log:
* added a small delay in testClientInfo() to make the test pass in the remoting case (Connection.start() is a one-way operation and we query the server right after sending the message and before it has been treated by the server)

Modified: trunk/tests/src/org/jboss/test/messaging/jms/server/JMSServerManagerTest.java
===================================================================
--- trunk/tests/src/org/jboss/test/messaging/jms/server/JMSServerManagerTest.java	2008-02-25 13:19:59 UTC (rev 3785)
+++ trunk/tests/src/org/jboss/test/messaging/jms/server/JMSServerManagerTest.java	2008-02-25 13:45:12 UTC (rev 3786)
@@ -21,6 +21,7 @@
    */
 package org.jboss.test.messaging.jms.server;
 
+import org.apache.tools.ant.taskdefs.Sleep;
 import org.jboss.test.messaging.JBMServerTestCase;
 import org.jboss.jms.server.JMSServerManager;
 import org.jboss.jms.server.ClientInfo;
@@ -225,6 +226,9 @@
       assertEquals("guest", clientInfo.getUser());
       assertEquals(ClientInfo.status.STOPPED, clientInfo.getStatus());
       conn.start();
+      // starting a connection is a remoting async operation
+      // wait a little before querying clients infos from the server
+      sleepIfRemoting(250);
       clientInfos = jmsServerManager.getClients();
       assertNotNull(clientInfos);
       assertEquals(1, clientInfos.size());