Fix issue with cleanig up leftover images from previous run (#105)

Author: NimaSoroush

CHANGELOG.md

@@ -1,3 +1,7 @@
+## [1.5.1] - 2018-10-07
+### Fixed
+- Fix issue with cleanig up leftover images from previous runs
+
 ## [1.5.0] - 2018-08-30
 ### Added
 - Making Differencify compatible with differencify-jest-reporter

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "differencify",
-  "version": "1.5.0",
+  "version": "1.5.1",
   "description": "Perceptual diffing tool",
   "main": "dist/index.js",
   "scripts": {

package.json

@@ -22,6 +22,16 @@ const saveDiff = (diff, diffPath) => new Promise((resolve, reject) => {
   diff.image.write(diffPath, cb);
 });
 
+const cleanUpImages = (images) => {
+  images.forEach((image) => {
+    try {
+      fs.unlinkSync(image);
+    } catch (e) {
+      // ignore error as left over image may not exist
+    }
+  });
+};
+
 const compareImage = async (capturedImage, globalConfig, testConfig) => {
   const prefixedLogger = logger.prefix(testConfig.testName);
   const snapshotsDir = globalConfig.imageSnapshotPathProvided
@@ -33,6 +43,11 @@ const compareImage = async (capturedImage, globalConfig, testConfig) => {
   const diffDir = getDiffDir(snapshotsDir);
   const diffPath = getDiffPath(diffDir, testConfig);
 
+  const currentImageDir = getCurrentImageDir(snapshotsDir);
+  const currentImagePath = getCurrentImagePath(currentImageDir, testConfig);
+
+  cleanUpImages([diffPath, currentImagePath]);
+
   if (fs.existsSync(snapshotPath) && !testConfig.isUpdate) {
     let snapshotImage;
     try {
@@ -60,8 +75,6 @@ const compareImage = async (capturedImage, globalConfig, testConfig) => {
       };
     }
     if (globalConfig.saveCurrentImage) {
-      const currentImageDir = getCurrentImageDir(snapshotsDir);
-      const currentImagePath = getCurrentImagePath(currentImageDir, testConfig);
       try {
         if (!fs.existsSync(currentImageDir)) {
           fs.mkdirSync(currentImageDir);