use platform agnostic paths everywhere

Author: jsajal

packages/flowtest-cli/graph/Graph.js

@@ -127,6 +127,7 @@ class Graph {
         this.logger.add(LogLevel.INFO, '', { type: 'delayNode', data: { delay } });
         result = {
           status: 'Success',
+          data: prevNodeOutputData,
         };
       }
 
@@ -136,6 +137,7 @@ class Graph {
         this.auth = node.data.type ? aNode.evaluate() : undefined;
         result = {
           status: 'Success',
+          data: prevNodeOutputData,
         };
       }
 
@@ -170,6 +172,7 @@ class Graph {
         } else {
           result = {
             status: 'Success',
+            data: prevNodeOutputData,
           };
         }
       }
@@ -194,6 +197,7 @@ class Graph {
         }
         result = {
           status: 'Success',
+          data: prevNodeOutputData,
         };
       }
 

packages/flowtest-electron/preload.js

@@ -1,10 +1,12 @@
 const { ipcRenderer, contextBridge } = require('electron');
 const path = require('path');
+const { isMacOS } = require('./src/utils/filemanager/filesystem');
 
 contextBridge.exposeInMainWorld('ipcRenderer', {
   invoke: (channel, ...args) => ipcRenderer.invoke(channel, ...args),
   on: (channel, handler) => ipcRenderer.on(channel, (event, ...args) => handler(...args)),
   join: (...args) => path.join(...args),
   relative: (...args) => path.relative(...args),
   dirname: (...args) => path.dirname(...args),
+  isMacOs: isMacOS,
 });

packages/flowtest-electron/src/utils/filemanager/createdirectory.js

@@ -1,23 +1,24 @@
 const fs = require('fs');
-const { isDirectory, concatRoute, pathExists } = require('./filesystem');
+const { isDirectory, pathExists } = require('./filesystem');
+const path = require('path');
 
-const createDirectory = (name, path) => {
+const createDirectory = (name, basePath) => {
   // now validate the name and path
   if (!name) {
     throw new Error('Directory name is required');
   }
 
-  if (!path) {
+  if (!basePath) {
     throw new Error('Directory path is required');
   }
 
   // check if the directory exists
-  if (!isDirectory(path)) {
+  if (!isDirectory(basePath)) {
     throw new Error('Path is not a directory');
   }
 
   // check if the directory already exists
-  const directoryPath = concatRoute(path, name);
+  const directoryPath = path.join(basePath, name);
 
   if (isDirectory(directoryPath)) {
     throw new Error('The directory already exists');

packages/flowtest-electron/src/utils/filemanager/filesystem.js

@@ -25,25 +25,6 @@ const pathExists = (path) => {
   }
 };
 
-function trim(text) {
-  return String(text).replace(/^\/|\/$/g, '');
-}
-
-/**
- * Concatenate the given paths to one single path
- *
- * @param   {...string} segments
- * @returns {string}
- */
-const concatRoute = (...segments) => {
-  let path = segments
-    .filter((value) => value && String(value).length > 0)
-    .map((segment) => '/' + trim(segment))
-    .join('');
-
-  return '/' + trim(path.replace(/(\/)+/g, '/'));
-};
-
 const getSubdirectoriesFromRoot = (rootPath, pathname) => {
   // convert to unix style path
   pathname = slash(pathname);
@@ -68,6 +49,8 @@ const isMacOS = () => {
   return process.platform === 'darwin';
 };
 
+const PATH_SEPARATOR = isWindowsOS() ? '\\' : '/';
+
 const slash = (path) => {
   const isExtendedLengthPath = /^\\\\\?\\/.test(path);
 
@@ -78,13 +61,11 @@ const slash = (path) => {
   return path.replace(/\\/g, '/');
 };
 
-const PATH_SEPARATOR = isWindowsOS() ? '\\' : '/';
-
 module.exports = {
   isDirectory,
   pathExists,
-  concatRoute,
   getSubdirectoriesFromRoot,
   getDirectoryName,
+  isMacOS,
   PATH_SEPARATOR,
 };

src/components/molecules/flow/graph/Graph.js

@@ -112,6 +112,7 @@ class Graph {
         this.logger.add(LogLevel.INFO, '', { type: 'delayNode', data: delay });
         result = {
           status: 'Success',
+          data: prevNodeOutputData,
         };
       }
 
@@ -120,6 +121,7 @@ class Graph {
         this.auth = node.data.type ? aNode.evaluate() : undefined;
         result = {
           status: 'Success',
+          data: prevNodeOutputData,
         };
       }
 
@@ -151,6 +153,7 @@ class Graph {
         } else {
           result = {
             status: 'Success',
+            data: prevNodeOutputData,
           };
         }
       }
@@ -173,6 +176,7 @@ class Graph {
         }
         result = {
           status: 'Success',
+          data: prevNodeOutputData,
         };
       }