consoldiated copy operation

Author: Nikhil Bhoski

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

@@ -65,6 +65,10 @@ private synchronized int execMatlabCommand(FilePath workspace, Launcher launcher
             ProcStarter matlabLauncher = getProcessToRunMatlabCommand(workspace, launcher, listener,
                     envVars, cmdPrefix + envVars.expand(getCommand()), uniqueTmpFldrName);
 
+            //Copy Scratch file needed to run MATLAB tests in workspace
+            copyFileInWorkspace(MatlabBuilderConstants.MATLAB_TESTS_RUNNER_RESOURCE,
+                    MatlabBuilderConstants.MATLAB_TESTS_RUNNER_TARGET_FILE, workspace);
+            
             //prepare temp folder by coping genscript package.
             prepareTmpFldr(genScriptLocation);
 

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

@@ -1,11 +1,4 @@
 package com.mathworks.ci;
-/**
- * Copyright 2020 The MathWorks, Inc.
- *  
- */
-import java.io.IOException;
-import java.io.InputStream;
-
 /**
  * Copyright 2020 The MathWorks, Inc.
  *  
@@ -104,16 +97,9 @@ public void setModelCoverageCobertura(String modelCoverageCobertura) {
 
     @Override
     public StepExecution start(StepContext context) throws Exception {
-        Launcher launcher = context.get(Launcher.class);
-        FilePath workspace = context.get(FilePath.class);
-        
-        //Copy Scratch file needed to run MATLAB tests in workspace
-        FilePath targetWorkspace = new FilePath(launcher.getChannel(), workspace.getRemote());
-        copyScratchFileInWorkspace(MatlabBuilderConstants.MATLAB_TESTS_RUNNER_RESOURCE,
-                MatlabBuilderConstants.MATLAB_TESTS_RUNNER_TARGET_FILE, targetWorkspace);
-        return new MatlabRunTestsStepExecution(context,constructCommandForTest(getInputArgs()));
+        return new MatlabRunTestsStepExecution(context, constructCommandForTest(getInputArgs()));
     }
-    
+
     @Extension
     public static class RunTestsStepDescriptor extends StepDescriptor {
 
@@ -169,17 +155,4 @@ private Map<String, String> getMatlabArgs() {
         args.put("CoberturaModelCoveragePath", getModelCoverageCobertura());
         return args;
     }
-    
-    /*
-     * Method to copy given file from source to target node specific workspace.
-     */
-    private void copyScratchFileInWorkspace(String sourceFile, String targetFile, FilePath targetWorkspace)
-            throws IOException, InterruptedException {
-        final ClassLoader classLoader = getClass().getClassLoader();
-        FilePath targetFilePath = new FilePath(targetWorkspace, targetFile);
-        InputStream in = classLoader.getResourceAsStream(sourceFile);
-        targetFilePath.copyFrom(in);
-        // set executable permission
-        targetFilePath.chmod(0755);
-    }
 }