fixed findbug issue and Scratch file copy issue.

Author: nbhoski

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

@@ -9,6 +9,8 @@
 import java.io.IOException;
 import java.io.InputStream;
 import java.util.UUID;
+import javax.annotation.Nonnull;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import hudson.EnvVars;
 import hudson.FilePath;
 import hudson.Launcher;
@@ -70,7 +72,9 @@ default void copyFileInWorkspace(String sourceFile, String targetFile, FilePath
         targetFilePath.chmod(0755);
     }
 
-    default FilePath getFilePathForUniqueFolder(Launcher launcher, String uniqueName,FilePath workspace)
+    @SuppressFBWarnings(value = "NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE",
+            justification = "workspace is passed through perform method, which will always have NonNull FilePath")
+    default FilePath getFilePathForUniqueFolder(Launcher launcher, String uniqueName, FilePath workspace)
             throws IOException, InterruptedException {
         /*Use of Computer is not recommended as jenkins hygeine for pipeline support
          * https://javadoc.jenkins-ci.org/jenkins/tasks/SimpleBuildStep.html */
@@ -85,6 +89,8 @@ default FilePath getFilePathForUniqueFolder(Launcher launcher, String uniqueName
         return new FilePath(launcher.getChannel(), tmpDir);
     }
 
+    @SuppressFBWarnings(value = "NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE",
+            justification = "workspace is passed through perform method, which will always have NonNull FilePath")
     default String getNodeSpecificTmpFolderPath(FilePath workspace) throws IOException, InterruptedException {
         Computer cmp = workspace.toComputer();
         String tmpDir = (String) cmp.getSystemProperties().get("java.io.tmpdir");

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

@@ -12,6 +12,7 @@ public class MatlabBuilderConstants {
     static final double BASE_MATLAB_VERSION_EXPORTSTMRESULTS_SUPPORT = 9.6;
 
     static final String MATLAB_RUNNER_TARGET_FILE = "Builder.matlab.runner.target.file.name";
+    static final String MATLAB_TESTS_RUNNER_TARGET_FILE = "runMatlabTests.m";
     static final String MATLAB_RUNNER_RESOURCE = "com/mathworks/ci/MatlabBuilder/runMatlabTests.m";
     static final String AUTOMATIC_OPTION = "RunTestsAutomaticallyOption";
 

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

@@ -292,7 +292,7 @@ private synchronized int execMatlabCommand(FilePath workspace, Launcher launcher
             // Copy MATLAB scratch file into the workspace.
             FilePath targetWorkspace = new FilePath(launcher.getChannel(), workspace.getRemote());
             copyFileInWorkspace(MatlabBuilderConstants.MATLAB_RUNNER_RESOURCE,
-                    MatlabBuilderConstants.MATLAB_RUNNER_TARGET_FILE, targetWorkspace);
+                    MatlabBuilderConstants.MATLAB_TESTS_RUNNER_TARGET_FILE, targetWorkspace);
             return matlabLauncher.join();
         } catch (Exception e) {
             listener.getLogger().println(e.getMessage());
@@ -308,8 +308,8 @@ private synchronized int execMatlabCommand(FilePath workspace, Launcher launcher
     }
 
     public String constructCommandForTest(String inputArguments) {
-        final String matlabFunctionName = FilenameUtils.removeExtension(
-                Message.getValue(MatlabBuilderConstants.MATLAB_RUNNER_TARGET_FILE));
+        final String matlabFunctionName =
+                FilenameUtils.removeExtension(MatlabBuilderConstants.MATLAB_TESTS_RUNNER_TARGET_FILE);
         final String runCommand = "exit(" + matlabFunctionName + "(" + inputArguments + "))";
         return runCommand;
     }

src/test/java/com/mathworks/ci/RunMatlabTestsBuilderTest.java

@@ -26,6 +26,7 @@
 import com.gargoylesoftware.htmlunit.WebAssert;
 import com.gargoylesoftware.htmlunit.html.HtmlCheckBoxInput;
 import com.gargoylesoftware.htmlunit.html.HtmlPage;
+import com.mathworks.ci.MatlabBuilder.RunTestsAutomaticallyOption;
 import hudson.FilePath;
 import hudson.matrix.Axis;
 import hudson.matrix.AxisList;
@@ -307,6 +308,20 @@ public void verifyMatrixBuildPasses() throws Exception {
 		jenkins.assertLogContains("R2018b completed", build);
 		jenkins.assertBuildStatus(Result.SUCCESS, build);
 	}
+	
+	 /*
+     * Test to verify if MATALB scratch file is generated in workspace.
+     */
+    @Test
+    public void verifyMATLABscratchFileGenerated() throws Exception {
+        this.buildWrapper.setMatlabRootFolder(getMatlabroot("R2018b"));  
+        project.getBuildWrappersList().add(this.buildWrapper);
+        setAllTestArtifacts(false, testBuilder);
+        project.getBuildersList().add(testBuilder);
+        FreeStyleBuild build = project.scheduleBuild2(0).get();
+        File matlabRunner = new File(build.getWorkspace() + File.separator + "runMatlabTests.m");
+        Assert.assertTrue(matlabRunner.exists());
+    }
 
 
     /*