From 4ff1d4ca6707275d1306d7f94c79b5d066a548ce Mon Sep 17 00:00:00 2001 From: Martin Ritchie Date: Thu, 9 Oct 2008 17:43:41 +0000 Subject: QPID-1331 : Modified the BrowserSubscription to be consistent with the NoAck Subscription. Added Test QueueBrowserUsesNoAckTest to validate the change. Note that the Credit Manager Suspends the subscriber not the channel when credit is exhausted. JIRA to follow. So to check if the subscription was suspended I needed to make a MockChannel and give it access to the subscriber map in the Channel. The test also needed to be able to interrogate the state of the Subscription which was not part of the Subscription interface, but was used by all subscriptions. So promoted to the interface and implemented the stubs in the various helper/test classes. Added the ability to browse() via the InternalBrokerBaseCase and prevented a NPE when there were no messages returned via getDelivers. git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk/qpid@703212 13f79535-47bb-0310-9956-ffa450edef68 --- .../java/org/apache/qpid/server/queue/SubscriptionTestHelper.java | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'java/systests') diff --git a/java/systests/src/main/java/org/apache/qpid/server/queue/SubscriptionTestHelper.java b/java/systests/src/main/java/org/apache/qpid/server/queue/SubscriptionTestHelper.java index eed60a1a7c..c20c0bc579 100644 --- a/java/systests/src/main/java/org/apache/qpid/server/queue/SubscriptionTestHelper.java +++ b/java/systests/src/main/java/org/apache/qpid/server/queue/SubscriptionTestHelper.java @@ -110,6 +110,12 @@ public class SubscriptionTestHelper implements Subscription { //To change body of implemented methods use File | Settings | File Templates. } + + @Override + public State getState() + { + return null; //To change body of implemented methods use File | Settings | File Templates. + } public QueueEntry getLastSeenEntry() { -- cgit v1.2.1