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/01/21 18:33:47 UTC

svn commit: r1436505 - /camel/trunk/camel-core/src/test/java/org/apache/camel/processor/ValidationTest.java

Author: bvahdat
Date: Mon Jan 21 17:33:47 2013
New Revision: 1436505

URL: http://svn.apache.org/viewvc?rev=1436505&view=rev
Log:
CAMEL-5983: Corrected the comment.

Modified:
    camel/trunk/camel-core/src/test/java/org/apache/camel/processor/ValidationTest.java

Modified: camel/trunk/camel-core/src/test/java/org/apache/camel/processor/ValidationTest.java
URL: http://svn.apache.org/viewvc/camel/trunk/camel-core/src/test/java/org/apache/camel/processor/ValidationTest.java?rev=1436505&r1=1436504&r2=1436505&view=diff
==============================================================================
--- camel/trunk/camel-core/src/test/java/org/apache/camel/processor/ValidationTest.java (original)
+++ camel/trunk/camel-core/src/test/java/org/apache/camel/processor/ValidationTest.java Mon Jan 21 17:33:47 2013
@@ -50,10 +50,11 @@ public class ValidationTest extends Cont
         try {
             template.sendBodyAndHeader("direct:start", "<invalid/>", "foo", "notMatchedHeaderValue");
         } catch (RuntimeCamelException e) {
-            // the expected empty catch block here is not intended for this class itself but the drived
-            // ones e.g. ValidationWithErrorInHandleAndFinallyBlockTest where noErrorHandler() is being
-            // installed. this's also why there's no fail("Should have thrown an exception") call here
-            // right after template.sendBodyAndHeader()
+            // the expected empty catch block here is not intended for this class itself but the subclasses
+            // e.g. ValidationWithErrorInHandleAndFinallyBlockTest where noErrorHandler() is being installed.
+            // this's also why there's no fail("Should have thrown an exception") call here right after
+            // template.sendBodyAndHeader() call as RuntimeCamelException will be not thrown by *all* subclasses
+            // but only by some of them.
         }
 
         assertMockEndpointsSatisfied();