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

Make CamelBlueprintTestSupport tests more predictable

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.13.1
    • Fix Version/s: 2.12.4, 2.13.2, 2.14.0
    • Component/s: camel-blueprint
    • Labels:
      None
    • Estimated Complexity:
      Unknown

      Description

      Currently the tests which override org.apache.camel.test.blueprint.CamelBlueprintTestSupport#useOverridePropertiesWithConfigAdmin or org.apache.camel.test.blueprint.CamelBlueprintTestSupport#loadConfigAdminConfigurationFile have race issue, because changing configuration leads to reload of Blueprint Container.

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user grgrzybek opened a pull request:

          https://github.com/apache/camel/pull/165

          CAMEL-7469 adding some synchronization for BP container and config admin

          I checked 200 runs of the same test - all have passed!

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/grgrzybek/camel CAMEL-7469

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/camel/pull/165.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #165


          commit e21a6930a3da4c2698d2c1acfca1c4c8401c97dd
          Author: Grzegorz Grzybek <gr.grzybek@gmail.com>
          Date: 2014-05-28T12:41:05Z

          CAMEL-7469 adding some synchronization for BP container and config admin updates


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user grgrzybek opened a pull request: https://github.com/apache/camel/pull/165 CAMEL-7469 adding some synchronization for BP container and config admin I checked 200 runs of the same test - all have passed! You can merge this pull request into a Git repository by running: $ git pull https://github.com/grgrzybek/camel CAMEL-7469 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/camel/pull/165.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #165 commit e21a6930a3da4c2698d2c1acfca1c4c8401c97dd Author: Grzegorz Grzybek <gr.grzybek@gmail.com> Date: 2014-05-28T12:41:05Z CAMEL-7469 adding some synchronization for BP container and config admin updates
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/camel/pull/165

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/camel/pull/165
          Hide
          davsclaus Claus Ibsen added a comment -

          Thanks for this PR

          Show
          davsclaus Claus Ibsen added a comment - Thanks for this PR

            People

            • Assignee:
              gzres Grzegorz Grzybek
              Reporter:
              gzres Grzegorz Grzybek
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development