@percy/report 0.0.5 → 0.0.7

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@percy/report",
   "description": "Package to generate a build report and project summary report for Percy, BrowserStack's visual testing platform",
-  "version": "0.0.5",
+  "version": "0.0.7",
   "main": "src/index.js",
   "license": "MIT",
   "author": "BrowserStack Pvt Ltd",
@@ -44,11 +44,12 @@
     "commander": "^9.4.0",
     "date-fns": "^2.29.3",
     "dotenv": "^16.0.3",
-    "ejs": "^3.1.8"
+    "ejs": "^3.1.8",
+    "p-queue": "^8.0.1"
   },
   "devDependencies": {
     "@percy/selenium-webdriver": "^1.0.2",
     "chai": "^4.3.7",
     "mocha": "^10.2.0"
   }
-}
+}

package/src/generate.js

@@ -9,6 +9,7 @@ const defaultConfig = {
     downloadPath: './Reports',
     diffThreshold: 1
 }
+let downloadQueue;
 module.exports.Generate = async function (config) {
     let { buildId, percyToken, apiUrl, downloadImages, downloadPath, diffThreshold } = Object.assign({}, defaultConfig, config)
     let axios = new Axios({
@@ -28,6 +29,11 @@ module.exports.Generate = async function (config) {
             throw res.data
         }
     })
+    if(!downloadQueue){
+        await import('p-queue').then(({default:PQueue})=>{
+            downloadQueue =  new PQueue({concurrency:1,intervalCap:5,interval:5000})
+        })
+    }
     const isApp = buildDetails['data']['attributes']['type'] == 'app'
     while (buildDetails.data && buildDetails.data.attributes.state !== 'finished') {
         console.log("Waiting for build to complete on Percy...")
@@ -101,10 +107,12 @@ module.exports.Generate = async function (config) {
                 }
             }
             if (downloadImages) {
+                
+                
                 ['base', 'head', 'diff'].forEach((val) => {
                     if (images[val]) {
                         images[val].file = downloadImage({
-                            name: String(snapshot?.['attributes'].name).replace('/', '-'),
+                            name: String(snapshot?.['attributes'].name),
                             compTag,
                             width: images[val].width,
                             type: val,
@@ -127,7 +135,7 @@ module.exports.Generate = async function (config) {
             
             comparison['diff-percentage'] = (comparison['diff-ratio'] * 100).toFixed(2)
             comparison['diff-color'] = "yellow"
-            if (comparison['diff-percentage'] > diffThreshold) {
+            if (Number(comparison['diff-percentage']) > Number(diffThreshold)) {
                 comparison['diff-color'] = "red"
             }
             return comparison
@@ -167,18 +175,23 @@ function downloadImage(options) {
     if (!fs.existsSync(`${baseDir}/${type}`)) {
         fs.mkdirSync(`${baseDir}/${type}`, { recursive: true })
     }
-    let path = `${baseDir}/${type}/${name}-${compTag}-${width}.png`
+    let composedName = `${name}-${compTag}-${width}.png`.replace(/[\\\/]/g,'_')
+    let path = `${baseDir}/${type}/${composedName}`
     try {
-        new Axios({ responseType: 'arraybuffer', url: url }).get(url).then((file) => {
+        downloadQueue.add(()=>new Axios({ responseType: 'arraybuffer', url: url }).get(url).then((file) => {
             console.log("Download Complete:" + path)
             fs.writeFileSync(path, file.data)
         }).catch((err) => {
             console.error("Failed to Download: " + path)
-            console.error(err)
-        })
-        return `file:./${type}/${name}-${compTag}-${width}.png`;
-    } catch {
-
+            if(err.code == 'ECONNRESET'){
+                console.log('Retrying '+ path)
+            }else{
+                console.error(`Failed ${path}, error: ${err.code}`)
+            }
+        }))
+        return `file:./${type}/${composedName}`;
+    } catch(err) {
+        
     }
 }