[GitHub] [camel-examples] essobedo opened a new pull request #62: CAMEL-11834: Add tests for basic/cdi/kotlin/loadbalancing/main-artemis/routeloader/routes-configuration/routetemplate

[GitBox <gi...@apache.org>]

essobedo opened a new pull request #62:
URL: https://github.com/apache/camel-examples/pull/62


   ## Motivation
   
   Most of the Camel examples don't have any test such that they need to be launched manually to ensure that they still work which is not convenient.
   
   ## Modifications:
   
   * Remove the `.gitignore` entries that were related to the location of activemq/artemis data since the location is now configured to use the target directory
   * Use `application.properties` files to configure the location of `activemq-data` in `activemq-tomcat`(one for war and another for the test.
   * Add a file `arquillian.xml` to indicate `arquillian` to use the target directory as location of the tomcat bundle
   * Refactor the entry class of the example basic to make them easier to test 
   * Remove useless namespace definitions
   * Fix the readme.doc (wording, grammar, out dated info, typos)
   * Add a build section in all readme.doc for consistency
   * Refactor the Spring configuration of `loadbalancing` to be a able to have all the routes within the same camel context for the test by relying on external `routeContext`s
   * Make the period of the timer configurable in the example main-artemis to reduce the duration of the test
   * Add the credentials in the `application.properties` of main-artemis since the readme.doc indicates that a user admin/admin must be created
   * Load the XML and YAML routes using explicitly the `PackageScanResourceResolver` and `RoutesLoader` in the tests as I could not find any better way
   * Remove the file `src/main/data/foo.properties` from route-template example as it is not used
   * Remove the option `hi` from the `application.properties` of route-template example as it is not used
   * Call `new MyTemplateBuilder().configure(context)` explicitly to create routes from the template as I could not find any better way


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [camel-examples] davsclaus merged pull request #62: CAMEL-11834: Add tests for basic/cdi/kotlin/loadbalancing/main-artemis/routeloader/routes-configuration/routetemplate

[GitBox <gi...@apache.org>]

davsclaus merged pull request #62:
URL: https://github.com/apache/camel-examples/pull/62


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org