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

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

              Dates

              • Created:
                Updated:
                Resolved: