Repository: qpid-jms Updated Branches: refs/heads/master 4d463bbb0 -> 9e3d5608e
add some tests for handling of Properties section to and reply-to fields Project: http://git-wip-us.apache.org/repos/asf/qpid-jms/repo Commit: http://git-wip-us.apache.org/repos/asf/qpid-jms/commit/fe790242 Tree: http://git-wip-us.apache.org/repos/asf/qpid-jms/tree/fe790242 Diff: http://git-wip-us.apache.org/repos/asf/qpid-jms/diff/fe790242 Branch: refs/heads/master Commit: fe79024294b9b161d506d3275762e8b93de975d6 Parents: 4d463bb Author: Robert Gemmell <[email protected]> Authored: Tue Oct 7 17:18:12 2014 +0100 Committer: Robert Gemmell <[email protected]> Committed: Tue Oct 7 17:18:12 2014 +0100 ---------------------------------------------------------------------- .../amqp/message/AmqpJmsMessageFacadeTest.java | 82 ++++++++++++++++++++ 1 file changed, 82 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/fe790242/qpid-jms-client/src/test/java/org/apache/qpid/jms/provider/amqp/message/AmqpJmsMessageFacadeTest.java ---------------------------------------------------------------------- diff --git a/qpid-jms-client/src/test/java/org/apache/qpid/jms/provider/amqp/message/AmqpJmsMessageFacadeTest.java b/qpid-jms-client/src/test/java/org/apache/qpid/jms/provider/amqp/message/AmqpJmsMessageFacadeTest.java index 58707ac..963fefd 100644 --- a/qpid-jms-client/src/test/java/org/apache/qpid/jms/provider/amqp/message/AmqpJmsMessageFacadeTest.java +++ b/qpid-jms-client/src/test/java/org/apache/qpid/jms/provider/amqp/message/AmqpJmsMessageFacadeTest.java @@ -37,6 +37,8 @@ import java.util.UUID; import javax.jms.JMSException; import javax.jms.MessageFormatException; +import javax.jms.Queue; +import javax.jms.Topic; import org.apache.qpid.jms.JmsDestination; import org.apache.qpid.jms.JmsQueue; @@ -656,6 +658,86 @@ public class AmqpJmsMessageFacadeTest { assertEquals("GroupSequence should be 0", 0, amqpMessageFacade.getGroupSequence()); } + // --- to field --- + + // Basic test to see things are wired up at all. See {@link AmqpDestinationHelperTest} + // for more comprehensive testing of the underlying bits. + + @Test + public void testSetGetDestination() { + String testToAddress = "myTestAddress"; + JmsTopic dest = new JmsTopic(testToAddress); + + AmqpJmsMessageFacade amqpMessageFacade = createNewMessageFacade(); + + Message underlyingMessage = amqpMessageFacade.getAmqpMessage(); + assertNull(underlyingMessage.getAddress()); + + amqpMessageFacade.setDestination(dest); + + assertNotNull(underlyingMessage.getAddress()); + assertEquals(testToAddress, underlyingMessage.getAddress()); + assertEquals(dest, amqpMessageFacade.getDestination()); + } + + @Test + public void testGetDestinationWithReceivedMessage() throws JMSException { + String testToAddress = "myTestAddress"; + + Message message = Proton.message(); + + Properties props = new Properties(); + props.setTo(testToAddress); + message.setProperties(props); + + AmqpJmsMessageFacade amqpMessageFacade = createReceivedMessageFacade(createMockAmqpConsumer(), message); + + JmsDestination dest = amqpMessageFacade.getDestination(); + //We didn't set any destination type annotations, so the consumer destination type will be used: a topic. + assertTrue(dest instanceof Topic); + assertEquals(testToAddress, ((Topic) dest).getTopicName()); + } + + // --- reply-to field --- + + // Basic test to see things are wired up at all. See {@link AmqpDestinationHelperTest} + // for more comprehensive testing of the underlying bits. + + @Test + public void testSetGetReplyTo() { + String testReplyToAddress = "myTestReplyTo"; + JmsTopic dest = new JmsTopic(testReplyToAddress); + + AmqpJmsMessageFacade amqpMessageFacade = createNewMessageFacade(); + + Message underlyingMessage = amqpMessageFacade.getAmqpMessage(); + assertNull(underlyingMessage.getReplyTo()); + + amqpMessageFacade.setReplyTo(dest); + + assertNotNull(underlyingMessage.getReplyTo()); + assertEquals(testReplyToAddress, underlyingMessage.getReplyTo()); + assertEquals(dest, amqpMessageFacade.getReplyTo()); + } + + @Test + public void testGetReplyToWithReceivedMessage() throws JMSException { + String testReplyToAddress = "myTestReplyTo"; + + Message message = Proton.message(); + + Properties props = new Properties(); + props.setReplyTo(testReplyToAddress); + message.setProperties(props); + + AmqpJmsMessageFacade amqpMessageFacade = createReceivedMessageFacade(createMockAmqpConsumer(), message); + + JmsDestination dest = amqpMessageFacade.getReplyTo(); + //We didn't set any destination type annotations, so the consumer destination type will be used: a topic. + assertTrue(dest instanceof Topic); + assertEquals(testReplyToAddress, ((Topic) dest).getTopicName()); + } + // --- message-id and correlation-id --- @Test --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
