[JENKINS-76360] Fix NPE on TAP Test Results screen

src/main/java/org/tap4j/plugin/TapTestResultAction.java

@@ -30,6 +30,7 @@
 import hudson.model.HealthReportingAction;
 import hudson.model.Job;
 import hudson.model.Run;
+import hudson.tasks.test.AbstractTestResultAction;
 import jenkins.model.RunAction2;
 import jenkins.tasks.SimpleBuildStep;
 import jenkins.util.NonLocalizable;
@@ -44,13 +45,9 @@
 /**
  * @since 0.1
  */
-public class TapTestResultAction
+public class TapTestResultAction extends AbstractTestResultAction<TapTestResultAction>
         implements StaplerProxy, SimpleBuildStep.LastBuildAction, HealthReportingAction, RunAction2 {
 
-    public transient Run<?, ?> run;
-    @Deprecated
-    public transient AbstractBuild<?, ?> owner;
-
     private TapResult tapResult;
 
     protected TapTestResultAction(Run<?, ?> r, TapResult tapResult) {
@@ -59,6 +56,13 @@
         this.tapResult = tapResult;
     }
 
+    /**
+     * @return the current Run to support new JUnit
+     */
+    public Run<?, ?> getObject() {
+        return run;
+    }
+
     /**
      * @return the tapResult
      */
@@ -99,7 +103,7 @@
     }
 
     public TapStreamResult getResult() {
-        return new TapStreamResult(owner, tapResult);
+        return new TapStreamResult(owner, tapResult, this);
     }
 
     /* (non-Javadoc)

src/main/java/org/tap4j/plugin/model/TapStreamResult.java

@@ -25,6 +25,7 @@
 
 import hudson.model.Run;
 import hudson.tasks.junit.CaseResult;
+import hudson.tasks.test.AbstractTestResultAction;
 import hudson.tasks.test.TabulatedResult;
 import hudson.tasks.test.TestObject;
 import hudson.tasks.test.TestResult;
@@ -33,6 +34,7 @@
 import org.kohsuke.stapler.export.Exported;
 import org.tap4j.model.TestSet;
 import org.tap4j.plugin.TapResult;
+import org.tap4j.plugin.TapTestResultAction;
 import org.tap4j.util.StatusValues;
 
 import javax.annotation.Nullable;
@@ -50,12 +52,14 @@
 
     private static final long serialVersionUID = 8337146933697574082L;
     private final transient Run<?, ?> owner;
+    private final transient AbstractTestResultAction tapTestResultAction;
     private List<TestResult> children = new ArrayList<>();
     private TapResult tapResult;
 
-    public TapStreamResult(Run<?, ?> owner, TapResult tapResult) {
+    public TapStreamResult(Run<?, ?> owner, TapResult tapResult, AbstractTestResultAction tapTestResultAction) {
         this.owner = owner;
         this.tapResult = tapResult;
+        this.tapTestResultAction = tapTestResultAction;
         setChildrenInfo();
     }
 
@@ -75,6 +79,15 @@
         return owner;
     }
 
+    public String getTitle() {
+        return getDisplayName();
+    }
+
+    @Override
+    public AbstractTestResultAction getParentAction() {
+        return tapTestResultAction;
+    }
+
     /* (non-Javadoc)
      * @see hudson.tasks.test.TestObject#getParent()
      */

src/main/java/org/tap4j/plugin/model/TapTestResultResult.java

@@ -106,7 +106,7 @@
             TestSetMap subTest = new TestSetMap(testSetMap.getFileName(), testSet);
             List<TestSetMap> list = new ArrayList<>();
             list.add(subTest);
-            parent = new TapStreamResult(owner, new TapResult("TAP Test Results", owner, list, todoIsFailure, includeCommentDiagnostics, validateNumberOfTests));
+            parent = new TapStreamResult(owner, new TapResult("TAP Test Results", owner, list, todoIsFailure, includeCommentDiagnostics, validateNumberOfTests), getTestResultAction());
         }
         return parent;
     }