summaryrefslogtreecommitdiff
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
commit3f91549931c2e0bfd5968a1260e8851b48ffc8a9 (patch)
treec73a372daee7c0351b90e73fa431e6b4e915c42b
parentb09aac0146eb61548f1f762366f868d00e20091a (diff)
downloadqpid-python-3f91549931c2e0bfd5968a1260e8851b48ffc8a9.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/qpid@1627472 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r--java/systests/src/test/java/org/apache/qpid/systest/management/jmx/ConnectionManagementTest.java10
1 files changed, 10 insertions, 0 deletions
diff --git a/java/systests/src/test/java/org/apache/qpid/systest/management/jmx/ConnectionManagementTest.java b/java/systests/src/test/java/org/apache/qpid/systest/management/jmx/ConnectionManagementTest.java
index 34b13dfaca..67af3e17e4 100644
--- a/java/systests/src/test/java/org/apache/qpid/systest/management/jmx/ConnectionManagementTest.java
+++ b/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());
}