@applitools/eyes-storybook 3.31.4 → 3.31.5

package/CHANGELOG.md

@@ -3,6 +3,13 @@
 ## Unreleased
 
 
+## 3.31.5 - 2023/3/28
+
+### Features
+### Bug fixes
+- Optimized number of requests during polling
+- Write result files without throwing error
+
 ## 3.31.4 - 2023/3/17
 
 ### Features

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applitools/eyes-storybook",
-  "version": "3.31.4",
+  "version": "3.31.5",
   "description": "",
   "keywords": [
     "applitools",
@@ -73,14 +73,14 @@
     }
   },
   "dependencies": {
-    "@applitools/core": "2.4.2",
+    "@applitools/core": "2.4.3",
     "@applitools/driver": "1.11.37",
     "@applitools/functional-commons": "1.6.0",
     "@applitools/logger": "1.1.48",
     "@applitools/monitoring-commons": "1.0.19",
     "@applitools/spec-driver-puppeteer": "1.1.49",
     "@applitools/test-server": "1.1.28",
-    "@applitools/ufg-client": "1.2.3",
+    "@applitools/ufg-client": "1.2.4",
     "@applitools/utils": "1.3.32",
     "boxen": "4.2.0",
     "chalk": "3.0.0",
@@ -94,7 +94,7 @@
     "yargs": "15.4.1"
   },
   "devDependencies": {
-    "@applitools/bongo": "^3.0.1",
+    "@applitools/bongo": "^3.0.3",
     "@applitools/sdk-shared": "0.9.15",
     "@applitools/snaptdout": "^1.0.1",
     "@storybook/addon-interactions": "^6.4.18",

package/src/cli.js

@@ -42,7 +42,7 @@ const {performance, timeItAsync} = makeTiming();
       process.exit(config.exitcode ? config.exitcode : 0);
     } else {
       const totalTime = performance['eyesStorybook'];
-      const {exitCode, formatter, outputStr} = processResults({
+      const {exitCode, summary, outputStr} = processResults({
         results,
         totalTime,
         testConcurrency: config.testConcurrency,
@@ -50,13 +50,13 @@ const {performance, timeItAsync} = makeTiming();
       });
       console.log(outputStr);
       if (config.jsonFilePath) {
-        handleJsonFile(config.jsonFilePath, formatter);
+        handleJsonFile(config.jsonFilePath, summary);
       }
       if (config.tapFilePath) {
-        handleTapFile(config.tapFilePath, formatter);
+        handleTapFile(config.tapFilePath, summary);
       }
       if (config.xmlFilePath) {
-        handleXmlFile(config.xmlFilePath, formatter, {totalTime});
+        handleXmlFile(config.xmlFilePath, summary, {totalTime});
       }
       process.exit(config.exitcode ? exitCode : 0);
     }

package/src/handleJsonFile.js

@@ -2,10 +2,11 @@
 
 const fs = require('fs');
 const {resolve} = require('path');
+const {formatters} = require('@applitools/core');
 
-function handleJsonFile(jsonFilePath, formatter) {
+function handleJsonFile(jsonFilePath, summary) {
   const path = resolve(jsonFilePath, 'eyes.json');
-  fs.writeFileSync(path, formatter.toJsonOutput());
+  fs.writeFileSync(path, formatters.toJsonOutput(summary));
   return path;
 }
 

package/src/handleTapFile.js

@@ -2,10 +2,17 @@
 
 const fs = require('fs');
 const {resolve} = require('path');
+const {formatters} = require('@applitools/core');
 
-function handleTapFile(tapFilePath, formatter) {
+function handleTapFile(tapFilePath, summary) {
   const path = resolve(tapFilePath, 'eyes.tap');
-  fs.writeFileSync(path, formatter.asHierarchicTAPString(false, true));
+  fs.writeFileSync(
+    path,
+    formatters.toHierarchicTAPString(summary.results, {
+      includeSubTests: false,
+      markNewAsPassed: true,
+    }),
+  );
   return path;
 }
 

package/src/handleXmlFile.js

@@ -2,10 +2,11 @@
 
 const fs = require('fs');
 const {resolve} = require('path');
+const {formatters} = require('@applitools/core');
 
-function handleXmlFile(xmlFilePath, formatter, {suiteName = 'Eyes Storybook', totalTime} = {}) {
+function handleXmlFile(xmlFilePath, summary, {suiteName = 'Eyes Storybook', totalTime} = {}) {
   const path = resolve(xmlFilePath, 'eyes.xml');
-  fs.writeFileSync(path, formatter.toXmlOutput({suiteName, totalTime}));
+  fs.writeFileSync(path, formatters.toXmlOutput(summary.results, {suiteName, totalTime}));
   return path;
 }
 

package/src/processResults.js

@@ -4,9 +4,8 @@ const chalk = require('chalk');
 const utils = require('@applitools/utils');
 const uniq = require('./uniq');
 const concurrencyMsg = require('./concurrencyMsg');
-const {formatters} = require('@applitools/core');
 
-function processResults({results = [], totalTime, testConcurrency, saveNewTests = true}) {
+function processResults({results, totalTime, testConcurrency, saveNewTests = true}) {
   let outputStr = '\n';
   const pluralize = utils.general.pluralize;
   let testResults = flatten(results.summary.results);
@@ -102,12 +101,11 @@ function processResults({results = [], totalTime, testConcurrency, saveNewTests
     // TODO require from core
     outputStr += `\n${concurrencyMsg}\n`;
   }
-  const formatter = formatters.toJsonOutput(results.summary);
   const exitCode =
     !warnForUnsavedNewTests && passedOrNew.length && !errors.length && !unresolved.length ? 0 : 1;
   return {
     outputStr,
-    formatter,
+    summary: results.summary,
     exitCode,
   };
 }