diff options
Diffstat (limited to 'java/broker-plugins/management-jmx/src/test')
2 files changed, 42 insertions, 11 deletions
diff --git a/java/broker-plugins/management-jmx/src/test/java/org/apache/qpid/server/jmx/mbeans/QueueMBeanTest.java b/java/broker-plugins/management-jmx/src/test/java/org/apache/qpid/server/jmx/mbeans/QueueMBeanTest.java index f94c206512..2874168ddf 100644 --- a/java/broker-plugins/management-jmx/src/test/java/org/apache/qpid/server/jmx/mbeans/QueueMBeanTest.java +++ b/java/broker-plugins/management-jmx/src/test/java/org/apache/qpid/server/jmx/mbeans/QueueMBeanTest.java @@ -40,6 +40,7 @@ import org.apache.qpid.server.jmx.ManagedObjectRegistry; import org.apache.qpid.server.jmx.mbeans.QueueMBean.GetMessageVisitor; import org.apache.qpid.server.message.ServerMessage; import org.apache.qpid.server.model.Exchange; +import org.apache.qpid.server.model.ExclusivityPolicy; import org.apache.qpid.server.model.LifetimePolicy; import org.apache.qpid.server.model.Queue; import org.apache.qpid.server.model.Statistics; @@ -154,7 +155,7 @@ public class QueueMBeanTest extends QpidTestCase public void testIsAutoDelete() throws Exception { - when(_mockQueue.getLifetimePolicy()).thenReturn(LifetimePolicy.AUTO_DELETE); + when(_mockQueue.getLifetimePolicy()).thenReturn(LifetimePolicy.DELETE_ON_NO_OUTBOUND_LINKS); assertTrue(_queueMBean.isAutoDelete()); } @@ -224,22 +225,31 @@ public class QueueMBeanTest extends QpidTestCase testSetAttribute("flowResumeCapacity", Queue.QUEUE_FLOW_RESUME_SIZE_BYTES,1048576l , 2097152l); } + + /********** Other attributes **********/ + + public void testIsExclusive() throws Exception { - assertAttribute("exclusive", Boolean.TRUE, Queue.EXCLUSIVE); + when(_mockQueue.getAttribute(Queue.EXCLUSIVE)).thenReturn(ExclusivityPolicy.CONTAINER); + MBeanTestUtils.assertMBeanAttribute(_queueMBean, "exclusive", true); } public void testIsNotExclusive() throws Exception { - assertAttribute("exclusive", Boolean.FALSE, Queue.EXCLUSIVE); + when(_mockQueue.getAttribute(Queue.EXCLUSIVE)).thenReturn(ExclusivityPolicy.NONE); + MBeanTestUtils.assertMBeanAttribute(_queueMBean, "exclusive", false); } public void testSetExclusive() throws Exception { - testSetAttribute("exclusive", Queue.EXCLUSIVE, Boolean.FALSE , Boolean.TRUE); - } + when(_mockQueue.getAttribute(Queue.EXCLUSIVE)).thenReturn(ExclusivityPolicy.NONE); - /********** Other attributes **********/ + MBeanTestUtils.setMBeanAttribute(_queueMBean, "exclusive", Boolean.TRUE); + + verify(_mockQueue).setAttribute(Queue.EXCLUSIVE, ExclusivityPolicy.NONE, ExclusivityPolicy.CONTAINER); + + } public void testGetAlternateExchange() { diff --git a/java/broker-plugins/management-jmx/src/test/java/org/apache/qpid/server/jmx/mbeans/VirtualHostManagerMBeanTest.java b/java/broker-plugins/management-jmx/src/test/java/org/apache/qpid/server/jmx/mbeans/VirtualHostManagerMBeanTest.java index 4240dd5280..2dc2cb8d3b 100644 --- a/java/broker-plugins/management-jmx/src/test/java/org/apache/qpid/server/jmx/mbeans/VirtualHostManagerMBeanTest.java +++ b/java/broker-plugins/management-jmx/src/test/java/org/apache/qpid/server/jmx/mbeans/VirtualHostManagerMBeanTest.java @@ -39,6 +39,7 @@ import org.apache.qpid.server.model.State; import org.apache.qpid.server.model.VirtualHost; import org.apache.qpid.server.queue.AMQQueueFactory; import org.apache.qpid.server.queue.QueueArgumentsConverter; +import org.mockito.ArgumentCaptor; public class VirtualHostManagerMBeanTest extends TestCase { @@ -68,8 +69,15 @@ public class VirtualHostManagerMBeanTest extends TestCase public void testCreateQueueWithNoOwner() throws Exception { _virtualHostManagerMBean.createNewQueue(TEST_QUEUE_NAME, null, true); + ArgumentCaptor<Map> argsCaptor = ArgumentCaptor.forClass(Map.class); + + verify(_mockVirtualHost).createQueue(argsCaptor.capture()); + + Map actualAttributes = argsCaptor.getValue(); + assertEquals(TEST_QUEUE_NAME, actualAttributes.get(Queue.NAME)); + assertEquals(Boolean.TRUE,actualAttributes.get(Queue.DURABLE)); + assertEquals(null,actualAttributes.get(Queue.OWNER)); - verify(_mockVirtualHost).createQueue(TEST_QUEUE_NAME, State.ACTIVE, true, false, LifetimePolicy.PERMANENT, 0, EMPTY_ARGUMENT_MAP); } /** @@ -79,9 +87,15 @@ public class VirtualHostManagerMBeanTest extends TestCase public void testCreateQueueWithOwnerMappedThroughToDescription() throws Exception { _virtualHostManagerMBean.createNewQueue(TEST_QUEUE_NAME, TEST_OWNER, true); + ArgumentCaptor<Map> argsCaptor = ArgumentCaptor.forClass(Map.class); + + verify(_mockVirtualHost).createQueue(argsCaptor.capture()); - Map<String, Object> expectedArguments = Collections.singletonMap(Queue.DESCRIPTION, (Object)TEST_OWNER); - verify(_mockVirtualHost).createQueue(TEST_QUEUE_NAME, State.ACTIVE, true, false, LifetimePolicy.PERMANENT, 0, expectedArguments); + Map actualAttributes = argsCaptor.getValue(); + assertEquals(TEST_QUEUE_NAME,actualAttributes.get(Queue.NAME)); + assertEquals(Boolean.TRUE,actualAttributes.get(Queue.DURABLE)); + assertEquals(null,actualAttributes.get(Queue.OWNER)); + assertEquals(TEST_OWNER, actualAttributes.get(Queue.DESCRIPTION)); } public void testCreateQueueWithOwnerAndDescriptionDiscardsOwner() throws Exception @@ -89,8 +103,15 @@ public class VirtualHostManagerMBeanTest extends TestCase Map<String, Object> arguments = Collections.singletonMap(QueueArgumentsConverter.X_QPID_DESCRIPTION, (Object)TEST_DESCRIPTION); _virtualHostManagerMBean.createNewQueue(TEST_QUEUE_NAME, TEST_OWNER, true, arguments); - Map<String, Object> expectedArguments = Collections.singletonMap(Queue.DESCRIPTION, (Object)TEST_DESCRIPTION); - verify(_mockVirtualHost).createQueue(TEST_QUEUE_NAME, State.ACTIVE, true, false, LifetimePolicy.PERMANENT, 0, expectedArguments); + ArgumentCaptor<Map> argsCaptor = ArgumentCaptor.forClass(Map.class); + + verify(_mockVirtualHost).createQueue(argsCaptor.capture()); + + Map actualAttributes = argsCaptor.getValue(); + assertEquals(TEST_QUEUE_NAME,actualAttributes.get(Queue.NAME)); + assertEquals(Boolean.TRUE,actualAttributes.get(Queue.DURABLE)); + assertEquals(null,actualAttributes.get(Queue.OWNER)); + assertEquals(TEST_DESCRIPTION, actualAttributes.get(Queue.DESCRIPTION)); } public void testDeleteQueue() throws Exception |