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 052d7ef0 PROTON-2780 Add missing AMQP header expectation to test script
052d7ef0 is described below
commit 052d7ef00dfff63082e5c60be0ea4081d048fdbb
Author: Timothy Bish <tabish...@gmail.com>
AuthorDate: Thu Dec 7 15:08:48 2023 -0500
PROTON-2780 Add missing AMQP header expectation to test script
Race on test completion can fail if a header which is script to fire is
read before the test completes as test lacks an expectation for the header.
---
.../src/main/java/org/apache/qpid/protonj2/test/driver/ScriptWriter.java | 1 +
1 file changed, 1 insertion(+)
diff --git
a/protonj2-test-driver/src/main/java/org/apache/qpid/protonj2/test/driver/ScriptWriter.java
b/protonj2-test-driver/src/main/java/org/apache/qpid/protonj2/test/driver/ScriptWriter.java
index 1138d8f3..e92b9098 100644
---
a/protonj2-test-driver/src/main/java/org/apache/qpid/protonj2/test/driver/ScriptWriter.java
+++
b/protonj2-test-driver/src/main/java/org/apache/qpid/protonj2/test/driver/ScriptWriter.java
@@ -629,6 +629,7 @@ public abstract class ScriptWriter {
remoteSaslInit().withMechanism("PLAIN").withInitialResponse(saslPlainInitialResponse(username,
password)).queue();
expectSaslOutcome().withCode(SaslCode.OK);
remoteAMQPHeader().queue();
+ expectAMQPHeader();
// This trigger the exchange of frames.
remoteSASLHeader().now();
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org
For additional commands, e-mail: commits-h...@qpid.apache.org
