CAMEL-11234: NullPointerException while trying to get the Route Status on startup
Project: http://git-wip-us.apache.org/repos/asf/camel/repo Commit: http://git-wip-us.apache.org/repos/asf/camel/commit/e313e6db Tree: http://git-wip-us.apache.org/repos/asf/camel/tree/e313e6db Diff: http://git-wip-us.apache.org/repos/asf/camel/diff/e313e6db Branch: refs/heads/camel-2.19.x Commit: e313e6dba38c282369b3601212620d63b44dc14f Parents: 7b23f11 Author: Prasanth Pagolu <ppag...@redhat.com> Authored: Mon May 15 13:55:51 2017 +0100 Committer: Claus Ibsen <davscl...@apache.org> Committed: Tue May 16 10:24:10 2017 +0200 ---------------------------------------------------------------------- .../java/org/apache/camel/impl/DefaultCamelContext.java | 3 ++- .../OnExceptionGlobalScopedRecursionTest.java | 12 ++++++------ .../processor/onexception/OnExceptionRecursionTest.java | 2 +- 3 files changed, 9 insertions(+), 8 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/camel/blob/e313e6db/camel-core/src/main/java/org/apache/camel/impl/DefaultCamelContext.java ---------------------------------------------------------------------- diff --git a/camel-core/src/main/java/org/apache/camel/impl/DefaultCamelContext.java b/camel-core/src/main/java/org/apache/camel/impl/DefaultCamelContext.java index 82644d5..65c181d 100644 --- a/camel-core/src/main/java/org/apache/camel/impl/DefaultCamelContext.java +++ b/camel-core/src/main/java/org/apache/camel/impl/DefaultCamelContext.java @@ -3041,7 +3041,8 @@ public class DefaultCamelContext extends ServiceSupport implements ModelCamelCon // count how many routes are actually started int started = 0; for (Route route : getRoutes()) { - if (getRouteStatus(route.getId()).isStarted()) { + ServiceStatus status = getRouteStatus(route.getId()); + if (status != null && status.isStarted()) { started++; } } http://git-wip-us.apache.org/repos/asf/camel/blob/e313e6db/camel-core/src/test/java/org/apache/camel/processor/onexception/OnExceptionGlobalScopedRecursionTest.java ---------------------------------------------------------------------- diff --git a/camel-core/src/test/java/org/apache/camel/processor/onexception/OnExceptionGlobalScopedRecursionTest.java b/camel-core/src/test/java/org/apache/camel/processor/onexception/OnExceptionGlobalScopedRecursionTest.java index 417d4fa..37f6088 100644 --- a/camel-core/src/test/java/org/apache/camel/processor/onexception/OnExceptionGlobalScopedRecursionTest.java +++ b/camel-core/src/test/java/org/apache/camel/processor/onexception/OnExceptionGlobalScopedRecursionTest.java @@ -42,7 +42,7 @@ public class OnExceptionGlobalScopedRecursionTest extends ContextTestSupport { .to("mock:c") .log("onException") .throwException(new NullPointerException("A NPE error here")) - .end(); + .end(); from("direct:test") .to("mock:a") @@ -78,7 +78,7 @@ public class OnExceptionGlobalScopedRecursionTest extends ContextTestSupport { .to("mock:c") .log("onException") .throwException(new NullPointerException("A NPE error here")) - .end(); + .end(); from("direct:test") .to("mock:a") @@ -114,7 +114,7 @@ public class OnExceptionGlobalScopedRecursionTest extends ContextTestSupport { .to("mock:c") .log("onException") .to("direct:error") - .end(); + .end(); from("direct:test") .to("mock:a") @@ -157,7 +157,7 @@ public class OnExceptionGlobalScopedRecursionTest extends ContextTestSupport { .to("mock:c") .log("onException") .to("direct:error") - .end(); + .end(); from("direct:test") .to("mock:a") @@ -199,7 +199,7 @@ public class OnExceptionGlobalScopedRecursionTest extends ContextTestSupport { .to("mock:c") .log("onException") .to("direct:error") - .end(); + .end(); from("direct:test") .to("mock:a") @@ -239,7 +239,7 @@ public class OnExceptionGlobalScopedRecursionTest extends ContextTestSupport { .to("mock:c") .log("onException") .to("direct:error") - .end(); + .end(); from("direct:test") .to("mock:a") http://git-wip-us.apache.org/repos/asf/camel/blob/e313e6db/camel-core/src/test/java/org/apache/camel/processor/onexception/OnExceptionRecursionTest.java ---------------------------------------------------------------------- diff --git a/camel-core/src/test/java/org/apache/camel/processor/onexception/OnExceptionRecursionTest.java b/camel-core/src/test/java/org/apache/camel/processor/onexception/OnExceptionRecursionTest.java index bac6e40..d832296 100644 --- a/camel-core/src/test/java/org/apache/camel/processor/onexception/OnExceptionRecursionTest.java +++ b/camel-core/src/test/java/org/apache/camel/processor/onexception/OnExceptionRecursionTest.java @@ -68,7 +68,7 @@ public class OnExceptionRecursionTest extends ContextTestSupport { assertEquals("Bad state", ise.getMessage()); } - assertMockEndpointsSatisfied(); + assertMockEndpointsSatisfied(); } public void testRecursionDirectNoErrorHandler() throws Exception {