You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ro...@apache.org on 2014/10/07 12:26:59 UTC

[5/6] git commit: add some tests for handling of Properties section creation-time

add some tests for handling of Properties section creation-time


Project: http://git-wip-us.apache.org/repos/asf/qpid-jms/repo
Commit: http://git-wip-us.apache.org/repos/asf/qpid-jms/commit/6d55ce2e
Tree: http://git-wip-us.apache.org/repos/asf/qpid-jms/tree/6d55ce2e
Diff: http://git-wip-us.apache.org/repos/asf/qpid-jms/diff/6d55ce2e

Branch: refs/heads/master
Commit: 6d55ce2e6454a705433eb3f08b869756dd3c3e7b
Parents: a3db689
Author: Robert Gemmell <ro...@apache.org>
Authored: Tue Oct 7 10:56:01 2014 +0100
Committer: Robert Gemmell <ro...@apache.org>
Committed: Tue Oct 7 10:56:01 2014 +0100

----------------------------------------------------------------------
 .../amqp/message/AmqpJmsMessageFacadeTest.java  | 42 ++++++++++++++++++++
 1 file changed, 42 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/6d55ce2e/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 7bd81e3..3a75d1a 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
@@ -749,6 +749,48 @@ public class AmqpJmsMessageFacadeTest {
         assertEquals("Expected creation-time field to be set on new Properties section", new Date(expected), amqpMessageFacade.getAmqpMessage().getProperties().getCreationTime());
     }
 
+    @Test
+    public void testGetTimestampIsZeroForNewMessage() {
+        AmqpJmsMessageFacade amqpMessageFacade = createNewMessageFacade();
+
+        assertEquals("Expected no timestamp", 0, amqpMessageFacade.getTimestamp());
+    }
+
+    @Test
+    public void testSetTimestampOnNewMessage() {
+        Long timestamp = System.currentTimeMillis();
+
+        AmqpJmsMessageFacade amqpMessageFacade = createNewMessageFacade();
+
+        amqpMessageFacade.setTimestamp(timestamp);
+
+        assertEquals("Expected creation-time field to be set", timestamp.longValue(), amqpMessageFacade.getAmqpMessage().getProperties().getCreationTime().getTime());
+        assertEquals("Expected timestamp", timestamp.longValue(), amqpMessageFacade.getTimestamp());
+    }
+
+    @Test
+    public void testSetTimestampZeroOnNewMessageDoesNotCreatePropertiesSection() {
+        AmqpJmsMessageFacade amqpMessageFacade = createNewMessageFacade();
+
+        amqpMessageFacade.setTimestamp(0);
+
+        assertNull("underlying message should have no properties section", amqpMessageFacade.getAmqpMessage().getProperties());
+        assertEquals("Timestamp should not be set", 0, amqpMessageFacade.getTimestamp());
+    }
+
+    @Test
+    public void testSetTimestampZeroOnMessageWithExistingTimestampClearsCreationTimeField() {
+        Long timestamp = System.currentTimeMillis();
+
+        AmqpJmsMessageFacade amqpMessageFacade = createNewMessageFacade();
+        amqpMessageFacade.setTimestamp(timestamp);
+
+        amqpMessageFacade.setTimestamp(0);
+
+        assertNull("Expected creation-time to be null", amqpMessageFacade.getAmqpMessage().getProperties().getCreationTime());
+        assertEquals("Expected no timestamp", 0, amqpMessageFacade.getTimestamp());
+    }
+
     // --- absolute-expiry-time field  ---
 
     @Test


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org