ARTEMIS-5815 countMessages(Filter) should not limit the browsing operation

Author: clebertsuconic

artemis-server/src/main/java/org/apache/activemq/artemis/core/management/impl/QueueControlImpl.java

@@ -1091,11 +1091,9 @@ private Map<String, Long> internalCountMessages(final String filterStr, final St
             if (filter == null && groupByProperty == null) {
                result.put(null, getMessageCount());
             } else {
-               final int limit = addressSettingsRepository.getMatch(address).getManagementBrowsePageSize();
-               int count = 0;
                try (LinkedListIterator<MessageReference> iterator = queue.browserIterator()) {
                   try {
-                     while (iterator.hasNext() && count++ < limit) {
+                     while (iterator.hasNext()) {
                         Message message = iterator.next().getMessage();
                         internalComputeMessage(result, filter, groupByProperty, message);
                      }

tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/management/QueueControlTest.java

@@ -4818,7 +4818,7 @@ public void testBrowseLimitOnListBrowseAndFilteredCount() throws Exception {
       assertEquals(5, array.size());
 
       // filer could match all
-      assertEquals(5, queueControl.countMessages("AMQSize > 0"));
+      assertEquals(10, queueControl.countMessages("AMQSize > 0"));
 
       session.deleteQueue(queue);
    }