This is an automated email from the ASF dual-hosted git repository.

tabish pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/qpid-protonj2.git


The following commit(s) were added to refs/heads/main by this push:
     new f2e62987 PROTON-2780 Remove leftover thread sleep from test
f2e62987 is described below

commit f2e629876c3aeef736cb0bd38552973a10b8200e
Author: Timothy Bish <[email protected]>
AuthorDate: Thu Dec 7 15:35:02 2023 -0500

    PROTON-2780 Remove leftover thread sleep from test
    
    Remove an old thread sleep and add in test expecation for connection
    drop to fully validate client close on request.
---
 .../org/apache/qpid/protonj2/test/driver/ProtonTestClientTest.java     | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/protonj2-test-driver/src/test/java/org/apache/qpid/protonj2/test/driver/ProtonTestClientTest.java
 
b/protonj2-test-driver/src/test/java/org/apache/qpid/protonj2/test/driver/ProtonTestClientTest.java
index be86733d..ae5f98b6 100644
--- 
a/protonj2-test-driver/src/test/java/org/apache/qpid/protonj2/test/driver/ProtonTestClientTest.java
+++ 
b/protonj2-test-driver/src/test/java/org/apache/qpid/protonj2/test/driver/ProtonTestClientTest.java
@@ -44,6 +44,7 @@ class ProtonTestClientTest extends TestPeerTestsBase {
     public void testClientCanConnectAndExchangeAMQPHeaders() throws Exception {
         try (ProtonTestServer peer = new ProtonTestServer()) {
             peer.expectAMQPHeader().respondWithAMQPHeader();
+            peer.expectConnectionToDrop();
             peer.start();
 
             URI remoteURI = peer.getServerURI();
@@ -58,8 +59,6 @@ class ProtonTestClientTest extends TestPeerTestsBase {
 
             LOG.info("Test started, peer listening on: {}", remoteURI);
 
-            Thread.sleep(100);
-
             peer.waitForScriptToComplete(5, TimeUnit.SECONDS);
         }
     }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to