You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by bv...@apache.org on 2013/10/30 23:42:43 UTC

git commit: Re-disabled the two itests failing on the CI-Server again, even after the recent upgrade to AMQ 5.9.

Updated Branches:
  refs/heads/master 4eb5a2921 -> aaf3eb917


Re-disabled the two itests failing on the CI-Server again, even after the recent upgrade to AMQ 5.9. 

Project: http://git-wip-us.apache.org/repos/asf/camel/repo
Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/aaf3eb91
Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/aaf3eb91
Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/aaf3eb91

Branch: refs/heads/master
Commit: aaf3eb9174df2bc1c897d8f6cf5575b2aa9a5c19
Parents: 4eb5a29
Author: Babak Vahdat <bv...@apache.org>
Authored: Wed Oct 30 23:42:37 2013 +0100
Committer: Babak Vahdat <bv...@apache.org>
Committed: Wed Oct 30 23:42:37 2013 +0100

----------------------------------------------------------------------
 .../itest/sql/FromJmsToJdbcIdempotentConsumerToJmsTest.java     | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/aaf3eb91/tests/camel-itest/src/test/java/org/apache/camel/itest/sql/FromJmsToJdbcIdempotentConsumerToJmsTest.java
----------------------------------------------------------------------
diff --git a/tests/camel-itest/src/test/java/org/apache/camel/itest/sql/FromJmsToJdbcIdempotentConsumerToJmsTest.java b/tests/camel-itest/src/test/java/org/apache/camel/itest/sql/FromJmsToJdbcIdempotentConsumerToJmsTest.java
index bbd04eb..a152ac0 100644
--- a/tests/camel-itest/src/test/java/org/apache/camel/itest/sql/FromJmsToJdbcIdempotentConsumerToJmsTest.java
+++ b/tests/camel-itest/src/test/java/org/apache/camel/itest/sql/FromJmsToJdbcIdempotentConsumerToJmsTest.java
@@ -28,6 +28,7 @@ import org.apache.camel.builder.RouteBuilder;
 import org.apache.camel.spi.IdempotentRepository;
 import org.apache.camel.test.spring.CamelSpringTestSupport;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.springframework.context.support.AbstractApplicationContext;
 import org.springframework.context.support.ClassPathXmlApplicationContext;
@@ -87,6 +88,7 @@ public class FromJmsToJdbcIdempotentConsumerToJmsTest extends CamelSpringTestSup
         assertEquals("DONE-A", out);
     }
 
+    @Ignore("see the TODO below")
     @Test
     public void testJmsToJdbcJmsRollbackAtA() throws Exception {
         checkInitialState();
@@ -94,6 +96,7 @@ public class FromJmsToJdbcIdempotentConsumerToJmsTest extends CamelSpringTestSup
         // use a notify to know that after 1+6 (1 original + 6 redelivery) attempts from AcitveMQ
         NotifyBuilder notify = new NotifyBuilder(context).whenDone(7).create();
 
+        // TODO: occasionally we get only 6 instead of 7 expected exchanges which's most probably an issue in ActiveMQ itself
         getMockEndpoint("mock:a").expectedMessageCount(7);
         // force exception to occur at mock a
         getMockEndpoint("mock:a").whenAnyExchangeReceived(new Processor() {
@@ -118,6 +121,7 @@ public class FromJmsToJdbcIdempotentConsumerToJmsTest extends CamelSpringTestSup
         assertEquals("A", consumer.receiveBody("activemq:queue:ActiveMQ.DLQ", 3000));
     }
 
+    @Ignore("see the TODO below")
     @Test
     public void testJmsToJdbcJmsRollbackAtB() throws Exception {
         checkInitialState();
@@ -125,6 +129,7 @@ public class FromJmsToJdbcIdempotentConsumerToJmsTest extends CamelSpringTestSup
         // use a notify to know that after 1+6 (1 original + 6 redelivery) attempts from AcitveMQ
         NotifyBuilder notify = new NotifyBuilder(context).whenDone(7).create();
 
+        // TODO: occasionally we get only 6 instead of 7 expected exchanges which's most probably an issue in ActiveMQ itself
         getMockEndpoint("mock:a").expectedMessageCount(7);
         getMockEndpoint("mock:b").expectedMessageCount(7);
         // force exception to occur at mock b