JENKINS-54559 set the result on the build on not on the context`

jenkinsci · Nov 11, 2018 · c355a75 · c355a75
1 parent 5644473
commit c355a75
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/...in/src/main/java/org/jenkinsci/plugins/pipeline/maven/publishers/JunitTestsPublisher.java b/...in/src/main/java/org/jenkinsci/plugins/pipeline/maven/publishers/JunitTestsPublisher.java
@@ -316,11 +316,12 @@ private void executeReporter(StepContext context, TaskListener listener, List<El
                         listener.getLogger().println("[withMaven] junitPublisher - unit tests are all successful");
                     }
                 } else {
-                    // TODO: Once JENKINS-43995 lands, update this to set the step status instead of the entire build.
                     if (LOGGER.isLoggable(Level.FINE)) {
                         listener.getLogger().println("[withMaven] junitPublisher - " + testResultAction.getResult().getFailCount() + " unit test failure(s) found, mark job as unstable");
                     }
-                    context.setResult(Result.UNSTABLE);
+                    // TODO: Once JENKINS-43995 lands, update this to set the step status instead of the entire build.
+                    // context.setResult(Result.UNSTABLE);
+                    run.setResult(Result.UNSTABLE);
                 }
             } catch (Exception e) {
                 listener.error("[withMaven] junitPublisher - Silently ignore exception archiving JUnit results for Maven artifact " + mavenArtifact.toString() + " generated by " + pluginInvocation + ": " + e);