Exit code and slash instead of backslash

Author: denise-khuu

localBuildRun.sh

@@ -6,5 +6,6 @@ cp -r runners build
 npm test
 if [ $? -eq 1 ]; then 
     echo 'tests failed' 
+    exit 1
 fi
 node build/engine/run.js

localBuildTest.sh

@@ -2,4 +2,5 @@ tsc
 npm test
 if [ $? -eq 1 ]; then 
     echo 'tests failed' 
+    exit 1
 fi

runners/katacoda/dirUtils.ts

@@ -19,14 +19,14 @@ export class DirUtils{
 
         //saves the remaining path, if currentdir is the prefix of targetDir
         if(targetDir.substring(0,currentDir.length) == currentDir){
-            return path.join(targetDir.replace(currentDir + path.sep, ''));
+            return path.join(targetDir.replace(currentDir + path.sep, '')).replace("\\", "/");
         }
 
 
         else{
             //returns the absolut directory, if the first parent folder is different
             if(currentPaths[1] != targetPaths[1]){
-                return targetDir;
+                return targetDir.replace("\\", "/");
             }
 
             //iterates throught currentPath array to compare parent directories
@@ -42,7 +42,7 @@ export class DirUtils{
             //slice targetPaths to get the relative path
             targetPaths = targetPaths.slice(index + 1, targetPaths.length);
 
-            return path.join(dirPath, targetPaths.join(path.sep));
+            return path.join(dirPath, targetPaths.join(path.sep)).replace("\\", "/");
 
         }
 

spec/runners/katacoda/dirUtilsSpec.ts

@@ -9,16 +9,16 @@ describe("DirUtils", () => {
       expect(target.getCdParam(path.join('/root'), path.join('/root'))).toBe('');
     });
     it("changes directly to the child folder, because currentDir is the prefix of dir", () => {
-      expect(target.getCdParam(path.join('/root/devonfw'), path.join('/root/devonfw/setup'))).toBe(path.join('setup'));
+      expect(target.getCdParam(path.join('/root/devonfw'), path.join('/root/devonfw/setup'))).toBe(path.join('setup').replace("\\", "/"));
     });
     it("returns an absolute path, because both dirs don't have matching parent folders", () => {
-      expect(target.getCdParam(path.join('/setup'), path.join('/root/devonfw/setup'))).toBe(path.join('/root/devonfw/setup'));
+      expect(target.getCdParam(path.join('/setup'), path.join('/root/devonfw/setup'))).toBe(path.join('/root/devonfw/setup').replace("\\", "/"));
     });
     it("changes to parent folder before changing to child folder", () => {
-      expect(target.getCdParam(path.join('/root/devonfw'), path.join('/root/setup/folder0/folder1'))).toBe(path.join('../setup/folder0/folder1'));
+      expect(target.getCdParam(path.join('/root/devonfw'), path.join('/root/setup/folder0/folder1'))).toBe(path.join('../setup/folder0/folder1').replace("\\", "/"));
     });
     it("changes to parent folder before changing to child folder and one child folder has the same position and name", () => {
-      expect(target.getCdParam(path.join('/root/devonfw/folder/setup'), path.join('/root/devonfw/setup/setup'))).toBe(path.join('../../setup/setup'));
+      expect(target.getCdParam(path.join('/root/devonfw/folder/setup'), path.join('/root/devonfw/setup/setup'))).toBe(path.join('../../setup/setup').replace("\\", "/"));
     });
 
   });