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 2009/07/03 12:10:31 UTC

svn commit: r790844 - in /camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel: component/dataset/ component/seda/ processor/ processor/aggregator/

Author: davsclaus
Date: Fri Jul  3 10:10:31 2009
New Revision: 790844

URL: http://svn.apache.org/viewvc?rev=790844&view=rev
Log:
Fixed unit test reported by TC.

Removed:
    camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/component/seda/SedaConsumerStartStopTest.java
Modified:
    camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/component/dataset/DataSetSedaTest.java
    camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/DelayInterceptorTest.java
    camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/PipelineConcurrentTest.java
    camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregateTimeoutOnlyTest.java

Modified: camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/component/dataset/DataSetSedaTest.java
URL: http://svn.apache.org/viewvc/camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/component/dataset/DataSetSedaTest.java?rev=790844&r1=790843&r2=790844&view=diff
==============================================================================
--- camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/component/dataset/DataSetSedaTest.java (original)
+++ camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/component/dataset/DataSetSedaTest.java Fri Jul  3 10:10:31 2009
@@ -27,9 +27,10 @@
  */
 public class DataSetSedaTest extends ContextTestSupport {
     private SimpleDataSet dataSet = new SimpleDataSet(200);
+    private String uri = "dataset:foo?produceDelay=3";
 
     public void test() throws Exception {
-        MockEndpoint endpoint = getMockEndpoint("dataset:foo");
+        MockEndpoint endpoint = getMockEndpoint(uri);
         endpoint.expectedMessageCount((int) dataSet.getSize());
 
         assertMockEndpointsSatisfied();
@@ -46,11 +47,11 @@
     protected RouteBuilder createRouteBuilder() throws Exception {
         return new RouteBuilder() {
             public void configure() throws Exception {
-                //TODO: remove this once the delegate processor supports async
                 errorHandler(noErrorHandler());
-                
-                from("dataset:foo").to("seda:queue:test?size=200");
-                from("seda:queue:test?size=200").to("dataset:foo");
+
+                from(uri).to("seda:test");
+
+                from("seda:test").to(uri);
             }
         };
     }

Modified: camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/DelayInterceptorTest.java
URL: http://svn.apache.org/viewvc/camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/DelayInterceptorTest.java?rev=790844&r1=790843&r2=790844&view=diff
==============================================================================
--- camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/DelayInterceptorTest.java (original)
+++ camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/DelayInterceptorTest.java Fri Jul  3 10:10:31 2009
@@ -36,7 +36,7 @@
         }
         long delta = System.currentTimeMillis() - start;
         assertTrue("Should be slower to run: " + delta, delta > 4000);
-        assertTrue("Should not take that long to run: " + delta, delta < 9000);
+        assertTrue("Should not take that long to run: " + delta, delta < 15000);
     }
 
     @Override

Modified: camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/PipelineConcurrentTest.java
URL: http://svn.apache.org/viewvc/camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/PipelineConcurrentTest.java?rev=790844&r1=790843&r2=790844&view=diff
==============================================================================
--- camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/PipelineConcurrentTest.java (original)
+++ camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/PipelineConcurrentTest.java Fri Jul  3 10:10:31 2009
@@ -30,8 +30,10 @@
  */
 public class PipelineConcurrentTest extends ContextTestSupport {
 
+    private String uri = "seda:in?size=2000&concurrentConsumers=10";
+
     public void testConcurrentPipeline() throws Exception {
-        int total = 10000;
+        int total = 2000;
         final int group = total / 20;
         MockEndpoint mock = getMockEndpoint("mock:result");
         mock.expectedMessageCount(total);
@@ -51,7 +53,7 @@
                         } catch (InterruptedException e) {
                             // ignore
                         }
-                        template.sendBody("seda:in?size=10000", "" + (start + i));
+                        template.sendBody(uri, "" + (start + i));
                     }
                 }
             });
@@ -64,11 +66,10 @@
     protected RouteBuilder createRouteBuilder() throws Exception {
         return new RouteBuilder() {
             public void configure() throws Exception {
-                // to force any exceptions coming forward imeddiately
+                // to force any exceptions coming forward immediately
                 errorHandler(noErrorHandler());
 
-                from("seda:in?size=10000")
-                    .thread(10)
+                from(uri)
                     .pipeline("direct:do", "mock:result");
 
                 from("direct:do")

Modified: camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregateTimeoutOnlyTest.java
URL: http://svn.apache.org/viewvc/camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregateTimeoutOnlyTest.java?rev=790844&r1=790843&r2=790844&view=diff
==============================================================================
--- camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregateTimeoutOnlyTest.java (original)
+++ camel/branches/camel-1.x/camel-core/src/test/java/org/apache/camel/processor/aggregator/AggregateTimeoutOnlyTest.java Fri Jul  3 10:10:31 2009
@@ -32,7 +32,7 @@
         // by default the use latest aggregatation strategy is used so we get message 9
         result.expectedBodiesReceived("Message 9");
         // should take 3 seconds to complete this one
-        result.setMinimumResultWaitTime(3000);
+        result.setMinimumResultWaitTime(2500);
 
         for (int i = 0; i < 10; i++) {
             template.sendBodyAndHeader("direct:start", "Message " + i, "id", "1");