datagrok-tools 4.13.44 → 4.13.46

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Datagrok-tools changelog
 
+## 4.13.46 (2024-12-23)
+
+### Features
+
+* Improved error handling:
+  - Different exit codes for package errors / grok script errors
+  - Graceful error handling when testing non-existing packages
+
+## 4.13.45 (2024-12-12)
+
+### Features
+
+* Test all workers count variable fixes
 
 ## 4.13.44 (2024-12-06)
 

package/bin/commands/help.js

@@ -13,7 +13,7 @@ var HELP_CREATE = "\nUsage: grok create [name]\n\nCreate a package:\n\ngrok crea
 var HELP_PUBLISH = "\nUsage: grok publish [host]\n\nUpload a package\n\nOptions:\n[--build|--rebuild] [--debug|--release] [-k | --key] [--suffix] [--all] [--refresh] [--link]\n\n--all             Publish all available packages \n--refresh         Publish all available already loaded packages \n--link  \t        Link the package to local utils\n\nRunning `grok publish` is the same as running `grok publish defaultHost --build --debug`\n";
 var HELP_CHECK = "\nUsage: grok check\n\nOptions:\n[-r | --recursive]\n\n--recursive       Check all packages in the current directory\n\nCheck package content (function signatures, import statements of external modules, etc.)\n";
 var HELP_TEST = "\nUsage: grok test\n\nOptions:\n[--package] [--category] [--test] [--host] [--csv] [--gui] [--skip-build] [--skip-publish] [--link] [--catchUnhandled] [--report] [--record] [--verbose] [--platform] [--benchmark] [--stress-test]\n\n--package           Specify a package name to run tests for\n--category          Specify a category name to run tests for\n--test              Specify a test name to run \n--host              Host alias as in the config file\n--csv               Save the test report in a CSV file\n--gui               Launch graphical interface (non-headless mode)\n--catchUnhandled    Catch unhandled exceptions during test execution (default=true)\n--report            Report failed tests to audit, notifies package author (default=false)\n--skip-build        Skip the package build step\n--skip-publish      Skip the package publication step\n--link  \t          Link the package to local utils\n--record            Records the test execution process in mp4 format\n--verbose           Prints detailed information about passed and skipped tests in the console\n--platform          Runs only platform tests (applicable for ApiTests package only)\n--core              Runs package & auto tests & core tests (core tests run only from DevTools package)\n--benchmark   \t    Runs tests in benchmark mode\n--stress-test       Runs shuffled stress-test only\n \nRun package tests\n\nSee instructions:\nhttps://datagrok.ai/help/develop/how-to/test-packages#local-testing\n";
-var HELP_TESTALL = "\nUsage: grok testall\n\nOptions:\n[--packages] [--host] [--csv] [--gui] [--skip-build] [--skip-publish] [--link-package] [--catchUnhandled] [--report] [--record] [--verbose] [--benchmark] [--stress-test] [--order] [--tags] [--testRepeat] [--workersCount]\n\n--packages          Specify a packages names to run tests for\n--host              Host alias as in the config file\n--csv               Save the test report in a CSV file\n--gui               Launch graphical interface (non-headless mode)\n--catchUnhandled    Catch unhandled exceptions during test execution (default=true)\n--report            Report failed tests to audit, notifies packages author (default=false)\n--skip-build        Skip the packages build step\n--skip-publish      Skip the packages publication step\n--link-package  \t  Link the packages to local utils\n--record            Records the test execution process in mp4 format\n--verbose           Prints detailed information about passed and skipped tests in the console\n--core              Runs packages & core tests (applicable  for DevTools packages only)\n--benchmark   \t    Runs tests in benchmark mode\n--stress-test       Runs shuffled stress-test only\n--order             Specify order for tests invocation\n--tags              Filter tests by tag name for run\n--testRepeat        Set amount of tests repeats\n--workersCount      Set amount of workers for tests run\n\nRun tests of all or specified packages \n\nSee instructions:\nhttps://datagrok.ai/help/develop/how-to/test-packages#local-testing\n";
+var HELP_TESTALL = "\nUsage: grok testall\n\nOptions:\n[--packages] [--host] [--csv] [--gui] [--skip-build] [--skip-publish] [--link-package] [--catchUnhandled] [--report] [--record] [--verbose] [--benchmark] [--stress-test] [--order] [--tags] [--testRepeat] [--workers-count]\n\n--packages          Specify a packages names to run tests for\n--host              Host alias as in the config file\n--csv               Save the test report in a CSV file\n--gui               Launch graphical interface (non-headless mode)\n--catchUnhandled    Catch unhandled exceptions during test execution (default=true)\n--report            Report failed tests to audit, notifies packages author (default=false)\n--skip-build        Skip the packages build step\n--skip-publish      Skip the packages publication step\n--link-package  \t  Link the packages to local utils\n--record            Records the test execution process in mp4 format\n--verbose           Prints detailed information about passed and skipped tests in the console\n--core              Runs packages & core tests (applicable  for DevTools packages only)\n--benchmark   \t    Runs tests in benchmark mode\n--stress-test       Runs shuffled stress-test only\n--order             Specify order for tests invocation\n--tags              Filter tests by tag name for run\n--testRepeat        Set amount of tests repeats\n--workers-count      Set amount of workers for tests run\n\nRun tests of all or specified packages \n\nSee instructions:\nhttps://datagrok.ai/help/develop/how-to/test-packages#local-testing\n";
 var HELP_LINK = "\nUsage: grok link\n\nLink `datagrok-api` and libraries for local development\n";
 
 // const HELP_MIGRATE = `

package/bin/commands/test-all.js

@@ -83,7 +83,7 @@ function _testAll() {
         case 10:
           filteredTests = _context.sent;
           _context.next = 13;
-          return setWorkersOrder(filteredTests, getEnumOrder((_args$order = args.order) !== null && _args$order !== void 0 ? _args$order : ''), args.workersCount, args.testRepeat);
+          return setWorkersOrder(filteredTests, getEnumOrder((_args$order = args.order) !== null && _args$order !== void 0 ? _args$order : ''), args['workers-count'], args.testRepeat);
         case 13:
           workersOrder = _context.sent;
           _context.next = 16;
@@ -296,27 +296,16 @@ function _runTests() {
             _iterator3.f();
           }
           workersPromises = [];
-          _i = 0, _workersCommands = workersCommands;
-        case 5:
-          if (!(_i < _workersCommands.length)) {
-            _context5.next = 13;
-            break;
+          for (_i = 0, _workersCommands = workersCommands; _i < _workersCommands.length; _i++) {
+            workerCommands = _workersCommands[_i];
+            workersPromises.push((0, _testUtils.runWorker)(workerCommands, workerOptions, workersStarted++, testInvocationTimeout));
           }
-          workerCommands = _workersCommands[_i];
-          workersPromises.push((0, _testUtils.runWorker)(workerCommands, workerOptions, workersStarted++, testInvocationTimeout));
-          _context5.next = 10;
-          return workersPromises[workersPromises.length];
-        case 10:
-          _i++;
-          _context5.next = 5;
-          break;
-        case 13:
-          _context5.next = 15;
+          _context5.next = 7;
           return Promise.all(workersPromises);
-        case 15:
+        case 7:
           resultObjects = _context5.sent;
           return _context5.abrupt("return", resultObjects);
-        case 17:
+        case 9:
         case "end":
           return _context5.stop();
       }

package/bin/commands/test.js

@@ -102,6 +102,21 @@ function _runTesting() {
           testsObj = _context3.sent;
           console.log(testsObj);
           parsed = (0, _orderFunctions.setAlphabeticalOrder)(testsObj, 1, 1);
+          if (!(parsed.length == 0)) {
+            _context3.next = 8;
+            break;
+          }
+          return _context3.abrupt("return", {
+            failed: true,
+            verbosePassed: 'Package not found',
+            verboseSkipped: 'Package not found',
+            verboseFailed: 'Package not found',
+            passedAmount: 0,
+            skippedAmount: 0,
+            failedAmount: 0,
+            csv: ''
+          });
+        case 8:
           organized = parsed[0].map(function (testObj) {
             return {
               "package": testObj.packageName,
@@ -135,7 +150,7 @@ function _runTesting() {
           color.info('Starting tests...');
           testsResults = [];
           workerId = 1;
-          _context3.next = 14;
+          _context3.next = 16;
           return (0, _testUtils.timeout)( /*#__PURE__*/(0, _asyncToGenerator2["default"])( /*#__PURE__*/_regenerator["default"].mark(function _callee2() {
             var _args$benchmark, _args$catchUnhandled, _args$gui, _args$record, _args$report, _args$verbose, _args$ciCd, testsLeft, testsToReproduce, _iterator2, _step2, testData, reproduced, _iterator3, _step3, _test2, _reproducedTest2;
             return _regenerator["default"].wrap(function _callee2$(_context2) {
@@ -225,12 +240,12 @@ function _runTesting() {
               }
             }, _callee2, null, [[12, 25, 28, 31]]);
           })), testInvocationTimeout);
-        case 14:
-          _context3.next = 16;
-          return (0, _testUtils.mergeWorkersResults)(testsResults);
         case 16:
+          _context3.next = 18;
+          return (0, _testUtils.mergeWorkersResults)(testsResults);
+        case 18:
           return _context3.abrupt("return", _context3.sent);
-        case 17:
+        case 19:
         case "end":
           return _context3.stop();
       }

package/bin/grok.js

@@ -31,7 +31,7 @@ if (command in commands) {
   } catch (err) { 
     console.error(err);
     console.log(help[command]);
-    exitWithCode(1);
+    exitWithCode(255);
   }
 } else 
   console.log(help.help);

package/package.json

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