You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by cl...@apache.org on 2018/05/05 02:47:56 UTC

[2/2] activemq-artemis git commit: NO-JIRA fix potential race in SessionTest

NO-JIRA fix potential race in SessionTest


Project: http://git-wip-us.apache.org/repos/asf/activemq-artemis/repo
Commit: http://git-wip-us.apache.org/repos/asf/activemq-artemis/commit/90f52920
Tree: http://git-wip-us.apache.org/repos/asf/activemq-artemis/tree/90f52920
Diff: http://git-wip-us.apache.org/repos/asf/activemq-artemis/diff/90f52920

Branch: refs/heads/master
Commit: 90f52920e5c2a29f3a958d187d7dfb9eb80e7169
Parents: 25f8820
Author: Justin Bertram <jb...@apache.org>
Authored: Fri May 4 13:26:14 2018 -0500
Committer: Clebert Suconic <cl...@apache.org>
Committed: Fri May 4 22:47:54 2018 -0400

----------------------------------------------------------------------
 .../activemq/artemis/tests/integration/client/SessionTest.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/90f52920/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/client/SessionTest.java
----------------------------------------------------------------------
diff --git a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/client/SessionTest.java b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/client/SessionTest.java
index 9954a4e..0175caa 100644
--- a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/client/SessionTest.java
+++ b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/client/SessionTest.java
@@ -342,7 +342,7 @@ public class SessionTest extends ActiveMQTestBase {
       Queue q = (Queue) server.getPostOffice().getBinding(new SimpleString(queueName)).getBindable();
       Assert.assertEquals(0, getMessageCount(q));
       clientSession.commit();
-      Assert.assertEquals(10, getMessageCount(q));
+      Assert.assertTrue(Wait.waitFor(() -> getMessageCount(q) == 10, 2000, 100));
       clientSession.close();
    }