You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by rg...@apache.org on 2019/05/27 09:51:19 UTC

[qpid-broker-j] branch master updated: QPID-8318 : Fix system test

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

rgodfrey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/qpid-broker-j.git


The following commit(s) were added to refs/heads/master by this push:
     new dcffe40  QPID-8318 : Fix system test
dcffe40 is described below

commit dcffe40f54366cb585344d0ec2d57e9dde6d62c8
Author: rgodfrey <rg...@apache.org>
AuthorDate: Mon May 27 11:50:30 2019 +0200

    QPID-8318 : Fix system test
---
 .../src/test/java/org/apache/qpid/tests/protocol/v0_8/QueueTest.java | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/systests/protocol-tests-amqp-0-8/src/test/java/org/apache/qpid/tests/protocol/v0_8/QueueTest.java b/systests/protocol-tests-amqp-0-8/src/test/java/org/apache/qpid/tests/protocol/v0_8/QueueTest.java
index 54a8f20..a5b5c26 100644
--- a/systests/protocol-tests-amqp-0-8/src/test/java/org/apache/qpid/tests/protocol/v0_8/QueueTest.java
+++ b/systests/protocol-tests-amqp-0-8/src/test/java/org/apache/qpid/tests/protocol/v0_8/QueueTest.java
@@ -430,15 +430,14 @@ public class QueueTest extends BrokerAdminUsingTestBase
                                                    .queue().purgeName(BrokerAdmin.TEST_QUEUE_NAME).purge()
                                                    .consumeResponse().getLatestResponse(QueuePurgeOkBody.class);
 
-            /* TODO purge currently always returns 0 */
-            //assertThat(response.getMessageCount(), is(equalTo(1L)));
+            assertThat(response.getMessageCount(), is(equalTo(1L)));
 
             QueueDeclareOkBody passive = interaction.queue()
                                                     .declareName(BrokerAdmin.TEST_QUEUE_NAME)
                                                     .declarePassive(true).declare()
                                                     .consumeResponse().getLatestResponse(QueueDeclareOkBody.class);
             assertThat(passive.getQueue(), is(equalTo(AMQShortString.valueOf(BrokerAdmin.TEST_QUEUE_NAME))));
-            assertThat(response.getMessageCount(), is(equalTo(0L)));
+            assertThat(passive.getMessageCount(), is(equalTo(0L)));
         }
     }
 


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