JENKINS-18750 - Fix possible NPE in build graph if build doesn't yet …

…have a result #resolve #comment should be fixed in v1.3
jenkinsci · Jul 20, 2013 · f4f457e · f4f457e
1 parent ca1c5e3
commit f4f457e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/README b/README
@@ -19,6 +19,7 @@ Release Notes
 ###
 * Added: JENKINS-11899 - Print all URLs as HTML anchors (when escaping description and exception message)
 * Added: JENKINS-12275 - TestNG plugin should contribute to Dashboard View plugin
+* Fixed: JENKINS-18750 - NPE in testng graph
 * Fixed: JENKINS-17564 - Reporter output being escaped incorrectly
 
 ### v1.2

diff --git a/src/main/java/hudson/plugins/testng/TestNGProjectAction.java b/src/main/java/hudson/plugins/testng/TestNGProjectAction.java
@@ -213,7 +213,7 @@ protected void populateDataSetBuilder(DataSetBuilder<String, ChartUtil.NumberOnl
          ChartUtil.NumberOnlyBuildLabel label = new ChartUtil.NumberOnlyBuildLabel(build);
          TestNGTestResultBuildAction action = build.getAction(getBuildActionClass());
 
-         if (build.getResult().isWorseThan(Result.UNSTABLE)) {
+         if (build.getResult() == null || build.getResult().isWorseThan(Result.UNSTABLE)) {
             //We don't want to add aborted, failed or builds with no results into the graph
             continue;
          }