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/09/30 16:32:30 UTC

git commit: enable test again after applying fix to proton-jms

Repository: qpid-jms
Updated Branches:
  refs/heads/master 065000dd7 -> 560e6fbd6


enable test again after applying fix to proton-jms


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

Branch: refs/heads/master
Commit: 560e6fbd68246a461fe360c83542f176f6609dd8
Parents: 065000d
Author: Robert Gemmell <ro...@apache.org>
Authored: Tue Sep 30 15:27:36 2014 +0100
Committer: Robert Gemmell <ro...@apache.org>
Committed: Tue Sep 30 15:27:53 2014 +0100

----------------------------------------------------------------------
 .../apache/qpid/jms/transactions/JmsTransactedConsumerTest.java  | 4 ----
 1 file changed, 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/qpid-jms/blob/560e6fbd/qpid-jms-interop-tests/qpid-jms-activemq-tests/src/test/java/org/apache/qpid/jms/transactions/JmsTransactedConsumerTest.java
----------------------------------------------------------------------
diff --git a/qpid-jms-interop-tests/qpid-jms-activemq-tests/src/test/java/org/apache/qpid/jms/transactions/JmsTransactedConsumerTest.java b/qpid-jms-interop-tests/qpid-jms-activemq-tests/src/test/java/org/apache/qpid/jms/transactions/JmsTransactedConsumerTest.java
index 14632ed..fdc3c3a 100644
--- a/qpid-jms-interop-tests/qpid-jms-activemq-tests/src/test/java/org/apache/qpid/jms/transactions/JmsTransactedConsumerTest.java
+++ b/qpid-jms-interop-tests/qpid-jms-activemq-tests/src/test/java/org/apache/qpid/jms/transactions/JmsTransactedConsumerTest.java
@@ -168,10 +168,6 @@ public class JmsTransactedConsumerTest extends AmqpTestSupport {
         assertEquals(0, proxy.getQueueSize());
     }
 
-    @Ignore // TODO: failing because the broker appears to be setting the AMQP
-    // delivery-count header too high after the first rollback. It
-    // arrives null initially (i.e, default 0), then jumps to 2 causing
-    // JMSXDeliveryCount to be 3 rather than 2.
     @Test(timeout=60000)
     public void testJMSXDeliveryCount() throws Exception {
         sendToAmqQueue(1);


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