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]