Uploaded image for project: 'camunda BPM'
  1. camunda BPM
  2. CAM-5042

Integration tests with _JBOSS suffix not executed on JBoss

    Details

    • Type: Task
    • Status: Closed
    • Priority: L3 - Default
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 7.7.0, 7.7.0-alpha1
    • Component/s: engine
    • Labels:
      None

      Description

      AT:

      • should be included in jboss profiles,
      • it is not necessary to exclude them in other profiles since they do not match the surefire naming conventions
      • should be backported

        Activity

        Hide
        sebastian.menski Sebastian Menski added a comment -

        We have to make sure that they don't actually are included either by explicite exclude them or remind everyone that the class name shouldn't start with Test.

        Show
        sebastian.menski Sebastian Menski added a comment - We have to make sure that they don't actually are included either by explicite exclude them or remind everyone that the class name shouldn't start with Test .
        Hide
        lipphardt Christian Lipphardt added a comment -

        We should decide on a strategy:

        • either name them Test**** and exclude them where they shouldn't run or
        • do not name them Test**** and include them explicitly
        Show
        lipphardt Christian Lipphardt added a comment - We should decide on a strategy: either name them Test**** and exclude them where they shouldn't run or do not name them Test**** and include them explicitly
        Show
        christopher.zell Christopher Zell added a comment - - edited Sebastian Menski Christian Lipphardt Is this issue already fixed? Because in the current builds test cases with _JBOSS postfix are already executed. https://hq2.camunda.com/jenkins/ci/view/7.7/job/7.7-engine-IT-jboss-72-sqlserver-2014/lastBuild/console https://hq2.camunda.com/jenkins/ci/job/7.7-engine-IT-jboss-72-h2/52/console https://hq2.camunda.com/jenkins/ci/job/7.7-engine-IT-jboss-72-db2-97/9/console https://hq2.camunda.com/jenkins/ci/job/7.7-engine-IT-jboss-72-db2-101/9/console https://hq2.camunda.com/jenkins/ci/job/7.7-engine-IT-jboss-72-mysql-56/9/console
        Hide
        christopher.zell Christopher Zell added a comment -

        Seems to be fixed with Thorbens commit

        Show
        christopher.zell Christopher Zell added a comment - Seems to be fixed with Thorbens commit
        Hide
        sebastian.menski Sebastian Menski added a comment -

        Seems Thorben Lindhauer renamed the test over a year ago: https://github.com/camunda/camunda-bpm-platform/commit/1142b9f3246471da2d79760d620aea21fe340bf6

        Not sure if we want to revert this and go for an explicit inclusion instead of an explicit exclusion (see previous comments).

        Show
        sebastian.menski Sebastian Menski added a comment - Seems Thorben Lindhauer renamed the test over a year ago: https://github.com/camunda/camunda-bpm-platform/commit/1142b9f3246471da2d79760d620aea21fe340bf6 Not sure if we want to revert this and go for an explicit inclusion instead of an explicit exclusion (see previous comments).

          People

          • Assignee:
            Unassigned
            Reporter:
            meyer Daniel Meyer
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: