Made test more robust.

The check for the connection being closed may fail spuriously because
the notification of CompleteListener happens before the connection is
closed.

1 files changed, 4 insertions, 0 deletions

diff --git a/jetty-client/src/test/java/org/eclipse/jetty/client/HttpClientTest.java b/jetty-client/src/test/java/org/eclipse/jetty/client/HttpClientTest.java
index 87c03e43f7..f7ccdfcf63 100644
--- a/jetty-client/src/test/java/org/eclipse/jetty/client/HttpClientTest.java
+++ b/jetty-client/src/test/java/org/eclipse/jetty/client/HttpClientTest.java
@@ -1137,6 +1137,10 @@ public class HttpClientTest extends AbstractHttpClientServerTest
             ContentResponse response = listener.get(2 * timeout, TimeUnit.MILLISECONDS);
 
             Assert.assertEquals(200, response.getStatus());
+            // The parser notifies end-of-content and therefore the CompleteListener
+            // before closing the connection, so we need to wait before checking
+            // that the connection is closed to avoid races.
+            Thread.sleep(1000);
             Assert.assertTrue(((HttpConnectionOverHTTP)connection).isClosed());
         }
     }