@applitools/dom-snapshot - npm Package Compare versions

Comparing version 1.2.7 to 1.2.8

dist/processPage.js

@@ -1,2 +0,2 @@
-// @applitools/dom-snapshot@1.2.7
+// @applitools/dom-snapshot@1.2.8
 function __processPage() {
@@ -278,3 +278,3 @@ var processPage = (function () {
             } else if (/svg/.test(type)) {
-              resourceUrls = extractResourcesFromSvg({svgArrayBuffer: value});
+              resourceUrls = extractResourcesFromSvg(value);
             }
@@ -311,25 +311,27 @@
 
-  function extractResourcesFromSvg({svgArrayBuffer, parser, decoder}) {
-    let svgStr;
-    let urls = [];
-    try {
-      const decooder = decoder || new TextDecoder('utf-8');
-      svgStr = decooder.decode(svgArrayBuffer);
-      const domparser = parser || new DOMParser();
-      const doc = domparser.parseFromString(svgStr, 'image/svg+xml');
+  function makeExtractResourcesFromSvg({parser, decoder}) {
+    return function(svgArrayBuffer) {
+      let svgStr;
+      let urls = [];
+      try {
+        const decooder = decoder || new TextDecoder('utf-8');
+        svgStr = decooder.decode(svgArrayBuffer);
+        const domparser = parser || new DOMParser();
+        const doc = domparser.parseFromString(svgStr, 'image/svg+xml');
 
-      const fromImages = Array.from(doc.getElementsByTagName('image'))
-        .concat(Array.from(doc.getElementsByTagName('use')))
-        .map(e => e.getAttribute('href') || e.getAttribute('xlink:href'));
-      const fromObjects = Array.from(doc.getElementsByTagName('object')).map(e =>
-        e.getAttribute('data'),
-      );
-      urls = [...fromImages, ...fromObjects].filter(u => u[0] !== '#');
-    } catch (e) {
-      console.log('could not parse svg content', e);
-    }
-    return urls;
+        const fromImages = Array.from(doc.getElementsByTagName('image'))
+          .concat(Array.from(doc.getElementsByTagName('use')))
+          .map(e => e.getAttribute('href') || e.getAttribute('xlink:href'));
+        const fromObjects = Array.from(doc.getElementsByTagName('object')).map(e =>
+          e.getAttribute('data'),
+        );
+        urls = [...fromImages, ...fromObjects].filter(u => u[0] !== '#');
+      } catch (e) {
+        console.log('could not parse svg content', e);
+      }
+      return urls;
+    };
   }
 
-  var extractResourcesFromSvg_1 = extractResourcesFromSvg;
+  var makeExtractResourcesFromSvg_1 = makeExtractResourcesFromSvg;
 
@@ -439,2 +441,3 @@ /* global window */
     const extractResourcesFromStyleSheet$$1 = extractResourcesFromStyleSheet({styleSheetCache});
+    const extractResourcesFromSvg = makeExtractResourcesFromSvg_1({});
     const findStyleSheetByUrl$$1 = findStyleSheetByUrl({styleSheetCache});
@@ -445,3 +448,3 @@ const processResource$$1 = processResource({
       extractResourcesFromStyleSheet: extractResourcesFromStyleSheet$$1,
-      extractResourcesFromSvg: extractResourcesFromSvg_1,
+      extractResourcesFromSvg,
       absolutizeUrl: absolutizeUrl_1,
@@ -448,0 +451,0 @@ isSameOrigin: isSameOrigin_1,

dist/processPageAndPoll.js

@@ -1,2 +0,2 @@
-// @applitools/dom-snapshot@1.2.7
+// @applitools/dom-snapshot@1.2.8
 function __processPageAndPoll() {
@@ -330,3 +330,3 @@ var processPageAndPoll = (function () {
             } else if (/svg/.test(type)) {
-              resourceUrls = extractResourcesFromSvg({svgArrayBuffer: value});
+              resourceUrls = extractResourcesFromSvg(value);
             }
@@ -363,25 +363,27 @@
 
-  function extractResourcesFromSvg({svgArrayBuffer, parser, decoder}) {
-    let svgStr;
-    let urls = [];
-    try {
-      const decooder = decoder || new TextDecoder('utf-8');
-      svgStr = decooder.decode(svgArrayBuffer);
-      const domparser = parser || new DOMParser();
-      const doc = domparser.parseFromString(svgStr, 'image/svg+xml');
+  function makeExtractResourcesFromSvg({parser, decoder}) {
+    return function(svgArrayBuffer) {
+      let svgStr;
+      let urls = [];
+      try {
+        const decooder = decoder || new TextDecoder('utf-8');
+        svgStr = decooder.decode(svgArrayBuffer);
+        const domparser = parser || new DOMParser();
+        const doc = domparser.parseFromString(svgStr, 'image/svg+xml');
 
-      const fromImages = Array.from(doc.getElementsByTagName('image'))
-        .concat(Array.from(doc.getElementsByTagName('use')))
-        .map(e => e.getAttribute('href') || e.getAttribute('xlink:href'));
-      const fromObjects = Array.from(doc.getElementsByTagName('object')).map(e =>
-        e.getAttribute('data'),
-      );
-      urls = [...fromImages, ...fromObjects].filter(u => u[0] !== '#');
-    } catch (e) {
-      console.log('could not parse svg content', e);
-    }
-    return urls;
+        const fromImages = Array.from(doc.getElementsByTagName('image'))
+          .concat(Array.from(doc.getElementsByTagName('use')))
+          .map(e => e.getAttribute('href') || e.getAttribute('xlink:href'));
+        const fromObjects = Array.from(doc.getElementsByTagName('object')).map(e =>
+          e.getAttribute('data'),
+        );
+        urls = [...fromImages, ...fromObjects].filter(u => u[0] !== '#');
+      } catch (e) {
+        console.log('could not parse svg content', e);
+      }
+      return urls;
+    };
   }
 
-  var extractResourcesFromSvg_1 = extractResourcesFromSvg;
+  var makeExtractResourcesFromSvg_1 = makeExtractResourcesFromSvg;
 
@@ -491,2 +493,3 @@ /* global window */
     const extractResourcesFromStyleSheet$$1 = extractResourcesFromStyleSheet({styleSheetCache});
+    const extractResourcesFromSvg = makeExtractResourcesFromSvg_1({});
     const findStyleSheetByUrl$$1 = findStyleSheetByUrl({styleSheetCache});
@@ -497,3 +500,3 @@ const processResource$$1 = processResource({
       extractResourcesFromStyleSheet: extractResourcesFromStyleSheet$$1,
-      extractResourcesFromSvg: extractResourcesFromSvg_1,
+      extractResourcesFromSvg,
       absolutizeUrl: absolutizeUrl_1,
@@ -500,0 +503,0 @@ isSameOrigin: isSameOrigin_1,

dist/processPageAndSerialize.js

@@ -1,2 +0,2 @@
-// @applitools/dom-snapshot@1.2.7
+// @applitools/dom-snapshot@1.2.8
 function __processPageAndSerialize() {
@@ -330,3 +330,3 @@ var processPageAndSerialize = (function () {
             } else if (/svg/.test(type)) {
-              resourceUrls = extractResourcesFromSvg({svgArrayBuffer: value});
+              resourceUrls = extractResourcesFromSvg(value);
             }
@@ -363,25 +363,27 @@
 
-  function extractResourcesFromSvg({svgArrayBuffer, parser, decoder}) {
-    let svgStr;
-    let urls = [];
-    try {
-      const decooder = decoder || new TextDecoder('utf-8');
-      svgStr = decooder.decode(svgArrayBuffer);
-      const domparser = parser || new DOMParser();
-      const doc = domparser.parseFromString(svgStr, 'image/svg+xml');
+  function makeExtractResourcesFromSvg({parser, decoder}) {
+    return function(svgArrayBuffer) {
+      let svgStr;
+      let urls = [];
+      try {
+        const decooder = decoder || new TextDecoder('utf-8');
+        svgStr = decooder.decode(svgArrayBuffer);
+        const domparser = parser || new DOMParser();
+        const doc = domparser.parseFromString(svgStr, 'image/svg+xml');
 
-      const fromImages = Array.from(doc.getElementsByTagName('image'))
-        .concat(Array.from(doc.getElementsByTagName('use')))
-        .map(e => e.getAttribute('href') || e.getAttribute('xlink:href'));
-      const fromObjects = Array.from(doc.getElementsByTagName('object')).map(e =>
-        e.getAttribute('data'),
-      );
-      urls = [...fromImages, ...fromObjects].filter(u => u[0] !== '#');
-    } catch (e) {
-      console.log('could not parse svg content', e);
-    }
-    return urls;
+        const fromImages = Array.from(doc.getElementsByTagName('image'))
+          .concat(Array.from(doc.getElementsByTagName('use')))
+          .map(e => e.getAttribute('href') || e.getAttribute('xlink:href'));
+        const fromObjects = Array.from(doc.getElementsByTagName('object')).map(e =>
+          e.getAttribute('data'),
+        );
+        urls = [...fromImages, ...fromObjects].filter(u => u[0] !== '#');
+      } catch (e) {
+        console.log('could not parse svg content', e);
+      }
+      return urls;
+    };
   }
 
-  var extractResourcesFromSvg_1 = extractResourcesFromSvg;
+  var makeExtractResourcesFromSvg_1 = makeExtractResourcesFromSvg;
 
@@ -491,2 +493,3 @@ /* global window */
     const extractResourcesFromStyleSheet$$1 = extractResourcesFromStyleSheet({styleSheetCache});
+    const extractResourcesFromSvg = makeExtractResourcesFromSvg_1({});
     const findStyleSheetByUrl$$1 = findStyleSheetByUrl({styleSheetCache});
@@ -497,3 +500,3 @@ const processResource$$1 = processResource({
       extractResourcesFromStyleSheet: extractResourcesFromStyleSheet$$1,
-      extractResourcesFromSvg: extractResourcesFromSvg_1,
+      extractResourcesFromSvg,
       absolutizeUrl: absolutizeUrl_1,
@@ -500,0 +503,0 @@ isSameOrigin: isSameOrigin_1,

dist/processPageCjs.js

@@ -275,3 +275,3 @@ 'use strict';
           } else if (/svg/.test(type)) {
-            resourceUrls = extractResourcesFromSvg({svgArrayBuffer: value});
+            resourceUrls = extractResourcesFromSvg(value);
           }
@@ -308,25 +308,27 @@
 
-function extractResourcesFromSvg({svgArrayBuffer, parser, decoder}) {
-  let svgStr;
-  let urls = [];
-  try {
-    const decooder = decoder || new TextDecoder('utf-8');
-    svgStr = decooder.decode(svgArrayBuffer);
-    const domparser = parser || new DOMParser();
-    const doc = domparser.parseFromString(svgStr, 'image/svg+xml');
+function makeExtractResourcesFromSvg({parser, decoder}) {
+  return function(svgArrayBuffer) {
+    let svgStr;
+    let urls = [];
+    try {
+      const decooder = decoder || new TextDecoder('utf-8');
+      svgStr = decooder.decode(svgArrayBuffer);
+      const domparser = parser || new DOMParser();
+      const doc = domparser.parseFromString(svgStr, 'image/svg+xml');
 
-    const fromImages = Array.from(doc.getElementsByTagName('image'))
-      .concat(Array.from(doc.getElementsByTagName('use')))
-      .map(e => e.getAttribute('href') || e.getAttribute('xlink:href'));
-    const fromObjects = Array.from(doc.getElementsByTagName('object')).map(e =>
-      e.getAttribute('data'),
-    );
-    urls = [...fromImages, ...fromObjects].filter(u => u[0] !== '#');
-  } catch (e) {
-    console.log('could not parse svg content', e);
-  }
-  return urls;
+      const fromImages = Array.from(doc.getElementsByTagName('image'))
+        .concat(Array.from(doc.getElementsByTagName('use')))
+        .map(e => e.getAttribute('href') || e.getAttribute('xlink:href'));
+      const fromObjects = Array.from(doc.getElementsByTagName('object')).map(e =>
+        e.getAttribute('data'),
+      );
+      urls = [...fromImages, ...fromObjects].filter(u => u[0] !== '#');
+    } catch (e) {
+      console.log('could not parse svg content', e);
+    }
+    return urls;
+  };
 }
 
-var extractResourcesFromSvg_1 = extractResourcesFromSvg;
+var makeExtractResourcesFromSvg_1 = makeExtractResourcesFromSvg;
 
@@ -436,2 +438,3 @@ /* global window */
   const extractResourcesFromStyleSheet$$1 = extractResourcesFromStyleSheet({styleSheetCache});
+  const extractResourcesFromSvg = makeExtractResourcesFromSvg_1({});
   const findStyleSheetByUrl$$1 = findStyleSheetByUrl({styleSheetCache});
@@ -442,3 +445,3 @@ const processResource$$1 = processResource({
     extractResourcesFromStyleSheet: extractResourcesFromStyleSheet$$1,
-    extractResourcesFromSvg: extractResourcesFromSvg_1,
+    extractResourcesFromSvg,
     absolutizeUrl: absolutizeUrl_1,
@@ -445,0 +448,0 @@ isSameOrigin: isSameOrigin_1,

index.js

 'use strict';
 
 const makeGetScript = require('./src/getScript');
+const makeExtractResourcesFromSvg = require('./src/browser/makeExtractResourcesFromSvg');
 const getProcessPageScript = makeGetScript('processPage');
@@ -12,2 +13,3 @@ const getProcessPageAndSerializeScript = makeGetScript('processPageAndSerialize');
   getProcessPageAndPollScript,
+  makeExtractResourcesFromSvg,
 };

package.json

 {
   "name": "@applitools/dom-snapshot",
-  "version": "1.2.7",
+  "version": "1.2.8",
   "main": "index.js",
@@ -5,0 +5,0 @@ "license": "MIT",

src/browser/processPage.js

@@ -8,3 +8,3 @@ 'use strict';
 const makeProcessResource = require('./processResource');
-const extractResourcesFromSvg = require('./extractResourcesFromSvg');
+const makeExtractResourcesFromSvg = require('./makeExtractResourcesFromSvg');
 const fetchUrl = require('./fetchUrl');
@@ -23,2 +23,3 @@ const makeFindStyleSheetByUrl = require('./findStyleSheetByUrl');
   const extractResourcesFromStyleSheet = makeExtractResourcesFromStyleSheet({styleSheetCache});
+  const extractResourcesFromSvg = makeExtractResourcesFromSvg({});
   const findStyleSheetByUrl = makeFindStyleSheetByUrl({styleSheetCache});
@@ -25,0 +26,0 @@ const processResource = makeProcessResource({

src/browser/processResource.js

@@ -38,3 +38,3 @@ 'use strict';
           } else if (/svg/.test(type)) {
-            resourceUrls = extractResourcesFromSvg({svgArrayBuffer: value});
+            resourceUrls = extractResourcesFromSvg(value);
           }
@@ -41,0 +41,0 @@

No alert changes

Improved metrics

Total package byte prevSize

99550

increased by0.66%

Lines of code

2552

increased by0.67%

Number of maintainers

55

increased by1.85%

Worsened metrics

Published versions last year

18

decreased by-5.26%

No dependency changes