Merge pull request #73 from uktrade/fix/move-sync-permission-error

fix: ensure the file has the right permission before moving

Author: PippoRaimondi

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))
   })
 

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)