fix: ensure the file has the right permission before moving

haim-io · Mar 10, 2022 · a1535e6 · a1535e6
1 parent 59a9386
commit a1535e6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/plugin.js b/src/plugin.js
@@ -133,6 +133,7 @@ const getCompareSnapshotsPlugin = on => {
   on('after:screenshot', details => {
     // Change screenshots file permission so it can be moved from drive to drive
     setFilePermission(details.path, 0o777)
+    setFilePermission(paths.image.comparison(details.name), 0o777)
     renameAndMoveFile(details.path, paths.image.comparison(details.name))
   })
 

diff --git a/src/utils.js b/src/utils.js
@@ -23,8 +23,10 @@ const readDir = dir => {
 
 const setFilePermission = (dir, permission) => {
   try {
-    const fd = fs.openSync(dir, 'r')
-    fs.fchmodSync(fd, permission)
+    if (fs.existsSync(dir)) {
+      const fd = fs.openSync(dir, 'r')
+      fs.fchmodSync(fd, permission)
+    }
   } catch (error) {
     // eslint-disable-next-line no-console
     console.log(error)