[CONNECTORS-1509] The failsafe and surefire test configuration needs to be improved to properly include all the required test cases

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Not A Problem
• Affects Version/s: ManifoldCF 2.10
• Component/s: CMIS Output Connector
• Labels: CMIS outputconnector testsuite
• Assignee: Karl Wright
• Reporter: Irindu Nugawela
• Created: 15/Jun/18 09:25
• Updated: 15/Jun/18 15:03
• Resolved: 15/Jun/18 15:03

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