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/10/25 16:47:14 UTC

qpid-jms git commit: NO-JIRA: fix test to avoid spurious failure due to race formed by old changes in connect behaviour

Repository: qpid-jms
Updated Branches:
  refs/heads/master 0089ed185 -> 50118e010


NO-JIRA: fix test to avoid spurious failure due to race formed by old changes in connect behaviour


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

Branch: refs/heads/master
Commit: 50118e0109e309f9825444b70d1218071f005645
Parents: 0089ed1
Author: Robbie Gemmell <ro...@apache.org>
Authored: Thu Oct 25 17:33:20 2018 +0100
Committer: Robbie Gemmell <ro...@apache.org>
Committed: Thu Oct 25 17:35:55 2018 +0100

----------------------------------------------------------------------
 .../org/apache/qpid/jms/provider/amqp/AmqpProviderTest.java     | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/50118e01/qpid-jms-client/src/test/java/org/apache/qpid/jms/provider/amqp/AmqpProviderTest.java
----------------------------------------------------------------------
diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/provider/amqp/AmqpProviderTest.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/provider/amqp/AmqpProviderTest.java
index bcafa45..fd4be1b 100644
--- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/provider/amqp/AmqpProviderTest.java
+++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/provider/amqp/AmqpProviderTest.java
@@ -162,11 +162,12 @@ public class AmqpProviderTest extends QpidJmsTestCase {
         try (TestAmqpPeer testPeer = new TestAmqpPeer()) {
 
             provider = new AmqpProviderFactory().createProvider(getPeerURI(testPeer));
-
             provider.setSaslLayer(false);
-            provider.connect(connectionInfo);
 
             testPeer.expectSaslLayerDisabledConnect(null);
+
+            provider.connect(connectionInfo);
+
             testPeer.expectClose();
 
             provider.close();


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