fix: pass current image path to reference image error

gemini-testing · Aug 31, 2017 · 95e9286 · 95e9286
1 parent a1c409d
commit 95e9286
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 4 deletions.
diff --git a/lib/state-processor/capture-processor/index.js b/lib/state-processor/capture-processor/index.js
@@ -4,8 +4,13 @@ const Promise = require('bluebird');
 const CaptureProcessor = require('./capture-processor');
 const utils = require('./utils');
 const NoRefImageError = require('../../errors/no-ref-image-error');
+const temp = require('../../temp');
 
-const throwNoRefError = (referencePath) => Promise.reject(new NoRefImageError(referencePath));
+const throwNoRefError = (referencePath, capture) => {
+    const currentPath = temp.path({suffix: '.png'});
+    return capture.image.save(currentPath)
+        .then(() => Promise.reject(new NoRefImageError(referencePath, currentPath)));
+};
 const notUpdated = (referencePath) => ({imagePath: referencePath, updated: false});
 
 const saveRef = (referencePath, capture) => {

diff --git a/test/unit/state-processor/capture-processor/capture-processor.js b/test/unit/state-processor/capture-processor/capture-processor.js
@@ -109,10 +109,35 @@ describe('state-processor/capture-processor/capture-processor', () => {
             exec_ = mkExecMethod(tester);
         });
 
-        it('should be rejected with "NoRefImageError" if reference image does not exist', () => {
-            utils.existsRef.resolves(false);
+        describe('reference image does not exist', () => {
+            beforeEach(() => {
+                utils.existsRef.withArgs('/non-existent/path').resolves(false);
+            });
+
+            it('should save current image to a temporary directory', () => {
+                temp.path.withArgs({suffix: '.png'}).returns('/temp/path');
+
+                return exec_({referencePath: '/non-existent/path'})
+                    .catch(() => assert.calledOnceWith(Image.prototype.save, '/temp/path'));
+            });
 
-            return assert.isRejected(exec_({refPath: '/non-existent/path'}), NoRefImageError);
+            it('should be rejected with "NoRefImageError"', () => {
+                return assert.isRejected(exec_({referencePath: '/non-existent/path'}), NoRefImageError);
+            });
+
+            it('should pass reference and current image paths to "NoRefImageError"', () => {
+                sandbox.stub(NoRefImageError.prototype, '__constructor');
+                temp.path.withArgs({suffix: '.png'}).returns('/temp/path');
+
+                return exec_({referencePath: '/non-existent/path'})
+                    .catch(() => {
+                        assert.calledOnceWith(
+                            NoRefImageError.prototype.__constructor,
+                            '/non-existent/path',
+                            '/temp/path'
+                        );
+                    });
+            });
         });
 
         describe('should return image comparison result if images are', () => {