summaryrefslogtreecommitdiff
path: root/qpid/java
diff options
context:
space:
mode:
authorKeith Wall <kwall@apache.org>2014-09-25 07:39:24 +0000
committerKeith Wall <kwall@apache.org>2014-09-25 07:39:24 +0000
commit5beb9bb52c58ea9c36f2946aaae3c26e4e9c9471 (patch)
tree649b817ba5bc03dd3cadc8391ac8180916da40c5 /qpid/java
parentd7bdf7c5906657680ea3ca5ea0251c137086cb63 (diff)
downloadqpid-python-5beb9bb52c58ea9c36f2946aaae3c26e4e9c9471.tar.gz
NO-JIRA: [Java Broker Tests] Fix for occasionally failing ConnectionManagementTest on 0-10 path
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1627472 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'qpid/java')
-rw-r--r--qpid/java/systests/src/test/java/org/apache/qpid/systest/management/jmx/ConnectionManagementTest.java10
1 files changed, 10 insertions, 0 deletions
diff --git a/qpid/java/systests/src/test/java/org/apache/qpid/systest/management/jmx/ConnectionManagementTest.java b/qpid/java/systests/src/test/java/org/apache/qpid/systest/management/jmx/ConnectionManagementTest.java
index 34b13dfaca..67af3e17e4 100644
--- a/qpid/java/systests/src/test/java/org/apache/qpid/systest/management/jmx/ConnectionManagementTest.java
+++ b/qpid/java/systests/src/test/java/org/apache/qpid/systest/management/jmx/ConnectionManagementTest.java
@@ -82,6 +82,16 @@ public class ConnectionManagementTest extends QpidBrokerTestCase
assertEquals("Expected one managed connection", 1, getManagedConnections().size());
_connection.close();
+
+ // On the 0-10 path, the connection close ok is sent *before* the model is updated, so we need poll
+ // to reliable detect the state change.
+ int counter = 0;
+ while(getManagedConnections().size() > 0 && counter < 50)
+ {
+ sleep();
+ counter++;
+ }
+
assertEquals("Expected no managed connections after client connection closed", 0, getManagedConnections().size());
}