Add build results support in Run Command step and skipped task logs link in table (#354)

* add build table to command step and link to skipped logs

* add skipped log link in status as well

* fix pipeline failure

* temp commit for checkout

* update as per review comments

* remove MatlabBuild.java and update tests

* temp commit for checkout

* update as per review

* fix pipeline failure

* format code

* remove teardown method overriding

* add super to method calls

Author: mw-kapilg

.gitignore

@@ -7,4 +7,6 @@ src/main/resources/matlab-script-generator.zip
 src/main/resources/**/run-matlab-command*
 src/main/resources/license.txt
 
+.idea/
+
 **/.DS_Store

src/main/java/com/mathworks/ci/BuildArtifactAction.java

@@ -1,5 +1,10 @@
 package com.mathworks.ci;
 
+/**
+ * Copyright 2024 The MathWorks, Inc.
+ *
+ */
+
 import hudson.FilePath;
 import hudson.model.Action;
 import hudson.model.Run;
@@ -18,15 +23,13 @@
 import org.json.simple.parser.JSONParser;
 import org.json.simple.parser.ParseException;
 
-
 public class BuildArtifactAction implements Action {
     private Run<?, ?> build;
     private int totalCount;
     private int skipCount;
     private int failCount;
     private String actionID;
     private static final String ROOT_ELEMENT = "taskDetails";
-    private static final String BUILD_ARTIFACT_FILE = "buildArtifact";
 
     public BuildArtifactAction(Run<?, ?> build, String actionID) {
         this.build = build;
@@ -69,10 +72,10 @@ public List<BuildArtifactData> getBuildArtifact() throws ParseException, Interru
         FilePath fl;
         if(this.actionID == null){
             fl = new FilePath(new File(build.getRootDir().getAbsolutePath() + "/" +
-                    BUILD_ARTIFACT_FILE + ".json"));
+                    MatlabBuilderConstants.BUILD_ARTIFACT + ".json"));
         } else {
             fl = new FilePath(new File(build.getRootDir().getAbsolutePath() + "/" +
-                    BUILD_ARTIFACT_FILE + this.actionID + ".json"));
+                    MatlabBuilderConstants.BUILD_ARTIFACT + this.actionID + ".json"));
         }
         try (InputStreamReader reader = new InputStreamReader(new FileInputStream(new File(fl.toURI())), "UTF-8")) {
             Object obj = new JSONParser().parse(reader);
@@ -141,16 +144,15 @@ public void setOwner(Run owner) {
         this.build = owner;
     }
 
-
     private void setCounts() throws InterruptedException, ParseException {
         List<BuildArtifactData> artifactData = new ArrayList<BuildArtifactData>();
         FilePath fl;
         if(this.actionID == null){
             fl = new FilePath(new File(build.getRootDir().getAbsolutePath() + "/" +
-                    BUILD_ARTIFACT_FILE + ".json"));
+                    MatlabBuilderConstants.BUILD_ARTIFACT + ".json"));
         } else {
             fl = new FilePath(new File(build.getRootDir().getAbsolutePath() + "/" +
-                    BUILD_ARTIFACT_FILE + this.actionID + ".json"));
+                    MatlabBuilderConstants.BUILD_ARTIFACT + this.actionID + ".json"));
         }
         try (InputStreamReader reader = new InputStreamReader(new FileInputStream(new File(fl.toURI())), "UTF-8")) {
             Object obj = new JSONParser().parse(reader);

src/main/java/com/mathworks/ci/BuildArtifactData.java

@@ -1,5 +1,10 @@
 package com.mathworks.ci;
 
+/**
+ * Copyright 2024 The MathWorks, Inc.
+ *
+ */
+
 public class BuildArtifactData {
 
     private String taskName;
@@ -12,7 +17,6 @@ public class BuildArtifactData {
     public BuildArtifactData() {
     }
 
-
     public String getTaskDuration() {
         return this.taskDuration;
     }

src/main/java/com/mathworks/ci/BuildConsoleAnnotator.java

@@ -1,5 +1,10 @@
 package com.mathworks.ci;
 
+/**
+ * Copyright 2024 The MathWorks, Inc.
+ *
+ */
+
 import com.google.common.base.Charsets;
 import hudson.console.ConsoleLogFilter;
 import hudson.console.LineTransformationOutputStream;

src/main/java/com/mathworks/ci/BuildTargetNote.java

@@ -1,5 +1,10 @@
 package com.mathworks.ci;
 
+/**
+ * Copyright 2024 The MathWorks, Inc.
+ *
+ */
+
 import com.google.common.annotations.VisibleForTesting;
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import hudson.Extension;

src/main/java/com/mathworks/ci/FormValidationUtil.java

@@ -6,6 +6,7 @@
  * This is Utility class which provides commonly used methods for form validations across builders
  * 
  */
+
 import java.util.List;
 import java.util.function.Function;
 import hudson.util.FormValidation;

src/main/java/com/mathworks/ci/ListenerLogDecorator.java

@@ -1,4 +1,5 @@
 package com.mathworks.ci;
+
 /*
 * Copyright 2018 The MathWorks, Inc.
 */

src/main/java/com/mathworks/ci/MatlabBuild.java

@@ -1,4 +1,5 @@
 package com.mathworks.ci;
+
 /*
  * Copyright 2019-2020 The MathWorks, Inc.
  */
@@ -32,6 +33,12 @@ public class MatlabBuilderConstants {
 
     //Temporary MATLAB folder name in workspace 
     public static final String TEMP_MATLAB_FOLDER_NAME = ".matlab";
+
+    // MATLAB default function/plugin paths
+    public static final String DEFAULT_PLUGIN = "+ciplugins/+jenkins/getDefaultPlugins.m";
+    public static final String BUILD_REPORT_PLUGIN = "+ciplugins/+jenkins/BuildReportPlugin.m";
+    public static final String TASK_RUN_PROGRESS_PLUGIN = "+ciplugins/+jenkins/TaskRunProgressPlugin.m";
+    public static final String BUILD_ARTIFACT = "buildArtifact";
 
     public static final String NEW_LINE = System.getProperty("line.separator");
 

src/main/java/com/mathworks/ci/MatlabBuilderConstants.java

@@ -1,5 +1,10 @@
 package com.mathworks.ci;
 
+/**
+ * Copyright 2021 The MathWorks, Inc.
+ *
+ */
+
 import java.lang.Exception;
 
 public class MatlabExecutionException extends Exception {