Cleanup build after splitting the tests (#4282)

* Use default failsafe section to override summary location

* Remove unnecessary compile step

Author: uglide

Makefile

@@ -522,10 +522,10 @@ stop:
 test: | start mvn-test-local stop
 
 mvn-test-local:
-	@TEST_ENV_PROVIDER=local mvn -Dtest=${TEST} clean compile verify
+	@TEST_ENV_PROVIDER=local mvn -Dtest=${TEST} clean verify
 
 mvn-test:
-	mvn -Dtest=${TEST} clean compile verify
+	mvn -Dtest=${TEST} clean verify
 
 package: | start mvn-package stop
 

pom.xml

@@ -354,6 +354,7 @@
 					<systemPropertyVariables>
 						<redis-hosts>${redis-hosts}</redis-hosts>
 					</systemPropertyVariables>
+					<summaryFile>${project.build.directory}/failsafe-summary.xml</summaryFile>
 					<!-- Default includes used when invoking failsafe goals directly (e.g., mvn failsafe:integration-test) -->
 					<includes>
 						<include>**/*IntegrationTest.java</include>
@@ -403,9 +404,6 @@
 						<goals>
 							<goal>verify</goal>
 						</goals>
-						<configuration>
-							<summaryFile>${project.build.directory}/failsafe-summary.xml</summaryFile>
-						</configuration>
 					</execution>
 				</executions>
 			</plugin>