Skip to content
Permalink
Browse files

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

…have a result

#resolve #comment should be fixed in v1.3
  • Loading branch information
nullin committed Jul 20, 2013
1 parent ca1c5e3 commit f4f457e1a95b18773a59f1529e732b2656a52fb7
Showing with 2 additions and 1 deletion.
  1. +1 −0 README
  2. +1 −1 src/main/java/hudson/plugins/testng/TestNGProjectAction.java
1 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
@@ -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;
}

0 comments on commit f4f457e

Please sign in to comment.
You can’t perform that action at this time.