You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ro...@apache.org on 2018/06/25 09:15:49 UTC

qpid-jms git commit: QPIDJMS-395: fix some c&p test comments

Repository: qpid-jms
Updated Branches:
  refs/heads/master 133340800 -> ed27c4ea5


QPIDJMS-395: fix some c&p test comments


Project: http://git-wip-us.apache.org/repos/asf/qpid-jms/repo
Commit: http://git-wip-us.apache.org/repos/asf/qpid-jms/commit/ed27c4ea
Tree: http://git-wip-us.apache.org/repos/asf/qpid-jms/tree/ed27c4ea
Diff: http://git-wip-us.apache.org/repos/asf/qpid-jms/diff/ed27c4ea

Branch: refs/heads/master
Commit: ed27c4ea51f19b822902e289b6d6b5023e5925b8
Parents: 1333408
Author: Robbie Gemmell <ro...@apache.org>
Authored: Mon Jun 25 10:15:33 2018 +0100
Committer: Robbie Gemmell <ro...@apache.org>
Committed: Mon Jun 25 10:15:33 2018 +0100

----------------------------------------------------------------------
 .../apache/qpid/jms/integration/ConnectionIntegrationTest.java  | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/ed27c4ea/qpid-jms-client/src/test/java/org/apache/qpid/jms/integration/ConnectionIntegrationTest.java
----------------------------------------------------------------------
diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/integration/ConnectionIntegrationTest.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/integration/ConnectionIntegrationTest.java
index 8455640..3293dfd 100644
--- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/integration/ConnectionIntegrationTest.java
+++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/integration/ConnectionIntegrationTest.java
@@ -670,11 +670,10 @@ public class ConnectionIntegrationTest extends QpidJmsTestCase {
             testPeer.expectBegin();
             Session session = connection.createSession(false, Session.AUTO_ACKNOWLEDGE);
 
-            // Expect producer creation, don't give it credit.
+            // Expect producer creation, and a message to be sent, but don't return a disposition.
+            // Instead, close the connection.
             testPeer.expectSenderAttach();
             testPeer.expectTransferButDoNotRespond(new TransferPayloadCompositeMatcher());
-
-            // Producer has no credit so the send should block waiting for it.
             testPeer.remotelyCloseConnection(true, ConnectionError.CONNECTION_FORCED, BREAD_CRUMB, 50);
 
             Queue queue = session.createQueue("myQueue");


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org