Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-13724

camel route customized id isn't correct if there are more than one Rest DSL route availble

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.23.3, 2.24.1
    • Fix Version/s: 3.0.0, 2.23.4, 2.24.2, 2.25.0, 3.0.0.M4
    • Component/s: rest
    • Labels:
      None
    • Estimated Complexity:
      Unknown

      Description

      If rest dsl routes have their own customer ids, the id would be ignored, hence generate invalid swagger operation Id
      the testcase here can expose this problem

      --- a/core/camel-core/src/test/java/org/apache/camel/impl/RouteIdRestDefinitionTest.java
      +++ b/core/camel-core/src/test/java/org/apache/camel/impl/RouteIdRestDefinitionTest.java
      @@ -39,7 +39,8 @@ public class RouteIdRestDefinitionTest extends ContextTestSupport {
                   public void configure() throws Exception {
                       from("direct:start1?timeout=30000").to("mock:result");
                       from("direct:start2").to("mock:result");
      -                rest("/say/hello").get("/bar").id("getSayHelloBar").to("mock:result");
      +                rest("/say/hello").get("/bar").id("getSayHelloBar").to("mock:result")
      +                .get("/bar/{user}").id("getSayHelloBarWithUser").to("mock:result");
                   }
               };
           }
      @@ -47,6 +48,7 @@ public class RouteIdRestDefinitionTest extends ContextTestSupport {
           @Test
           public void testSayHelloBar() {
               assertEquals("getSayHelloBar", context.getRouteDefinitions().get(2).getId());
      +        assertEquals("getSayHelloBarWithUser", context.getRouteDefinitions().get(3).getId());
           }
           
       }
      

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                ffang Freeman Fang
                Reporter:
                ffang Freeman Fang
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: