newman-reporter-qase 2.3.2 → 2.3.3

package/changelog.md

@@ -1,3 +1,9 @@
+# newman-reporter-qase@2.3.3
+
+## Fixed
+
+- Qase test case IDs that are `<= 0` (e.g. `// qase: 0` in a Postman test script) are now dropped with a warning instead of being sent to the API, which previously rejected the whole batch with HTTP 400. Bumped `qase-javascript-commons` pin to `~2.7.4`.
+
 # newman-reporter-qase@2.3.2
 
 ## Fixed

package/dist/modules/metadataExtractor.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.MetadataExtractor = void 0;
+const internal_1 = require("qase-javascript-commons/internal");
 // eslint-disable-next-line @typescript-eslint/no-extraneous-class
 class MetadataExtractor {
     static qaseIdRegExp = /\/\/\s*?[qQ]ase:\s?((?:[\d]+[\s,]{0,})+)/;
@@ -19,7 +20,7 @@ class MetadataExtractor {
                 });
             }
         });
-        return ids;
+        return (0, internal_1.filterPositiveIds)(ids);
     }
     static getProjectMapping(eventList) {
         const projectMapping = {};
@@ -31,7 +32,11 @@ class MetadataExtractor {
                     while ((m = re.exec(line)) !== null) {
                         const projectCode = m[1]?.trim();
                         const idsStr = (m[2] ?? '').replace(/\s/g, '');
-                        const ids = idsStr.split(',').map((s) => parseInt(s, 10)).filter((n) => !Number.isNaN(n));
+                        const rawIds = idsStr
+                            .split(',')
+                            .map((s) => parseInt(s, 10))
+                            .filter((n) => !Number.isNaN(n));
+                        const ids = (0, internal_1.filterPositiveIds)(rawIds);
                         if (projectCode && projectCode.toUpperCase() !== 'ID' && ids.length > 0) {
                             const existing = projectMapping[projectCode] ?? [];
                             projectMapping[projectCode] = [...existing, ...ids];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "newman-reporter-qase",
-  "version": "2.3.2",
+  "version": "2.3.3",
   "description": "Qase TMS Newman Reporter",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
@@ -40,7 +40,7 @@
   "author": "Qase Team <support@qase.io>",
   "license": "Apache-2.0",
   "dependencies": {
-    "qase-javascript-commons": "~2.7.3",
+    "qase-javascript-commons": "~2.7.4",
     "semver": "^7.7.3"
   },
   "devDependencies": {