You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ri...@apache.org on 2007/10/11 13:28:37 UTC

svn commit: r583776 - /incubator/qpid/branches/M2.1/java/systests/src/main/java/org/apache/qpid/test/framework/localcircuit/LocalCircuitImpl.java

Author: ritchiem
Date: Thu Oct 11 04:28:36 2007
New Revision: 583776

URL: http://svn.apache.org/viewvc?rev=583776&view=rev
Log:
QPID-632 ImmediateMessageTest may not have sufficient time to see the exception. Performing the close before checking for exceptions should give it this time.

Modified:
    incubator/qpid/branches/M2.1/java/systests/src/main/java/org/apache/qpid/test/framework/localcircuit/LocalCircuitImpl.java

Modified: incubator/qpid/branches/M2.1/java/systests/src/main/java/org/apache/qpid/test/framework/localcircuit/LocalCircuitImpl.java
URL: http://svn.apache.org/viewvc/incubator/qpid/branches/M2.1/java/systests/src/main/java/org/apache/qpid/test/framework/localcircuit/LocalCircuitImpl.java?rev=583776&r1=583775&r2=583776&view=diff
==============================================================================
--- incubator/qpid/branches/M2.1/java/systests/src/main/java/org/apache/qpid/test/framework/localcircuit/LocalCircuitImpl.java (original)
+++ incubator/qpid/branches/M2.1/java/systests/src/main/java/org/apache/qpid/test/framework/localcircuit/LocalCircuitImpl.java Thu Oct 11 04:28:36 2007
@@ -406,11 +406,11 @@
         // Request a status report.
         check();
 
-        // Apply all of the requested assertions, keeping record of any that fail.
-        List<Assertion> failures = applyAssertions(assertions);
-
         // Clean up the publisher/receivers/controlSession/connections.
         close();
+
+        // Apply all of the requested assertions, keeping record of any that fail.
+        List<Assertion> failures = applyAssertions(assertions);
 
         // Return any failed assertions to the caller.
         return failures;