[camel] 02/03: CAMEL-12282: rest-dsl - Inlined route has route id wrongly assigned

[davsclaus]

This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 2e87df38fc9c3d0f3fb5ababc9c0d21835b54908
Author: Claus Ibsen <claus.ib...@gmail.com>
AuthorDate: Wed Feb 21 10:33:17 2018 +0100

    CAMEL-12282: rest-dsl - Inlined route has route id wrongly assigned
---
 .../src/main/java/org/apache/camel/model/rest/RestDefinition.java     | 2 +-
 .../test/java/org/apache/camel/management/ManagedFromRestGetTest.java | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git 
a/camel-core/src/main/java/org/apache/camel/model/rest/RestDefinition.java 
b/camel-core/src/main/java/org/apache/camel/model/rest/RestDefinition.java
index fa024ae..ab43326 100644
--- a/camel-core/src/main/java/org/apache/camel/model/rest/RestDefinition.java
+++ b/camel-core/src/main/java/org/apache/camel/model/rest/RestDefinition.java
@@ -910,7 +910,7 @@ public class RestDefinition extends 
OptionalIdentifiedDefinition<RestDefinition>
 
             // the route should be from this rest endpoint
             route.fromRest(from);
-            route.id(routeId);
+            route.routeId(routeId);
             route.setRestDefinition(this);
             answer.add(route);
         }
diff --git 
a/camel-core/src/test/java/org/apache/camel/management/ManagedFromRestGetTest.java
 
b/camel-core/src/test/java/org/apache/camel/management/ManagedFromRestGetTest.java
index 2b69d03..e544a09 100644
--- 
a/camel-core/src/test/java/org/apache/camel/management/ManagedFromRestGetTest.java
+++ 
b/camel-core/src/test/java/org/apache/camel/management/ManagedFromRestGetTest.java
@@ -76,6 +76,10 @@ public class ManagedFromRestGetTest extends 
ManagementTestSupport {
         // and we should have rest in the routes that indicate its from a rest 
dsl
         assertTrue(xml2.contains("rest=\"true\""));
 
+        assertTrue(xml2.contains(" <to id=\"to1\" uri=\"direct:hello\"/>"));
+        assertTrue(xml2.contains("<to id=\"to2\" uri=\"direct:bye\"/>"));
+        assertTrue(xml2.contains("<to id=\"to3\" uri=\"mock:update\"/>"));
+
         // there should be 3 + 2 routes
         assertEquals(3 + 2, context.getRouteDefinitions().size());
     }

-- 
To stop receiving notification emails like this one, please contact
davscl...@apache.org.