activemq-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From clebertsuco...@apache.org
Subject activemq-artemis git commit: ARTEMIS-1256 PagingOMETest.testPageCleanup fails
Date Wed, 28 Jun 2017 14:09:53 GMT
Repository: activemq-artemis
Updated Branches:
  refs/heads/1.x bc815004d -> c5e88c5e2


ARTEMIS-1256 PagingOMETest.testPageCleanup fails

Using a Wait condition.

This closes #1370

(cherry picked from commit 5c2144b7826c25e8510f870b588e873685291d74)


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

Branch: refs/heads/1.x
Commit: c5e88c5e2da7a8efa708195f9c420bbd7fafe8a0
Parents: bc81500
Author: Clebert Suconic <clebertsuconic@apache.org>
Authored: Tue Jun 27 12:48:33 2017 -0400
Committer: Clebert Suconic <clebertsuconic@apache.org>
Committed: Wed Jun 28 10:09:44 2017 -0400

----------------------------------------------------------------------
 .../activemq/artemis/tests/extras/byteman/PagingOMETest.java      | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/activemq-artemis/blob/c5e88c5e/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingOMETest.java
----------------------------------------------------------------------
diff --git a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingOMETest.java
b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingOMETest.java
index cdf1c56..7ef4e9f 100644
--- a/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingOMETest.java
+++ b/tests/extra-tests/src/test/java/org/apache/activemq/artemis/tests/extras/byteman/PagingOMETest.java
@@ -33,6 +33,7 @@ import org.apache.activemq.artemis.core.server.ActiveMQServer;
 import org.apache.activemq.artemis.core.server.Queue;
 import org.apache.activemq.artemis.core.settings.impl.AddressSettings;
 import org.apache.activemq.artemis.tests.util.ActiveMQTestBase;
+import org.apache.activemq.artemis.tests.util.Wait;
 import org.jboss.byteman.contrib.bmunit.BMRule;
 import org.jboss.byteman.contrib.bmunit.BMRules;
 import org.jboss.byteman.contrib.bmunit.BMUnitRunner;
@@ -145,7 +146,7 @@ public class PagingOMETest extends ActiveMQTestBase {
 
       session.start();
 
-      assertEquals(numberOfMessages, queue.getMessageCount());
+      Assert.assertTrue(Wait.waitFor(() -> numberOfMessages == queue.getMessageCount()));
 
       // The consumer has to be created after the queue.getMessageCount assertion
       // otherwise delivery could alter the messagecount and give us a false failure


Mime
View raw message