Michael Blow has submitted this change and it was merged. Change subject: [NO ISSUE] Move jenkins test suites into poms ......................................................................
[NO ISSUE] Move jenkins test suites into poms Avoid issues corodinating git & Jenkins config changes to test config, by defining job profiles in the pom files. Change-Id: I3fed5c3d578d8c5faa95d22cbdb33b1bac8f7190 Reviewed-on: https://asterix-gerrit.ics.uci.edu/2306 Sonar-Qube: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Integration-Tests: Jenkins <[email protected]> Contrib: Michael Blow <[email protected]> Contrib: Jenkins <[email protected]> Reviewed-by: Murtadha Hubail <[email protected]> --- M asterixdb/asterix-app/pom.xml M asterixdb/pom.xml M hyracks-fullstack/pom.xml 3 files changed, 48 insertions(+), 6 deletions(-) Approvals: Jenkins: Verified; No violations found; ; Verified Michael Blow: Murtadha Hubail: Looks good to me, approved diff --git a/asterixdb/asterix-app/pom.xml b/asterixdb/asterix-app/pom.xml index dc00bea..c32872a 100644 --- a/asterixdb/asterix-app/pom.xml +++ b/asterixdb/asterix-app/pom.xml @@ -280,6 +280,34 @@ </plugins> </build> </profile> + <profile> + <id>asterix-gerrit-asterix-app</id> + <properties> + <test.excludes>**/SqlppExecutionWithCancellationTest.java,**/DmlTest.java,**/RepeatedTest.java,**/SqlppExecutionTest.java</test.excludes> + </properties> + </profile> + <profile> + <id>asterix-gerrit-asterix-app-sql-execution</id> + <properties> + <test.includes/> + <itest.includes>**/SqlppExecution*IT.java</itest.includes> + <failIfNoTests>false</failIfNoTests> + </properties> + </profile> + <profile> + <id>asterix-gerrit-verify-asterix-app</id> + <properties> + <test.includes/> + <itest.excludes>**/SqlppExecution*IT.java,**/RebalanceWithCancellationIT.java</itest.excludes> + <failIfNoTests>false</failIfNoTests> + </properties> + </profile> + <profile> + <id>asterix-gerrit-rebalance-cancellation</id> + <properties> + <test.includes>**/RebalanceWithCancellationIT.java</test.includes> + </properties> + </profile> </profiles> <dependencies> <dependency> diff --git a/asterixdb/pom.xml b/asterixdb/pom.xml index a3f8848..3850a85 100644 --- a/asterixdb/pom.xml +++ b/asterixdb/pom.xml @@ -62,6 +62,12 @@ <invalid.tests>**/DmlTest.java</invalid.tests> <global.test.includes>**/*TestSuite.java,**/*Test.java</global.test.includes> <global.test.excludes>${invalid.tests},${repeated.tests}</global.test.excludes> + <test.includes>${global.test.includes}</test.includes> + <test.excludes>${global.test.excludes}</test.excludes> + <global.itest.includes>**/*IT.java,**/*IT.java,**/*ITCase.java</global.itest.includes> + <global.itest.excludes/> + <itest.includes>${global.itest.includes}</itest.includes> + <itest.excludes>${global.itest.excludes}</itest.excludes> <!-- Versions under dependencymanagement or used in many projects via properties --> <algebricks.version>0.3.3-SNAPSHOT</algebricks.version> @@ -101,10 +107,10 @@ ${coverageArgLine} </argLine> <includes> - <include>${global.test.includes},${test.includes}</include> + <include>${test.includes}</include> </includes> <excludes> - <exclude>${global.test.excludes},${test.excludes}</exclude> + <exclude>${test.excludes}</exclude> </excludes> <skipTests>${skip.surefire.tests}</skipTests> </configuration> @@ -120,6 +126,12 @@ <systemProperties> <log4j.configurationFile>${testLog4jConfigFile}</log4j.configurationFile> </systemProperties> + <includes> + <include>${itest.includes}</include> + </includes> + <excludes> + <exclude>${itest.excludes}</exclude> + </excludes> </configuration> <executions> <execution> diff --git a/hyracks-fullstack/pom.xml b/hyracks-fullstack/pom.xml index f08bec6..6e3d71f 100644 --- a/hyracks-fullstack/pom.xml +++ b/hyracks-fullstack/pom.xml @@ -57,6 +57,7 @@ <jvm.extraargs /> <sonar.jacoco.reportPath>${env.PWD}/target/jacoco-merged.exec</sonar.jacoco.reportPath> <source-format.skip>false</source-format.skip> + <skipTests>false</skipTests> <skip.surefire.tests>${skipTests}</skip.surefire.tests> <skip.testResources>${maven.test.skip}</skip.testResources> <testLog4jConfigFile>${root.dir}/src/test/resources/log4j2-test.xml</testLog4jConfigFile> @@ -64,8 +65,9 @@ <hanging.pregelix.tests>**/pregelix/**/FailureRecovery*.java</hanging.pregelix.tests> <hivesterix.perf.tests>**/hivesterix/perf/PerfTestSuite.java</hivesterix.perf.tests> <global.test.includes>**/*TestSuite.java,**/*Test.java</global.test.includes> - <global.test.excludes>**/Abstract*.java,${hanging.pregelix.tests},${hivesterix.perf.tests} - </global.test.excludes> + <global.test.excludes>**/Abstract*.java,${hanging.pregelix.tests},${hivesterix.perf.tests}</global.test.excludes> + <test.includes>${global.test.includes}</test.includes> + <test.excludes>${global.test.excludes}</test.excludes> <!-- Versions under dependencymanagement or used in many projects via properties --> <hadoop.version>2.2.0</hadoop.version> <jacoco.version>0.7.6.201602180812</jacoco.version> @@ -267,10 +269,10 @@ <log4j.configurationFile>${testLog4jConfigFile}</log4j.configurationFile> </systemPropertyVariables> <includes> - <include>${global.test.includes},${test.includes}</include> + <include>${test.includes}</include> </includes> <excludes combine.children="append"> - <exclude>${global.test.excludes},${test.excludes}</exclude> + <exclude>${test.excludes}</exclude> </excludes> <skipTests>${skip.surefire.tests}</skipTests> </configuration> -- To view, visit https://asterix-gerrit.ics.uci.edu/2306 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3fed5c3d578d8c5faa95d22cbdb33b1bac8f7190 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow <[email protected]> Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]>
