Uploaded image for project: 'ManifoldCF'
  1. ManifoldCF
  2. CONNECTORS-1509

The failsafe and surefire test configuration needs to be improved to properly include all the required test cases

    XMLWordPrintableJSON

Details

    Description

      The current surefire plugin configuration has the following configurations 

      <excludes>
      <exclude>**/Postgresql.java</exclude>
      <exclude>**/MySQL.java</exclude>
      </excludes>

      this will exclude  SanityPostgresqlTest which is a unit test from being executed which is questionable behaviour and <exclude>**/MySQL.java</exclude> serves no purpose since there are no any test classes matching the pattern

      Also I suggest adding the following pattern to exclude integration tests 

      <exclude>**/*IT.java</exclude> which seems to be our convention for naming integration tests.

      The current failsafe plugin configuration also share the above configurations which is 

      <excludes>
      <exclude>**/Postgresql.java</exclude>
      <exclude>**/MySQL.java</exclude>
      </excludes>

      which seems of no purpose to me there are no any cases matching the above configuration and matching the default configuration which is 

      <includes>
      <include>*/IT.java</include>
      <include>**/*IT.java</include>
      <include>**/*ITCase.java</include>
      </includes>

      Also NavigationHSQLDBUI test does not seem to be executed with the maven build process

      Attachments

        Activity

          People

            kwright@metacarta.com Karl Wright
            irinduPera Irindu Nugawela
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: