You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2012/01/18 09:59:28 UTC

svn commit: r1232790 - /camel/trunk/camel-core/src/test/java/org/apache/camel/management/ManagedRouteAddFromRouteTest.java

Author: davsclaus
Date: Wed Jan 18 08:59:28 2012
New Revision: 1232790

URL: http://svn.apache.org/viewvc?rev=1232790&view=rev
Log:
CAMEL-4911: SedaConsumer should not poll if CamelContext is starting.

Modified:
    camel/trunk/camel-core/src/test/java/org/apache/camel/management/ManagedRouteAddFromRouteTest.java

Modified: camel/trunk/camel-core/src/test/java/org/apache/camel/management/ManagedRouteAddFromRouteTest.java
URL: http://svn.apache.org/viewvc/camel/trunk/camel-core/src/test/java/org/apache/camel/management/ManagedRouteAddFromRouteTest.java?rev=1232790&r1=1232789&r2=1232790&view=diff
==============================================================================
--- camel/trunk/camel-core/src/test/java/org/apache/camel/management/ManagedRouteAddFromRouteTest.java (original)
+++ camel/trunk/camel-core/src/test/java/org/apache/camel/management/ManagedRouteAddFromRouteTest.java Wed Jan 18 08:59:28 2012
@@ -39,6 +39,10 @@ public class ManagedRouteAddFromRouteTes
         return new RouteBuilder() {
             @Override
             public void configure() throws Exception {
+                // put a message pre-early on the seda queue, to trigger the route, which
+                // then would add a 2nd route during CamelContext startup. This is a test
+                // to ensure the foo route is not started too soon, and thus adding the 2nd
+                // route works as expected
                 SedaEndpoint seda = context.getEndpoint("seda:start", SedaEndpoint.class);
                 seda.getQueue().put(new DefaultExchange(context));
 
@@ -70,7 +74,9 @@ public class ManagedRouteAddFromRouteTes
 
         MockEndpoint result = getMockEndpoint("mock:result");
         result.expectedMessageCount(1);
-        template.sendBody("seda:start", "Hello World");
+
+        // should route the message we put on the seda queue before
+
         result.assertIsSatisfied();
 
         // find the 2nd route