New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Camel Documentation does not state citrusCamelContext as default name #242

Closed
gucce opened this Issue May 22, 2017 · 0 comments

Comments

Projects
None yet
2 participants
@gucce
Contributor

gucce commented May 22, 2017

Problem

In the documentation on the Camel module it is not explicitly stated that the default camel context bean name is citrusCamelContext.

Examples from the documentation.

An example camel context with bean id camelContext appears at the beginning.

<camelContext id="camelContext" xmlns="http://camel.apache.org/schema/spring">
    <route id="newsRoute">
        <from uri="direct:news"/>
        <to uri="log:com.consol.citrus.camel?level=INFO"/>
        <to uri="seda:news-feed"/>
    </route>
</camelContext>

Then a example on starting routes explicitly states which Camel context to use by bean id.

<camel:start-routes camel-context="camelContext">
    <route id="route_1"/>
</camel:start-routes>

Later an example on creating routes does not state any context. In this case the implicit default name is citrusCamelContext which is not stated in the documentation.

<camel:create-routes>
    <routeContext xmlns="http://camel.apache.org/schema/spring">
        <route id="route_1">
            <from uri="direct:test1"/>
            <to uri="mock:test1"/>
        </route>
    </routeContext>
</camel:create-routes>

Solution

Please explicitly point out the default name citrusCamelContext of a Camel context defined in citrus.

@christophd christophd added the READY label May 28, 2017

@christophd christophd added this to the v2.7.2 milestone May 28, 2017

@christophd christophd added IN PROGRESS and removed READY labels Jun 28, 2017

@christophd christophd self-assigned this Jun 28, 2017

@christophd christophd removed the IN PROGRESS label Jun 29, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment