From 3f91549931c2e0bfd5968a1260e8851b48ffc8a9 Mon Sep 17 00:00:00 2001 From: Keith Wall Date: Thu, 25 Sep 2014 07:39:24 +0000 Subject: 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 --- .../qpid/systest/management/jmx/ConnectionManagementTest.java | 10 ++++++++++ 1 file changed, 10 insertions(+) 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()); } -- cgit v1.2.1