qase-javascript-commons 2.7.3 → 2.7.4

package/changelog.md

@@ -1,3 +1,9 @@
+## 2.7.4
+
+### Fixed
+
+- Qase test case IDs that are `<= 0` (e.g. an accidental `(Qase ID: 0)` in a test title or `@qaseid(0)` tag) no longer cause the API to reject the whole result batch with HTTP 400. The invalid ID is dropped with a warning and the result is sent without an ID, exactly as if no ID had been specified. Filtering happens in a new internal `filterPositiveIds()` helper applied by `parseQaseIdsFromString`, `parseProjectMappingFromTitle`, and `parseProjectMappingFromTags`. The warning routes through the provided `LoggerInterface` (logged at INFO level with a `Warning:` prefix in the message) or falls back to `console.warn` when no logger is available.
+
 ## 2.7.3
 
 ### Fixed

package/dist/internal/filter-positive-ids.d.ts

@@ -0,0 +1,17 @@
+import { LoggerInterface } from '../utils/logger';
+/**
+ * Filter a list of Qase test case IDs, dropping non-positive (<= 0) and
+ * non-finite numbers. Each dropped non-positive ID produces a warning so
+ * users can spot accidental "0" IDs in their test code.
+ *
+ * NaN and non-finite values are dropped silently — they come from upstream
+ * parse failures (parseInt('abc') etc.) that are already handled at their
+ * source and are not user-facing mistakes worth warning about.
+ *
+ * Warning routing:
+ *   - if a logger is provided, it goes through logger.log (INFO level, since
+ *     the LoggerInterface has no warn level);
+ *   - otherwise, it falls back to console.warn with a "[qase] " prefix so the
+ *     message still reaches the user.
+ */
+export declare function filterPositiveIds(ids: number[], logger?: LoggerInterface): number[];

package/dist/internal/filter-positive-ids.js

@@ -0,0 +1,38 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.filterPositiveIds = filterPositiveIds;
+/**
+ * Filter a list of Qase test case IDs, dropping non-positive (<= 0) and
+ * non-finite numbers. Each dropped non-positive ID produces a warning so
+ * users can spot accidental "0" IDs in their test code.
+ *
+ * NaN and non-finite values are dropped silently — they come from upstream
+ * parse failures (parseInt('abc') etc.) that are already handled at their
+ * source and are not user-facing mistakes worth warning about.
+ *
+ * Warning routing:
+ *   - if a logger is provided, it goes through logger.log (INFO level, since
+ *     the LoggerInterface has no warn level);
+ *   - otherwise, it falls back to console.warn with a "[qase] " prefix so the
+ *     message still reaches the user.
+ */
+function filterPositiveIds(ids, logger) {
+    const valid = [];
+    for (const n of ids) {
+        if (!Number.isFinite(n)) {
+            continue;
+        }
+        if (n <= 0) {
+            const message = `Warning: Qase test case ID must be greater than 0, got "${n}". This ID will be ignored and the result will be sent without it.`;
+            if (logger) {
+                logger.log(message);
+            }
+            else {
+                console.warn(`[qase] ${message}`);
+            }
+            continue;
+        }
+        valid.push(n);
+    }
+    return valid;
+}

package/dist/internal/ids-parser.d.ts

@@ -1,7 +1,9 @@
+import { LoggerInterface } from '../utils/logger';
 /**
  * Parses a comma-separated string of Qase IDs into an array of numbers.
  * Whitespace around each entry is trimmed; non-numeric entries are skipped.
+ * IDs that are not positive integers (<= 0) are filtered out with a warning.
  *
  * Examples: "1,2,3" → [1, 2, 3], "42" → [42], "" → [].
  */
-export declare function parseQaseIdsFromString(value: string): number[];
+export declare function parseQaseIdsFromString(value: string, logger?: LoggerInterface): number[];

package/dist/internal/ids-parser.js

@@ -1,18 +1,21 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.parseQaseIdsFromString = parseQaseIdsFromString;
+const filter_positive_ids_1 = require("./filter-positive-ids");
 /**
  * Parses a comma-separated string of Qase IDs into an array of numbers.
  * Whitespace around each entry is trimmed; non-numeric entries are skipped.
+ * IDs that are not positive integers (<= 0) are filtered out with a warning.
  *
  * Examples: "1,2,3" → [1, 2, 3], "42" → [42], "" → [].
  */
-function parseQaseIdsFromString(value) {
+function parseQaseIdsFromString(value, logger) {
     if (!value?.trim()) {
         return [];
     }
-    return value
+    const parsed = value
         .split(',')
         .map((part) => parseInt(part.trim(), 10))
         .filter((n) => !Number.isNaN(n));
+    return (0, filter_positive_ids_1.filterPositiveIds)(parsed, logger);
 }

package/dist/internal/index.d.ts

@@ -4,4 +4,5 @@ export type { ExtractedStep } from './step-parser';
 export { getFile } from './suite-file';
 export type { FileSuiteNode } from './suite-file';
 export { parseQaseIdsFromString } from './ids-parser';
+export { filterPositiveIds } from './filter-positive-ids';
 export { normalizeSuitePart } from './suite-normalizer';

package/dist/internal/index.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.normalizeSuitePart = exports.parseQaseIdsFromString = exports.getFile = exports.extractAndCleanStep = exports.removeQaseIdsFromTitle = void 0;
+exports.normalizeSuitePart = exports.filterPositiveIds = exports.parseQaseIdsFromString = exports.getFile = exports.extractAndCleanStep = exports.removeQaseIdsFromTitle = void 0;
 var title_1 = require("./title");
 Object.defineProperty(exports, "removeQaseIdsFromTitle", { enumerable: true, get: function () { return title_1.removeQaseIdsFromTitle; } });
 var step_parser_1 = require("./step-parser");
@@ -9,5 +9,7 @@ var suite_file_1 = require("./suite-file");
 Object.defineProperty(exports, "getFile", { enumerable: true, get: function () { return suite_file_1.getFile; } });
 var ids_parser_1 = require("./ids-parser");
 Object.defineProperty(exports, "parseQaseIdsFromString", { enumerable: true, get: function () { return ids_parser_1.parseQaseIdsFromString; } });
+var filter_positive_ids_1 = require("./filter-positive-ids");
+Object.defineProperty(exports, "filterPositiveIds", { enumerable: true, get: function () { return filter_positive_ids_1.filterPositiveIds; } });
 var suite_normalizer_1 = require("./suite-normalizer");
 Object.defineProperty(exports, "normalizeSuitePart", { enumerable: true, get: function () { return suite_normalizer_1.normalizeSuitePart; } });

package/dist/utils/project-mapping-utils.d.ts

@@ -1,4 +1,5 @@
 import type { TestopsProjectMapping } from '../models';
+import { LoggerInterface } from './logger';
 /**
  * Result of parsing project/ID markers from a test title.
  * - legacyIds: from "(Qase ID: 123)" or "(Qase ID: 123,124)" — single-project mode.
@@ -25,7 +26,7 @@ export interface ParsedTagsProjectMapping {
  * @param tags — e.g. ["@qaseid(1,2)", "@qaseid.PROJ2(3)"]
  * @returns legacyIds from @qaseid(...), projectMapping from @qaseid.PROJ(...)
  */
-export declare function parseProjectMappingFromTags(tags: string[]): ParsedTagsProjectMapping;
+export declare function parseProjectMappingFromTags(tags: string[], logger?: LoggerInterface): ParsedTagsProjectMapping;
 /**
  * Parse multi-project and legacy Qase ID markers from a test title.
  * - "(Qase ID: 123)" or "(Qase ID: 123,124)" → legacyIds, single-project.
@@ -34,7 +35,7 @@ export declare function parseProjectMappingFromTags(tags: string[]): ParsedTagsP
  * @param title — test title that may contain "(Qase ID: …)" or "(Qase PROJECT_CODE: …)".
  * @returns legacyIds, projectMapping, and cleanedTitle with all markers stripped.
  */
-export declare function parseProjectMappingFromTitle(title: string): ParsedProjectMapping;
+export declare function parseProjectMappingFromTitle(title: string, logger?: LoggerInterface): ParsedProjectMapping;
 /**
  * Build a test title with multi-project markers for use in test names.
  * Use this (or framework-specific qase.projects()) so the reporter can parse project and IDs.

package/dist/utils/project-mapping-utils.js

@@ -3,15 +3,16 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.parseProjectMappingFromTags = parseProjectMappingFromTags;
 exports.parseProjectMappingFromTitle = parseProjectMappingFromTitle;
 exports.formatTitleWithProjectMapping = formatTitleWithProjectMapping;
-/** Matches @qaseid(123) or @qaseid.PROJ1(123,124). */
-const QASEID_TAG_REGEXP = /@qaseid(?:\.([A-Za-z0-9_]+))?\(([\d,]+)\)/gi;
+const filter_positive_ids_1 = require("../internal/filter-positive-ids");
+/** Matches @qaseid(123) or @qaseid.PROJ1(123,124). Allows negative numbers. */
+const QASEID_TAG_REGEXP = /@qaseid(?:\.([A-Za-z0-9_]+))?\(([-\d,]+)\)/gi;
 /**
  * Parse @qaseid and @qaseid.PROJECT tags into legacy IDs and project mapping.
  *
  * @param tags — e.g. ["@qaseid(1,2)", "@qaseid.PROJ2(3)"]
  * @returns legacyIds from @qaseid(...), projectMapping from @qaseid.PROJ(...)
  */
-function parseProjectMappingFromTags(tags) {
+function parseProjectMappingFromTags(tags, logger) {
     const legacyIds = [];
     const projectMapping = {};
     for (const tag of tags) {
@@ -20,7 +21,11 @@ function parseProjectMappingFromTags(tags) {
         while ((m = re.exec(tag)) !== null) {
             const projectCode = m[1]; // undefined for @qaseid(1)
             const idsStr = m[2] ?? '';
-            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, filter_positive_ids_1.filterPositiveIds)(rawIds, logger);
             if (!projectCode || projectCode.toUpperCase() === 'ID') {
                 legacyIds.push(...ids);
             }
@@ -32,8 +37,8 @@ function parseProjectMappingFromTags(tags) {
     }
     return { legacyIds, projectMapping };
 }
-/** Matches "(Qase ID: 123)" or "(Qase PROJ1: 123,124)" — optional space after "Qase". */
-const QASE_MARKER_REGEXP = /\(Qase\s+([A-Za-z0-9_]+):\s*([\d,]+)\)/gi;
+/** Matches "(Qase ID: 123)" or "(Qase PROJ1: 123,124)" — optional space after "Qase". Allows negative numbers. */
+const QASE_MARKER_REGEXP = /\(Qase\s+([A-Za-z0-9_]+):\s*([-\d,]+)\)/gi;
 /**
  * Parse multi-project and legacy Qase ID markers from a test title.
  * - "(Qase ID: 123)" or "(Qase ID: 123,124)" → legacyIds, single-project.
@@ -42,7 +47,7 @@ const QASE_MARKER_REGEXP = /\(Qase\s+([A-Za-z0-9_]+):\s*([\d,]+)\)/gi;
  * @param title — test title that may contain "(Qase ID: …)" or "(Qase PROJECT_CODE: …)".
  * @returns legacyIds, projectMapping, and cleanedTitle with all markers stripped.
  */
-function parseProjectMappingFromTitle(title) {
+function parseProjectMappingFromTitle(title, logger) {
     const legacyIds = [];
     const projectMapping = {};
     let cleanedTitle = title;
@@ -51,7 +56,11 @@ function parseProjectMappingFromTitle(title) {
     while ((m = re.exec(title)) !== null) {
         const projectCode = m[1] ?? '';
         const idsStr = m[2] ?? '';
-        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, filter_positive_ids_1.filterPositiveIds)(rawIds, logger);
         if (projectCode.toUpperCase() === 'ID') {
             legacyIds.push(...ids);
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "qase-javascript-commons",
-  "version": "2.7.3",
+  "version": "2.7.4",
   "description": "Qase JS Reporters",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",