Eclipse/jetty.project
1
0
Fork 0
mirror of https://github.com/jetty/jetty.project.git synced 2025-03-04 12:59:30 +00:00
Code Issues Packages Projects Releases Wiki Activity

Made test more robust.

Browse Source 
The check for the connection being closed may fail spuriously because
the notification of CompleteListener happens before the connection is
closed.
This commit is contained in:
Simone Bordet 2014-02-06 11:56:25 +01:00
parent 5edf2799e9
commit 9356ab46da
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
jetty-client/src/test/java/org/eclipse/jetty/client/HttpClientTest.java
View File

@ -1137,6 +1137,10 @@ public class HttpClientTest extends AbstractHttpClientServerTest
ContentResponse response = listener.get(2 * timeout, TimeUnit.MILLISECONDS); ContentResponse response = listener.get(2 * timeout, TimeUnit.MILLISECONDS);
Assert.assertEquals(200, response.getStatus()); 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()); Assert.assertTrue(((HttpConnectionOverHTTP)connection).isClosed());
} }
} }