@probe.gl/test-utils - npm Package Compare versions

Comparing version 3.0.0-alpha.3 to 3.0.0-alpha.4

dist/es5/browser-automation/browser-test-driver.js

@@ -217,3 +217,3 @@ "use strict";
       }).then(function (result) {
-        if (!result.success && opts.saveOnFail) {
+        if (!result.success && opts.saveOnFail && result.source1) {
           var filename = opts.saveAs || '[name]-failed.png';
@@ -227,6 +227,7 @@ filename = filename.replace('[name]', opts.goldenImage.replace(/\.\w+$/, ''));
           headless: _this3.headless,
-          match: result.match,
-          matchPercentage: result.matchPercentage,
+          match: result.match || 0,
+          matchPercentage: result.matchPercentage || 'N/A',
           success: result.success,
-          diffImage: result.diffImage
+          diffImage: result.diffImage || null,
+          error: result.error || null
         };
@@ -233,0 +234,0 @@ }).catch(function (error) {

dist/es5/utils/diff-images.js

@@ -25,3 +25,9 @@ "use strict";
 
-    var result = diffPNGs(image1, image2, options);
+    return diffPNGs(image1, image2, options);
+  }).catch(function (error) {
+    return {
+      success: false,
+      error: error.message
+    };
+  }).then(function (result) {
     return Object.assign(result, {
@@ -28,0 +34,0 @@ source1: source1,

dist/es6/browser-automation/browser-test-driver.js

@@ -152,3 +152,3 @@ import BrowserDriver from './browser-driver';
     return this.page.screenshot(screenshotOptions).then(image => diffImages(image, opts.goldenImage, opts)).then(result => {
-      if (!result.success && opts.saveOnFail) {
+      if (!result.success && opts.saveOnFail && result.source1) {
         let filename = opts.saveAs || '[name]-failed.png';
@@ -162,6 +162,7 @@ filename = filename.replace('[name]', opts.goldenImage.replace(/\.\w+$/, ''));
         headless: this.headless,
-        match: result.match,
-        matchPercentage: result.matchPercentage,
+        match: result.match || 0,
+        matchPercentage: result.matchPercentage || 'N/A',
         success: result.success,
-        diffImage: result.diffImage
+        diffImage: result.diffImage || null,
+        error: result.error || null
       };
@@ -168,0 +169,0 @@ }).catch(error => {

dist/es6/utils/diff-images.js

@@ -5,10 +5,10 @@ import fs from 'fs';
 export default function diffImages(source1, source2, options = {}) {
-  return Promise.all([parsePNG(source1), parsePNG(source2)]).then(([image1, image2]) => {
-    const result = diffPNGs(image1, image2, options);
-    return Object.assign(result, {
-      source1,
-      source2,
-      options
-    });
-  });
+  return Promise.all([parsePNG(source1), parsePNG(source2)]).then(([image1, image2]) => diffPNGs(image1, image2, options)).catch(error => ({
+    success: false,
+    error: error.message
+  })).then(result => Object.assign(result, {
+    source1,
+    source2,
+    options
+  }));
 }
@@ -15,0 +15,0 @@

dist/esm/browser-automation/browser-test-driver.js

@@ -200,3 +200,3 @@ import _classCallCheck from "@babel/runtime/helpers/esm/classCallCheck";
       }).then(function (result) {
-        if (!result.success && opts.saveOnFail) {
+        if (!result.success && opts.saveOnFail && result.source1) {
           var filename = opts.saveAs || '[name]-failed.png';
@@ -210,6 +210,7 @@ filename = filename.replace('[name]', opts.goldenImage.replace(/\.\w+$/, ''));
           headless: _this3.headless,
-          match: result.match,
-          matchPercentage: result.matchPercentage,
+          match: result.match || 0,
+          matchPercentage: result.matchPercentage || 'N/A',
           success: result.success,
-          diffImage: result.diffImage
+          diffImage: result.diffImage || null,
+          error: result.error || null
         };
@@ -216,0 +217,0 @@ }).catch(function (error) {

dist/esm/utils/diff-images.js

@@ -12,3 +12,9 @@ import _slicedToArray from "@babel/runtime/helpers/esm/slicedToArray";
 
-    var result = diffPNGs(image1, image2, options);
+    return diffPNGs(image1, image2, options);
+  }).catch(function (error) {
+    return {
+      success: false,
+      error: error.message
+    };
+  }).then(function (result) {
     return Object.assign(result, {
@@ -15,0 +21,0 @@ source1: source1,

package.json

@@ -5,3 +5,3 @@ {
   "license": "MIT",
-  "version": "3.0.0-alpha.3",
+  "version": "3.0.0-alpha.4",
   "keywords": [
@@ -8,0 +8,0 @@ "integration test",

src/browser-automation/browser-test-driver.js

@@ -189,3 +189,3 @@ // Copyright (c) 2015 - 2017 Uber Technologies, Inc.
       .then(result => {
-        if (!result.success && opts.saveOnFail) {
+        if (!result.success && opts.saveOnFail && result.source1) {
           let filename = opts.saveAs || '[name]-failed.png';
@@ -197,6 +197,7 @@ filename = filename.replace('[name]', opts.goldenImage.replace(/\.\w+$/, ''));
           headless: this.headless,
-          match: result.match,
-          matchPercentage: result.matchPercentage,
+          match: result.match || 0,
+          matchPercentage: result.matchPercentage || 'N/A',
           success: result.success,
-          diffImage: result.diffImage
+          diffImage: result.diffImage || null,
+          error: result.error || null
         };
@@ -203,0 +204,0 @@ })

src/utils/diff-images.js

@@ -7,10 +7,15 @@ /* global Buffer */
 export default function diffImages(source1, source2, options = {}) {
-  return Promise.all([parsePNG(source1), parsePNG(source2)]).then(([image1, image2]) => {
-    const result = diffPNGs(image1, image2, options);
-    return Object.assign(result, {
-      source1,
-      source2,
-      options
-    });
-  });
+  return Promise.all([parsePNG(source1), parsePNG(source2)])
+    .then(([image1, image2]) => diffPNGs(image1, image2, options))
+    .catch(error => ({
+      success: false,
+      error: error.message
+    }))
+    .then(result =>
+      Object.assign(result, {
+        source1,
+        source2,
+        options
+      })
+    );
 }
@@ -17,0 +22,0 @@

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

207106

increased by0.97%

Lines of code

2249

increased by0.94%

No dependency changes