Updated as per review comments

Author: nbhoski

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

@@ -45,6 +45,7 @@ public BuildArtifactAction(Run<?, ?> build, String actionID) {
     public String getActionID(){
         return this.actionID;
     }
+
     @CheckForNull
     @Override
     public String getIconFileName() {
@@ -66,7 +67,8 @@ public String getUrlName() {
     public List<BuildArtifactData> getBuildArtifact() throws ParseException, InterruptedException, IOException {
         List<BuildArtifactData> artifactData = new ArrayList<BuildArtifactData>();
 
-        FilePath fl = new FilePath(new File(build.getRootDir().getAbsolutePath() + "/" + BUILD_ARTIFACT_FILE + this.actionID +".json"));
+        FilePath fl = new FilePath(new File(build.getRootDir().getAbsolutePath() + "/" +
+                BUILD_ARTIFACT_FILE + this.actionID + ".json"));
         try (InputStreamReader reader = new InputStreamReader(new FileInputStream(new File(fl.toURI())), "UTF-8")) {
             Object obj = new JSONParser().parse(reader);
             JSONObject jo = (JSONObject) obj;

src/main/java/com/mathworks/ci/actions/RunMatlabBuildAction.java

@@ -35,12 +35,9 @@ public String getActionID(){
         return this.actionID;
     }
 
-    private void setActionID(){
-        this.actionID = RandomStringUtils.randomAlphanumeric(8);
-    }
-
     public RunMatlabBuildAction(MatlabCommandRunner runner, BuildConsoleAnnotator annotator, BuildActionParameters params) {
         this.runner = runner;
+        this.actionID = RandomStringUtils.randomAlphanumeric(8);
         this.annotator = annotator;
         this.params = params;
     }
@@ -54,8 +51,6 @@ public RunMatlabBuildAction(BuildActionParameters params) throws IOException, In
     }
 
     public void run() throws IOException, InterruptedException, MatlabExecutionException {
-        // set unique action ID for each build task
-        this.setActionID();
         // Copy plugins and override default plugins function
         runner.copyFileToTempFolder(DEFAULT_PLUGIN, DEFAULT_PLUGIN);
         runner.copyFileToTempFolder(BUILD_REPORT_PLUGIN, BUILD_REPORT_PLUGIN);
@@ -126,4 +121,4 @@ public void run() throws IOException, InterruptedException, MatlabExecutionExcep
         }
 
     }
-}
+}

src/test/java/integ/com/mathworks/ci/BuildArtifactActionTest.java

@@ -78,7 +78,7 @@ public void verifyBuildArtifactsReturned() throws ExecutionException, Interrupte
         copyFileInWorkspace("buildArtifacts/t1/buildArtifact.json",targetFile,artifactRoot);
         List<BuildArtifactData> ba = ac.getBuildArtifact();
         int expectedSize = ba.size();
-        Assert.assertEquals("The build names are not matching",3,expectedSize);
+        Assert.assertEquals("Incorrect build artifact",3,expectedSize);
     }
 
     /**