diff --git a/changelog.html b/changelog.html index b31454c907eaae37cedb98fbe5eb3b359c35b9c9..3bf5879c2696759db4597b0b3177f79e14dbad47 100644 --- a/changelog.html +++ b/changelog.html @@ -61,6 +61,9 @@ Upcoming changes
  • “Build” from job context menu produced a confusing warning page. (issue 16844) +
  • + Maven2 builds with non-standard test plugins failed. + (issue 16928) diff --git a/maven-plugin/src/main/java/hudson/maven/reporters/SurefireArchiver.java b/maven-plugin/src/main/java/hudson/maven/reporters/SurefireArchiver.java index 8341b853533c74f92d9cea02589242f0db287adb..52c8cf798833a2f434b18515e773c79b8cbbbc92 100644 --- a/maven-plugin/src/main/java/hudson/maven/reporters/SurefireArchiver.java +++ b/maven-plugin/src/main/java/hudson/maven/reporters/SurefireArchiver.java @@ -93,7 +93,7 @@ public class SurefireArchiver extends TestFailureDetector { // Many test plugins have - as surefire - a configuration key 'testFailureIgnore' which defaults to // ${maven.test.failure.ignore}, so just try that one and change value to true, // if it's still at that default: - XmlPlexusConfiguration c = (XmlPlexusConfiguration) mojo.configuration.getChild("testFailureIgnore"); + XmlPlexusConfiguration c = (XmlPlexusConfiguration) mojo.configuration.getChild("testFailureIgnore",false); if(c!=null && c.getValue() != null && c.getValue().equals("${maven.test.failure.ignore}") && System.getProperty("maven.test.failure.ignore")==null) { if (build.getMavenBuildInformation().isMaven3OrLater()) { String fieldName = "testFailureIgnore";