datagrok-tools 4.14.57 → 4.14.59

package/bin/commands/test-all.js

@@ -79,9 +79,10 @@ async function testAll(args) {
     (0, _testUtils.printBrowsersResult)(result, args.verbose);
   }
   if (args.csv) {
+    console.log(`Saving ${testsResults.length} results`);
     (0, _testUtils.saveCsvResults)(testsResults.map(result => result.csv), csvReportDir);
   }
-  return !testsResults.map(test => test.failed).some(failStatus => failStatus === true);
+  return testsResults.every(test => !test.failed);
 }
 async function filterTests(tests, tags, stressTest = false, benchmark = false) {
   let filteredTests = [];
@@ -121,9 +122,9 @@ async function runTests(browsersOrder, browserOptions) {
   }
   let resultObjects = await Promise.all(browsersPromises);
   for (let i = 0; i < resultObjects.length; i++) {
-    resultObjects[i].csv = await (0, _testUtils.addColumnToCsv)(resultObjects[i].csv, "browser", i);
-    resultObjects[i].csv = await (0, _testUtils.addColumnToCsv)(resultObjects[i].csv, "stress_test", browserOptions.stressTest ?? false);
-    resultObjects[i].csv = await (0, _testUtils.addColumnToCsv)(resultObjects[i].csv, "benchmark", browserOptions.benchmark ?? false);
+    resultObjects[i].csv = (0, _testUtils.addColumnToCsv)(resultObjects[i].csv, "browser", i);
+    resultObjects[i].csv = (0, _testUtils.addColumnToCsv)(resultObjects[i].csv, "stress_test", browserOptions.stressTest ?? false);
+    resultObjects[i].csv = (0, _testUtils.addColumnToCsv)(resultObjects[i].csv, "benchmark", browserOptions.benchmark ?? false);
   }
   return resultObjects;
 }

package/bin/commands/test.js

@@ -140,8 +140,8 @@ async function runTesting(args) {
           if (reproducedTest && !reproducedTest.failed) r = await updateResultsByReproduced(r, reproducedTest, test);
         }
       }
-      r.csv = await (0, _testUtils.addColumnToCsv)(r.csv, 'stress_test', args['stress-test'] ?? false);
-      r.csv = await (0, _testUtils.addColumnToCsv)(r.csv, 'benchmark', args.benchmark ?? false);
+      r.csv = (0, _testUtils.addColumnToCsv)(r.csv, 'stress_test', args['stress-test'] ?? false);
+      r.csv = (0, _testUtils.addColumnToCsv)(r.csv, 'benchmark', args.benchmark ?? false);
       testsResults.push(r);
       organized = testsLeft;
       browserId++;

package/bin/utils/test-utils.js

@@ -539,22 +539,18 @@ class TestContext {
   }
 }
 exports.TestContext = TestContext;
-async function addColumnToCsv(csv, columnName, defaultValue) {
-  let result = csv;
-  _papaparse.default.parse(csv, {
+function addColumnToCsv(csv, columnName, defaultValue) {
+  const results = _papaparse.default.parse(csv, {
     header: true,
-    skipEmptyLines: true,
-    complete: function (results) {
-      const dataWithDefaultColumn = results.data.map(row => {
-        row[columnName] = defaultValue;
-        return row;
-      });
-      result = _papaparse.default.unparse(dataWithDefaultColumn, {
-        header: true
-      });
-    }
+    skipEmptyLines: true
+  });
+  const dataWithDefaultColumn = results.data.map(row => ({
+    ...row,
+    [columnName]: defaultValue
+  }));
+  return _papaparse.default.unparse(dataWithDefaultColumn, {
+    header: true
   });
-  return result;
 }
 function escapeRegex(s) {
   return s.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "datagrok-tools",
-  "version": "4.14.57",
+  "version": "4.14.59",
   "description": "Utility to upload and publish packages to Datagrok",
   "homepage": "https://github.com/datagrok-ai/public/tree/master/tools#readme",
   "dependencies": {