Skip to content

Commit

Permalink
[FIXED JENKINS-10913] Add support for maven-android-plugin integratio…
Browse files Browse the repository at this point in the history
…n test

Earlier versions of the plugin (2.x and prior to 3.0.0-alpha6) do not produce
test reports and also do not accept the same configuration properties. Ignore
those earlier versions.
Also accept the new plugin name android-maven-plugin (changed to fit with
Maven plugin naming guidelines).

Originally-Committed-As: 952f19fbfac8c5bef9ffbb345f33e4f1f8978b32
  • Loading branch information
oldelvet committed Sep 30, 2011
1 parent c63bb1d commit 80d71ca
Showing 1 changed file with 12 additions and 1 deletion.
13 changes: 12 additions & 1 deletion src/main/java/hudson/maven/reporters/SurefireArchiver.java
Expand Up @@ -231,6 +231,7 @@ private boolean isSurefireTest(MojoInfo mojo) {
&& (!mojo.is("org.sonatype.tycho", "maven-osgi-test-plugin", "test"))
&& (!mojo.is("org.codehaus.mojo", "gwt-maven-plugin", "test"))
&& (!mojo.is("com.jayway.maven.plugins.android.generation2", "maven-android-plugin", "internal-integration-test"))
&& (!mojo.is("com.jayway.maven.plugins.android.generation2", "android-maven-plugin", "internal-integration-test"))
&& (!mojo.is("org.apache.maven.plugins", "maven-surefire-plugin", "test"))
&& (!mojo.is("org.apache.maven.plugins", "maven-failsafe-plugin", "integration-test")))
return false;
Expand Down Expand Up @@ -280,11 +281,21 @@ else if (mojo.is("org.sonatype.flexmojos", "flexmojos-maven-plugin", "test-run")
if (((skipTests != null) && (skipTests))) {
return false;
}
} else if (mojo.is("com.jayway.maven.plugins.android.generation2", "maven-android-plugin", "internal-integration-test")) {
} else if (mojo.is("com.jayway.maven.plugins.android.generation2", "android-maven-plugin", "internal-integration-test")) {
Boolean skipTests = mojo.getConfigurationValue("skipTests", Boolean.class);
if (((skipTests != null) && (skipTests))) {
return false;
}
} else if (mojo.is("com.jayway.maven.plugins.android.generation2", "maven-android-plugin", "internal-integration-test")) {
if (mojo.pluginName.version.compareTo("3.0.0-alpha-6") < 0) {
// Earlier versions do not support tests
return false;
} else {
Boolean skipTests = mojo.getConfigurationValue("skipTests", Boolean.class);
if (((skipTests != null) && (skipTests))) {
return false;
}
}
} else if (mojo.is("org.codehaus.mojo", "gwt-maven-plugin", "test") && mojo.pluginName.version.compareTo("1.2") < 0) {
// gwt-maven-plugin < 1.2 does not implement required Surefire option
return false;
Expand Down

0 comments on commit 80d71ca

Please sign in to comment.