[activemq-artemis] branch main updated: NO-JIRA Fixing intermittent failure in PagingTest

[clebertsuconic]

This is an automated email from the ASF dual-hosted git repository.

clebertsuconic pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git


The following commit(s) were added to refs/heads/main by this push:
     new cffc06aec8 NO-JIRA Fixing intermittent failure in PagingTest
cffc06aec8 is described below

commit cffc06aec85f20a561f8ba8d56854a8de8897550
Author: Clebert Suconic <clebertsuco...@apache.org>
AuthorDate: Sun Oct 23 12:16:23 2022 -0400

    NO-JIRA Fixing intermittent failure in PagingTest
---
 .../activemq/artemis/tests/integration/paging/PagingTest.java  | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git 
a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/paging/PagingTest.java
 
b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/paging/PagingTest.java
index 0752dc999b..0da8db5dd5 100644
--- 
a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/paging/PagingTest.java
+++ 
b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/paging/PagingTest.java
@@ -2492,7 +2492,7 @@ public class PagingTest extends ActiveMQTestBase {
    }
 
    @Test
-   public void testReceive() throws Exception {
+   public void testReceiveImmediate() throws Exception {
       clearDataRecreateServerDirs();
 
       Configuration config = 
createDefaultInVMConfig().setJournalSyncNonTransactional(false);
@@ -2557,15 +2557,15 @@ public class PagingTest extends ActiveMQTestBase {
 
       Wait.assertEquals(numberOfMessages, queue::getMessageCount);
 
-      int msgReceived = 0;
       ClientSession sessionConsumer = sf.createSession(false, false, false);
       sessionConsumer.start();
       ClientConsumer consumer = 
sessionConsumer.createConsumer(PagingTest.ADDRESS);
       for (int msgCount = 0; msgCount < numberOfMessages; msgCount++) {
          logger.debug("Received {}", msgCount);
-         msgReceived++;
-         Wait.assertTrue(() -> queue.peekFirstMessage() != null, 2000, 100);
-         ClientMessage msg = consumer.receive(1000);
+         ClientMessage msg = null;
+         for (int retry = 0; retry < 10 && msg == null; retry++) {
+            msg = consumer.receiveImmediate();
+         }
          if (msg == null) {
             logger.debug("It's null. leaving now");
             sessionConsumer.commit();