You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by bh...@apache.org on 2007/03/09 15:10:35 UTC

svn commit: r516412 - in /incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server: exchange/AbstractHeadersExchangeTestBase.java protocol/AMQProtocolSessionMBeanTest.java queue/MockProtocolSession.java

Author: bhupendrab
Date: Fri Mar  9 06:10:34 2007
New Revision: 516412

URL: http://svn.apache.org/viewvc?view=rev&rev=516412
Log:
Fixed the test failures due to changes in the broker code

Modified:
    incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/exchange/AbstractHeadersExchangeTestBase.java
    incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/protocol/AMQProtocolSessionMBeanTest.java
    incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/queue/MockProtocolSession.java

Modified: incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/exchange/AbstractHeadersExchangeTestBase.java
URL: http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/exchange/AbstractHeadersExchangeTestBase.java?view=diff&rev=516412&r1=516411&r2=516412
==============================================================================
--- incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/exchange/AbstractHeadersExchangeTestBase.java (original)
+++ incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/exchange/AbstractHeadersExchangeTestBase.java Fri Mar  9 06:10:34 2007
@@ -158,7 +158,6 @@
         private boolean _mandatory;
         private AMQShortString _routingKey;
 
-
         public MessagePublishInfoImpl(AMQShortString routingKey)
         {
             _routingKey = routingKey;
@@ -217,31 +216,7 @@
 
     static MessagePublishInfo getPublishRequest(final String id)
     {
-        MessagePublishInfo request = new MessagePublishInfo()
-        {
-
-            public AMQShortString getExchange()
-            {
-                return null;
-            }
-
-            public boolean isImmediate()
-            {
-                return false;
-            }
-
-            public boolean isMandatory()
-            {
-                return false;
-            }
-
-            public AMQShortString getRoutingKey()
-            {
-                return new AMQShortString(id);
-            }
-        };
-                                                      
-        return request;
+        return new MessagePublishInfoImpl(null, false, false, new AMQShortString(id));
     }
 
     static ContentHeaderBody getContentHeader(FieldTable headers)

Modified: incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/protocol/AMQProtocolSessionMBeanTest.java
URL: http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/protocol/AMQProtocolSessionMBeanTest.java?view=diff&rev=516412&r1=516411&r2=516412
==============================================================================
--- incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/protocol/AMQProtocolSessionMBeanTest.java (original)
+++ incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/protocol/AMQProtocolSessionMBeanTest.java Fri Mar  9 06:10:34 2007
@@ -24,10 +24,7 @@
 import org.apache.qpid.server.virtualhost.VirtualHost;
 import org.apache.qpid.server.registry.IApplicationRegistry;
 import org.apache.qpid.server.registry.ApplicationRegistry;
-import org.apache.qpid.server.exchange.DefaultExchangeFactory;
-import org.apache.qpid.server.exchange.DefaultExchangeRegistry;
 import org.apache.qpid.server.exchange.ExchangeRegistry;
-import org.apache.qpid.server.queue.DefaultQueueRegistry;
 import org.apache.qpid.server.queue.QueueRegistry;
 import org.apache.qpid.server.queue.AMQQueue;
 import org.apache.qpid.server.store.MessageStore;
@@ -42,14 +39,10 @@
  */
 public class AMQProtocolSessionMBeanTest   extends TestCase
 {
-    private IoSession _mockIOSession;
     private MessageStore _messageStore = new SkeletonMessageStore();
     private AMQMinaProtocolSession _protocolSession;
     private AMQChannel _channel;
-    private QueueRegistry _queueRegistry;
-    private ExchangeRegistry _exchangeRegistry;
     private AMQProtocolSessionMBean _mbean;
-    private VirtualHost _virtualHost;
 
     public void testChannels() throws Exception
     {
@@ -57,7 +50,10 @@
         int channelCount = _mbean.channels().size();
         assertTrue(channelCount == 1);
         AMQQueue queue = new org.apache.qpid.server.queue.AMQQueue(new AMQShortString("testQueue_" + System.currentTimeMillis()),
-                                                            false, new AMQShortString("test"), true, _virtualHost);
+                                                                   false,
+                                                                   new AMQShortString("test"),
+                                                                   true,
+                                                                   _protocolSession.getVirtualHost());
         AMQChannel channel = new AMQChannel(_protocolSession,2, _messageStore, null);
         channel.setDefaultQueue(queue);
         _protocolSession.addChannel(channel);
@@ -111,11 +107,11 @@
         super.setUp();
 
         IApplicationRegistry appRegistry = ApplicationRegistry.getInstance();
-        _virtualHost = appRegistry.getVirtualHostRegistry().getVirtualHost("test");
-        _queueRegistry = _virtualHost.getQueueRegistry();
-        _exchangeRegistry = _virtualHost.getExchangeRegistry();
-        _mockIOSession = new MockIoSession();
-        _protocolSession = new AMQMinaProtocolSession(_mockIOSession, appRegistry.getVirtualHostRegistry(), new AMQCodecFactory(true));
+        _protocolSession = new AMQMinaProtocolSession(new MockIoSession(),
+                                                      appRegistry.getVirtualHostRegistry(),
+                                                      new AMQCodecFactory(true),
+                                                      null);
+        _protocolSession.setVirtualHost(appRegistry.getVirtualHostRegistry().getVirtualHost("test"));
         _channel = new AMQChannel(_protocolSession,1, _messageStore, null);
         _protocolSession.addChannel(_channel);
         _mbean = (AMQProtocolSessionMBean)_protocolSession.getManagedObject();

Modified: incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/queue/MockProtocolSession.java
URL: http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/queue/MockProtocolSession.java?view=diff&rev=516412&r1=516411&r2=516412
==============================================================================
--- incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/queue/MockProtocolSession.java (original)
+++ incubator/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/queue/MockProtocolSession.java Fri Mar  9 06:10:34 2007
@@ -27,6 +27,7 @@
 import org.apache.qpid.framing.VersionSpecificRegistry;
 import org.apache.qpid.server.AMQChannel;
 import org.apache.qpid.server.output.ProtocolOutputConverter;
+import org.apache.qpid.server.output.ProtocolOutputConverterRegistry;
 import org.apache.qpid.server.virtualhost.VirtualHost;
 import org.apache.qpid.server.protocol.AMQProtocolSession;
 import org.apache.qpid.server.store.MessageStore;
@@ -173,7 +174,7 @@
 
     public ProtocolOutputConverter getProtocolOutputConverter()
     {
-        return null;  //To change body of implemented methods use File | Settings | File Templates.
+        return ProtocolOutputConverterRegistry.getConverter(this);
     }
 
     public byte getProtocolMajorVersion()