[jboss-cvs] JBossAS SVN: r69768 - branches/Branch_4_2/testsuite/src/main/org/jboss/test/jbossmq/test.

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

Author: adrian at jboss.org
Date: 2008-02-11 09:51:26 -0500 (Mon, 11 Feb 2008)
New Revision: 69768

Modified:
   branches/Branch_4_2/testsuite/src/main/org/jboss/test/jbossmq/test/ConnectionConsumerErrorFiredUnitTestCase.java
Log:
[JBAS-5088] - Fix the test to avoid a race between us saying we want it to throw an error and the background thread checking for this condition

Modified: branches/Branch_4_2/testsuite/src/main/org/jboss/test/jbossmq/test/ConnectionConsumerErrorFiredUnitTestCase.java
===================================================================
--- branches/Branch_4_2/testsuite/src/main/org/jboss/test/jbossmq/test/ConnectionConsumerErrorFiredUnitTestCase.java	2008-02-11 07:43:20 UTC (rev 69767)
+++ branches/Branch_4_2/testsuite/src/main/org/jboss/test/jbossmq/test/ConnectionConsumerErrorFiredUnitTestCase.java	2008-02-11 14:51:26 UTC (rev 69768)
@@ -51,8 +51,8 @@
       try
       {
          ServerSessionPool pool = new DummyServerSessionPool(connection);
+         setThrowError(connection, "receive");
          connection.createConnectionConsumer(new SpyQueue("QUEUE.testQueue"), null, pool, 1);
-         setThrowError(connection, "receive");
          connection.start();
          
          JMSException e = getException();