summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRobert Gemmell <robbie@apache.org>2012-03-01 00:01:14 +0000
committerRobert Gemmell <robbie@apache.org>2012-03-01 00:01:14 +0000
commit13a449fb5f416f83e242492bc4610f5f4dd5bbe4 (patch)
tree7721b5c840595cdeca42d1dbb65e2dc5625970e9
parente7d54437156176ffac3b576749ec2e91bcd55abb (diff)
downloadqpid-python-13a449fb5f416f83e242492bc4610f5f4dd5bbe4.tar.gz
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@1295348 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r--qpid/java/systests/src/main/java/org/apache/qpid/server/persistent/NoLocalAfterRecoveryTest.java6
1 files changed, 4 insertions, 2 deletions
diff --git a/qpid/java/systests/src/main/java/org/apache/qpid/server/persistent/NoLocalAfterRecoveryTest.java b/qpid/java/systests/src/main/java/org/apache/qpid/server/persistent/NoLocalAfterRecoveryTest.java
index 9dfd3c912a..bcad59a1fa 100644
--- a/qpid/java/systests/src/main/java/org/apache/qpid/server/persistent/NoLocalAfterRecoveryTest.java
+++ b/qpid/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<Message> 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();