Dev main (#357)

* Added check for command textbox

* Updated as per review comment

* Updated as per review comment

* 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

* Add skip reason for skipped tasks in build results table (#356)

* Add skipReason in build results table

* add skip reason texts

* add test and bump Maven task

* remove .idea from gitignore in this branch

* update as per review comments

* update as per review

* update function name

* update build results table

* update test

* update skip link

* update status to succeeded

* update data constructor

---------

Co-authored-by: Nikhil Bhoski <47204011+nbhoski@users.noreply.github.com>

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

azure-pipelines.yml

@@ -19,7 +19,7 @@ trigger:
   - master
 
 steps:
-  - task: Maven@3
+  - task: Maven@4
     inputs:
       mavenPomFile: 'pom.xml'
       mavenOptions: '-Xmx3072m -Dmaven.javadoc.skip=true'

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);
@@ -204,7 +206,7 @@ private void setCounts() throws InterruptedException, ParseException {
     private void iterateAllTaskAttributes(Entry pair, BuildArtifactData data) {
         // Iterates across all task attributes and updates
         String key = pair.getKey().toString();
-        switch(key.toLowerCase()){
+        switch(key){
             case "duration":
                 data.setTaskDuration(pair.getValue().toString());
                 break;
@@ -220,6 +222,26 @@ private void iterateAllTaskAttributes(Entry pair, BuildArtifactData data) {
             case "skipped":
                 data.setTaskSkipped((Boolean) pair.getValue());
                 break;
+            case "skipReason":
+                String skipReasonKey = pair.getValue().toString();
+                String skipReason;
+                switch(skipReasonKey){
+                    case "UpToDate":
+                        skipReason = "up-to-date";
+                        break;
+                    case "UserSpecified":
+                    case "UserRequested":
+                        skipReason = "user requested";
+                        break;
+                    case "DependencyFailed":
+                        skipReason = "dependency failed";
+                        break;
+                    default:
+                        skipReason = skipReasonKey;
+                        break;
+                }
+                data.setSkipReason(skipReason);
+                break;
             default :
                 break;
         }

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;
@@ -8,11 +13,11 @@ public class BuildArtifactData {
 
     private String taskDescription;
     private boolean taskSkipped;
+    private String skipReason;
 
     public BuildArtifactData() {
     }
 
-
     public String getTaskDuration() {
         return this.taskDuration;
     }
@@ -37,6 +42,14 @@ public void setTaskSkipped(boolean taskSkipped) {
         this.taskSkipped = taskSkipped;
     }
 
+    public String getSkipReason() {
+        return (this.skipReason == null) ? "" : this.skipReason;
+    }
+
+    public void setSkipReason(String skipReason) {
+        this.skipReason = skipReason;
+    }
+
     public boolean getTaskFailed() {
         return this.taskFailed;
     }

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");