From 881ee54a911025bde416bf51efa1cbf33a376794 Mon Sep 17 00:00:00 2001 From: Robert Gemmell Date: Thu, 1 Mar 2012 00:01:14 +0000 Subject: QPID-3605: update test comments to reflect change in broker behaviour after the changes in this JIRA git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@1295348 13f79535-47bb-0310-9956-ffa450edef68 --- .../org/apache/qpid/server/persistent/NoLocalAfterRecoveryTest.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'java') diff --git a/java/systests/src/main/java/org/apache/qpid/server/persistent/NoLocalAfterRecoveryTest.java b/java/systests/src/main/java/org/apache/qpid/server/persistent/NoLocalAfterRecoveryTest.java index 9dfd3c912a..bcad59a1fa 100644 --- a/java/systests/src/main/java/org/apache/qpid/server/persistent/NoLocalAfterRecoveryTest.java +++ b/java/systests/src/main/java/org/apache/qpid/server/persistent/NoLocalAfterRecoveryTest.java @@ -66,6 +66,8 @@ public class NoLocalAfterRecoveryTest extends QpidBrokerTestCase // Check messages can be received as expected. connection.start(); + //As the no-local subscriber was on the same connection the messages were + //published on, tit will receive no messages as they will be discarded on the broker List received = receiveMessage(noLocalSubscriber, SEND_COUNT); assertEquals("No Local Subscriber Received messages", 0, received.size()); @@ -77,8 +79,8 @@ public class NoLocalAfterRecoveryTest extends QpidBrokerTestCase normalSubscriber.close(); connection.close(); - //We didn't receive the messages on the durable queue for the no-local subscriber - //so they are still on the broker. Restart the broker, prompting their recovery. + //Ensure the no-local subscribers messages were discarded by restarting the broker + //and reconnecting to the subscription to ensure they were not recovered. restartBroker(); Connection connection2 = getConnection(); -- cgit v1.2.1