Updated jacoco coverage report to run in test phase. Closes #88.

Author: jonathanaustin

CHANGELOG.md

@@ -9,6 +9,7 @@
   * Changed JavadocPackage check to warning
   * Changed EmptyForIteratorPad check to warning as IntelliJ code formatting does not currently allow for this
 * Added a new checkstyle config file bt-checkstyle-format-only.xml for projects that only want to check basic code formatting.
+* Modified the jacoco report goal to the test phase and to make sure it runs after surefire, moved the jacoco plugin after the surefire plugin in the pom file which is how maven determines the run order of plugins defined on the same phase #88
 
 ## 1.0.19
 * Added bt.qa.fail property as a convenience property to allow projects to fail builds or only report QA violations. #84

qa-parent/pom.xml

@@ -182,6 +182,14 @@
 
 	<build>
 		<plugins>
+
+			<!-- Surefire -->
+			<plugin>
+				<groupId>org.apache.maven.plugins</groupId>
+				<artifactId>maven-surefire-plugin</artifactId>
+				<version>${bt.surefire.plugin.version}</version>
+			</plugin>
+
 			<!-- Code coverage. -->
 			<plugin>
 				<groupId>org.jacoco</groupId>
@@ -196,23 +204,17 @@
 						</goals>
 					</execution>
 					<!-- Jacoco Coverage Report -->
+					<!-- As this is bound to the same phase as surefire but needs to run after surefire it must be defined after the surefire plugin. -->
 					<execution>
 						<id>post-unit-test</id>
-						<phase>prepare-package</phase>
+						<phase>test</phase>
 						<goals>
 							<goal>report</goal>
 						</goals>
 					</execution>
 				</executions>
 			</plugin>
 
-			<!-- Surefire -->
-			<plugin>
-				<groupId>org.apache.maven.plugins</groupId>
-				<artifactId>maven-surefire-plugin</artifactId>
-				<version>${bt.surefire.plugin.version}</version>
-			</plugin>
-
 			<!-- Verify: Check the code style. -->
 			<plugin>
 				<groupId>org.apache.maven.plugins</groupId>