Fixing issue with freeze image (#94)

Fixing issue with freeze image

Author: NimaSoroush

API.md

@@ -441,5 +441,25 @@ In this example, differencify will got to different pages and compare screenshot
 In this example, you can specify the custom path for storing images.
 
 
+## Freezing an image
+
+```js
+(async () => {
+  await differencify
+    .init()
+    .newPage()
+    .setViewport({ width: 1600, height: 1200 })
+    .goto('https://i.giphy.com/media/xTiTnoUnHxVaaVNWhO/giphy.webp')
+    .waitFor('body > img')
+    .freezeImage('body > img')
+    .screenshot()
+    .toMatchSnapshot()
+    .close()
+    .end();
+})();
+```
+In this example, you can freeze an image by specifying the selector path.
+
+
 
 

CHANGELOG.md

@@ -1,3 +1,7 @@
+## [1.4.2] - 2018-7-13
+### Fixed
+- Fixing issue with freezeImage command
+
 ## [1.4.1] - 2018-6-22
 ### Added
 - A prepare script to keep the artifact uptodated

CONTRIBUTING.md

@@ -12,17 +12,9 @@ Differencify operates under a forking model. In order to contribute, please fork
 
 ### steps
 - Open terminal
-- cd to project root and build image
-```
-docker build -t differencify .
-```
-- And then run
-```
-docker run -t -d --name differencify differencify
-```
 - To run integration tests run
 ```
-docker exec -it differencify npm run test:integration
+npm run test:integration
 ```
 - Then copy over generated snapshots
 ```

package-lock.json

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

package.json

@@ -13,7 +13,7 @@ const functionToString = (func, ...args) => {
     }
   });
   const functionSource = func.toString();
-  return `return (${functionSource})(${funcArguments.join()})`;
+  return `(${functionSource})(${funcArguments.join()})`;
 };
 
 export default functionToString;

src/helpers/functionToString.js

@@ -5,11 +5,13 @@ const foo = function foo(a, b, c) { return a + b + c; };
 describe('functionToString', () => {
   it('Convert function to string with right arguments', () => {
     const strFunc = functionToString(foo, 1, 2, '3');
-    expect(strFunc).toEqual('return (function foo(a, b, c) {return a + b + c;})(1,2,"3")');
+    expect(strFunc).toEqual('(function foo(a, b, c) {return a + b + c;})(1,2,"3")');
+    expect(eval(strFunc)).toEqual('33'); // eslint-disable-line no-eval
   });
   it('Convert function to string with no arguments', () => {
     const strFunc = functionToString(foo);
-    expect(strFunc).toEqual('return (function foo(a, b, c) {return a + b + c;})()');
+    expect(strFunc).toEqual('(function foo(a, b, c) {return a + b + c;})()');
+    expect(eval(strFunc)).toEqual(NaN); // eslint-disable-line no-eval
   });
   it('Return null if non function passed', () => {
     const strFunc = functionToString('function');

src/helpers/functionToString.test.js

@@ -268,4 +268,17 @@ describe('Differencify', () => {
     await customDifferencify.cleanup();
     expect(result).toEqual(true);
   }, 30000);
+  it('Freeze image in page', async () => {
+    await differencify
+      .init()
+      .newPage()
+      .setViewport({ width: 1600, height: 1200 })
+      .goto('https://i.giphy.com/media/xTiTnoUnHxVaaVNWhO/giphy.webp')
+      .waitFor('body > img')
+      .freezeImage('body > img')
+      .screenshot()
+      .toMatchSnapshot()
+      .close()
+      .end();
+  }, 30000);
 });