Uploaded image for project: 'Application Server 7'
  1. Application Server 7
  2. AS7-856

Replace -DskipTests=true with -DallTests

    Details

    • Type: Task
    • Status: Resolved (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: 7.0.0.CR1
    • Component/s: None
    • Labels:
      None

      Description

      Using skipTests as a flag has a negative side effect such that you cannot do

      > mvn -Dtest=SomeTestCase test

      in an integration module any more. Instead you have to do

      > mvn -DskipTests=false -Dtest=SomeTestCase test

      This is non-intuitive and non-standard. Instead I propose to use an
      alternate switch like 'allTests' to turn on profiles which run the
      complete set of tests. IMHO its ok to exclude entire (testsuite) modules
      from the build (i.e. not to compile the tests) because compilation
      errors show up in the IDE and Hudson anyway.

      I propose

      1) By default, all normal modules and testsuite/smoke will be built in
      the standard way
      (i.e. failing tests will fail the build)

      2) By adding -DskipTests=true tests that would normally be executed are skipped

      3) By adding -DallTests all modules that contain tests will be executed. In Hudson you could add -fae
      which fails the build at the end - so catch all failing tests in a single run

      It boils down to whether you want to compile tests that you don't want to execute.
      IMHO that's not necessary in day to day work - Hudson can catch this.

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Assignee:
                thomas.diesler Thomas Diesler
                Reporter:
                thomas.diesler Thomas Diesler
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: