Merge pull request #124 from mathworks/revert-122-temp_path_issue

Revert "Fix Java Temp Path discrepancy for Window"

Author: abhargav-mw

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

@@ -76,21 +76,16 @@ default FilePath getFilePathForUniqueFolder(Launcher launcher, String uniqueName
          * https://javadoc.jenkins-ci.org/jenkins/tasks/SimpleBuildStep.html */
 
         String tmpDir = getNodeSpecificTmpFolderPath(workspace);
-
-        return new FilePath(launcher.getChannel(), tmpDir + "/" + uniqueName);
+        return new FilePath(launcher.getChannel(), tmpDir+"/"+uniqueName);
     }
 
     default String getNodeSpecificTmpFolderPath(FilePath workspace) throws IOException, InterruptedException {
         Computer cmp = workspace.toComputer();
         if (cmp == null) {
             throw new IOException(Message.getValue("build.workspace.computer.not.found"));
         }
-        
-        String tmpDirPath = (String) cmp.getSystemProperties().get("java.io.tmpdir");
-
-        // Invoke FilePath.normalize for clean file path on any channel.
-        FilePath tmpDir = new FilePath(cmp.getChannel(), tmpDirPath);
-        return tmpDir.getRemote();
+        String tmpDir = (String) cmp.getSystemProperties().get("java.io.tmpdir");
+        return tmpDir;
     }
 
     default String getUniqueNameForRunnerFile() {

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

@@ -249,9 +249,6 @@ public void verifyCommandSupportsEnvVar() throws Exception {
 
     /*
      * Test to verify if appropriate MATALB runner file is copied in workspace.
-     * 
-     * NOTE: This test assumes there is no MATLAB installed and is not on System Path.
-     * 
      */
     @Test
     public void verifyMATLABrunnerFileGenerated() throws Exception {
@@ -265,9 +262,6 @@ public void verifyMATLABrunnerFileGenerated() throws Exception {
 
 	/*
 	 * Test to verify if Matrix build fails when MATLAB is not available.
-     * 
-     * NOTE: This test assumes there is no MATLAB installed and is not on System Path.
-     * 
 	 */
 	@Test
 	public void verifyMatrixBuildFails() throws Exception {

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

@@ -334,9 +334,6 @@ public void veriyEmptyParameters() throws Exception {
 
     /*
      * Test to verify if appropriate MATALB runner file is copied in workspace.
-     * 
-     * NOTE: This test assumes there is no MATLAB installed and is not on System Path.
-     *
      */
     @Test
     public void verifyMATLABrunnerFileGeneratedForAutomaticOption() throws Exception {
@@ -349,9 +346,6 @@ public void verifyMATLABrunnerFileGeneratedForAutomaticOption() throws Exception
 
 	/*
 	 * Test to verify if Matrix build fails when MATLAB is not available.
-     * 
-     * NOTE: This test assumes there is no MATLAB installed and is not on System Path.
-     *
 	 */
 	@Test
 	public void verifyMatrixBuildFails() throws Exception {