[jira] [Comment Edited] (CAMEL-10247) OnException is overriding route Ids for the Route Definitions in the Camel Test

["Srinivas (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CAMEL-10247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15424564#comment-15424564 ] 

Srinivas edited comment on CAMEL-10247 at 8/17/16 2:05 PM:
-----------------------------------------------------------

Thanks for your response.

1. I have changed the bug example to use id("foo") because I am using Id not RouteId. (when I copied an example I messed up).
2. Your example works and the difference is as stated above, you are using routeId not Id. And my code also works when I have changed from Id to RouteId
4. I am pretty sure we are using id everywhere and id doesn't work when onException is present and routeId works.
3. Which one is recommend to use id or routeId in all situations?




was (Author: srinivas@live.com):

Thanks for your response.

1. I have changed the bug example to use id("foo") because I am using Id not RouteId. (when I copied an example I messed up).
2. Your example works and the difference is as stated above, you are using routeId not Id. And my code also works when I have changed from Id to RouteId
3. Which one is recommend to use id or routeId?.


> OnException is overriding route Ids for the Route Definitions in the Camel Test
> -------------------------------------------------------------------------------
>
>                 Key: CAMEL-10247
>                 URL: https://issues.apache.org/jira/browse/CAMEL-10247
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-blueprint, camel-core, camel-test
>    Affects Versions: 2.16.2
>         Environment: Camel, Karaf
>            Reporter: Srinivas
>            Priority: Minor
>
> When there is onException block in the Routes, Route Ids are overridden in the test (CBTS) class. 
> Ex: {code}
> public class TestRoute extends RouteBuilder {
> 	@Override
> 	public void configure() throws Exception {
> 		onException(Exception.class).id("ExceptionId")
> 			.maximumRedeliveries(0)
> 			.handled(true)
> 			.setHeader(Exchange.HTTP_RESPONSE_CODE).constant(HttpURLConnection.HTTP_NOT_FOUND)
> 			.end()
> 			.stop();
> 		from("file:foo").id("foo")
> 			.to("blah") 
> 			.end();
> 	}
> }
> {code}
> A Camel test (using Camel Blueprint Test Support) is written for the above route, then found that Route Ids is overridden with "route1" in the route definition. When OnException block code is commented, route id is replaced with "foo" in the route definition in CBTS. This behavior can be reproduced consistently.  
> Please let me know if this is known behavior or needs fix.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)