Repository: camel
Updated Branches:
  refs/heads/master a0eb2b35f -> 88ffc5714


Improved a bit on Camel route coverage


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

Branch: refs/heads/master
Commit: 88ffc571435cd593b91fb5292ef1271b37218249
Parents: ceccd28
Author: Claus Ibsen <davscl...@apache.org>
Authored: Tue Oct 17 11:12:11 2017 +0200
Committer: Claus Ibsen <davscl...@apache.org>
Committed: Tue Oct 17 11:22:11 2017 +0200

----------------------------------------------------------------------
 .../org/apache/camel/component/sql/SqlTransactedRouteTest.java | 6 +++---
 .../org/apache/camel/parser/helper/RouteCoverageHelper.java    | 3 +--
 2 files changed, 4 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/camel/blob/88ffc571/components/camel-sql/src/test/java/org/apache/camel/component/sql/SqlTransactedRouteTest.java
----------------------------------------------------------------------
diff --git 
a/components/camel-sql/src/test/java/org/apache/camel/component/sql/SqlTransactedRouteTest.java
 
b/components/camel-sql/src/test/java/org/apache/camel/component/sql/SqlTransactedRouteTest.java
index d1b83ef..288bf11 100644
--- 
a/components/camel-sql/src/test/java/org/apache/camel/component/sql/SqlTransactedRouteTest.java
+++ 
b/components/camel-sql/src/test/java/org/apache/camel/component/sql/SqlTransactedRouteTest.java
@@ -83,7 +83,7 @@ public class SqlTransactedRouteTest extends CamelTestSupport {
     public void testCommit() throws Exception {
         context.addRoutes(new RouteBuilder() {
             public void configure() throws Exception {
-                from("direct:start")
+                from("direct:start").routeId("commit")
                     .transacted("required")
                     .to(sqlEndpoint)
                     .process(new Processor() {
@@ -122,7 +122,7 @@ public class SqlTransactedRouteTest extends 
CamelTestSupport {
     public void testRollbackAfterExceptionInSecondStatement() throws Exception 
{
         context.addRoutes(new RouteBuilder() {
             public void configure() throws Exception {
-                from("direct:start")
+                from("direct:start").routeId("rollback")
                     .transacted("required")
                     .to(sqlEndpoint)
                     .process(new Processor() {
@@ -152,7 +152,7 @@ public class SqlTransactedRouteTest extends 
CamelTestSupport {
     public void testRollbackAfterAnException() throws Exception {
         context.addRoutes(new RouteBuilder() {
             public void configure() throws Exception {
-                from("direct:start")
+                from("direct:start").routeId("rollback2")
                     .transacted("required")
                     .to(sqlEndpoint)
                     .process(new Processor() {

http://git-wip-us.apache.org/repos/asf/camel/blob/88ffc571/tooling/camel-route-parser/src/main/java/org/apache/camel/parser/helper/RouteCoverageHelper.java
----------------------------------------------------------------------
diff --git 
a/tooling/camel-route-parser/src/main/java/org/apache/camel/parser/helper/RouteCoverageHelper.java
 
b/tooling/camel-route-parser/src/main/java/org/apache/camel/parser/helper/RouteCoverageHelper.java
index a124dda..655d974 100644
--- 
a/tooling/camel-route-parser/src/main/java/org/apache/camel/parser/helper/RouteCoverageHelper.java
+++ 
b/tooling/camel-route-parser/src/main/java/org/apache/camel/parser/helper/RouteCoverageHelper.java
@@ -130,8 +130,7 @@ public final class RouteCoverageHelper {
 
         // inlined error handler, on completion etc should be skipped (and 
currently not supported in route coverage)
         boolean skip = "onException".equals(key) || "onCompletion".equals(key)
-            || "intercept".equals(key) || "interceptFrom".equals(key) || 
"interceptSendToEndpoint".equals(key)
-            || "policy".equals(key) || "transacted".equals(key);
+            || "intercept".equals(key) || "interceptFrom".equals(key) || 
"interceptSendToEndpoint".equals(key);
 
         if (skip) {
             return;

Reply via email to