@testomatio/reporter 2.9.1-beta.1-allure-chunking → 2.9.1-beta.2-allure-tms-link

package/lib/allureReader.d.ts

@@ -52,6 +52,34 @@ declare class AllureReader {
     extractLinks(result: any): {
         label: string;
     }[];
+    /**
+     * Extract a Testomat.io test id from Allure links so reported tests match
+     * existing cases instead of creating duplicates.
+     *
+     * Allure's `@TmsLink("T1a2b3c4d")` produces a link with `type: "tms"`. Some exporters
+     * omit the type but still point the link URL at a Testomat.io test page; both are
+     * accepted. The link `name` is used as the id (falling back to the last URL segment).
+     *
+     * @param {object} result - Parsed Allure result JSON
+     * @returns {string|null} Normalized test id, or null when no usable link exists
+     */
+    extractTestId(result: object): string | null;
+    /**
+     * Normalize a value into a Testomat.io test id.
+     *
+     * Testomat.io test ids are exactly **8 word characters**. The value may arrive bare
+     * (`1a2b3c4d`), or carrying the `T` / `@T` markers Testomat uses in code and titles
+     * (`T1a2b3c4d`, `@T1a2b3c4d`). The markers are removed only when doing so still leaves
+     * a valid 8-char id, so a real id that happens to start with `T` is preserved.
+     *
+     * Anything that does not resolve to a valid 8-char id — a numeric Allure TestOps id
+     * like `12345`, a JIRA key, a 6-digit TMS number — is rejected (returns null) so we
+     * never send an unmatchable id that would create duplicates.
+     *
+     * @param {string|number|null|undefined} value
+     * @returns {string|null} The bare 8-char id, or null when the value is not a valid id
+     */
+    normalizeTestId(value: string | number | null | undefined): string | null;
     convertSteps(steps: any, depth?: number): any;
     calculateRunTime(item: any): number;
     convertParameters(parameters: any): {};

package/lib/allureReader.js

@@ -169,6 +169,12 @@ class AllureReader {
             create: true,
             overwrite: true,
         };
+        // Use the @TmsLink / Testomat.io link as the test id so reported tests MATCH
+        // existing cases instead of creating duplicates on every run.
+        const testId = this.extractTestId(result);
+        if (testId) {
+            test.test_id = testId;
+        }
         // Add description if present
         if (result.description) {
             test.description = result.description;
@@ -296,6 +302,56 @@ class AllureReader {
         }
         return links.length > 0 ? links : undefined;
     }
+    /**
+     * Extract a Testomat.io test id from Allure links so reported tests match
+     * existing cases instead of creating duplicates.
+     *
+     * Allure's `@TmsLink("T1a2b3c4d")` produces a link with `type: "tms"`. Some exporters
+     * omit the type but still point the link URL at a Testomat.io test page; both are
+     * accepted. The link `name` is used as the id (falling back to the last URL segment).
+     *
+     * @param {object} result - Parsed Allure result JSON
+     * @returns {string|null} Normalized test id, or null when no usable link exists
+     */
+    extractTestId(result) {
+        const links = result.links || [];
+        if (!links.length)
+            return null;
+        const isTmsLink = l => typeof l?.type === 'string' && l.type.toLowerCase() === 'tms';
+        const isTestomatioLink = l => typeof l?.url === 'string' && /testomat\.io\/[^\s]*\/test\//i.test(l.url);
+        const link = links.find(isTmsLink) || links.find(isTestomatioLink);
+        if (!link)
+            return null;
+        // Prefer the explicit link name; fall back to the id segment of a Testomat.io URL.
+        let id = this.normalizeTestId(link.name);
+        if (!id && typeof link.url === 'string') {
+            const fromUrl = link.url.match(/\/test\/([\w\d]{8})(?=$|[/?#])/i);
+            if (fromUrl)
+                id = fromUrl[1];
+        }
+        return id;
+    }
+    /**
+     * Normalize a value into a Testomat.io test id.
+     *
+     * Testomat.io test ids are exactly **8 word characters**. The value may arrive bare
+     * (`1a2b3c4d`), or carrying the `T` / `@T` markers Testomat uses in code and titles
+     * (`T1a2b3c4d`, `@T1a2b3c4d`). The markers are removed only when doing so still leaves
+     * a valid 8-char id, so a real id that happens to start with `T` is preserved.
+     *
+     * Anything that does not resolve to a valid 8-char id — a numeric Allure TestOps id
+     * like `12345`, a JIRA key, a 6-digit TMS number — is rejected (returns null) so we
+     * never send an unmatchable id that would create duplicates.
+     *
+     * @param {string|number|null|undefined} value
+     * @returns {string|null} The bare 8-char id, or null when the value is not a valid id
+     */
+    normalizeTestId(value) {
+        if (value === null || value === undefined)
+            return null;
+        const match = value.toString().trim().match(/^@?T?([\w\d]{8})$/);
+        return match ? match[1] : null;
+    }
     convertSteps(steps, depth = 0) {
         if (depth >= 10)
             return null;
@@ -413,8 +469,10 @@ class AllureReader {
                     t.code = code;
                     debug('Fetched code for test %s', t.title);
                 }
+                // Don't override an id already taken from a @TmsLink — the link is the
+                // explicit, source-independent match key the client maintains.
                 const testId = (0, utils_js_1.fetchIdFromCode)(contents, { lang: this.getLanguage() });
-                if (testId) {
+                if (testId && !t.test_id) {
                     t.test_id = testId;
                     debug('Fetched test id %s for test %s', testId, t.title);
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testomatio/reporter",
-  "version": "2.9.1-beta.1-allure-chunking",
+  "version": "2.9.1-beta.2-allure-tms-link",
   "description": "Testomatio Reporter Client",
   "engines": {
     "node": ">=18"

package/src/allureReader.js

@@ -193,6 +193,13 @@ class AllureReader {
       overwrite: true,
     };
 
+    // Use the @TmsLink / Testomat.io link as the test id so reported tests MATCH
+    // existing cases instead of creating duplicates on every run.
+    const testId = this.extractTestId(result);
+    if (testId) {
+      test.test_id = testId;
+    }
+
     // Add description if present
     if (result.description) {
       test.description = result.description;
@@ -346,6 +353,58 @@ class AllureReader {
     return links.length > 0 ? links : undefined;
   }
 
+  /**
+   * Extract a Testomat.io test id from Allure links so reported tests match
+   * existing cases instead of creating duplicates.
+   *
+   * Allure's `@TmsLink("T1a2b3c4d")` produces a link with `type: "tms"`. Some exporters
+   * omit the type but still point the link URL at a Testomat.io test page; both are
+   * accepted. The link `name` is used as the id (falling back to the last URL segment).
+   *
+   * @param {object} result - Parsed Allure result JSON
+   * @returns {string|null} Normalized test id, or null when no usable link exists
+   */
+  extractTestId(result) {
+    const links = result.links || [];
+    if (!links.length) return null;
+
+    const isTmsLink = l => typeof l?.type === 'string' && l.type.toLowerCase() === 'tms';
+    const isTestomatioLink = l => typeof l?.url === 'string' && /testomat\.io\/[^\s]*\/test\//i.test(l.url);
+
+    const link = links.find(isTmsLink) || links.find(isTestomatioLink);
+    if (!link) return null;
+
+    // Prefer the explicit link name; fall back to the id segment of a Testomat.io URL.
+    let id = this.normalizeTestId(link.name);
+    if (!id && typeof link.url === 'string') {
+      const fromUrl = link.url.match(/\/test\/([\w\d]{8})(?=$|[/?#])/i);
+      if (fromUrl) id = fromUrl[1];
+    }
+
+    return id;
+  }
+
+  /**
+   * Normalize a value into a Testomat.io test id.
+   *
+   * Testomat.io test ids are exactly **8 word characters**. The value may arrive bare
+   * (`1a2b3c4d`), or carrying the `T` / `@T` markers Testomat uses in code and titles
+   * (`T1a2b3c4d`, `@T1a2b3c4d`). The markers are removed only when doing so still leaves
+   * a valid 8-char id, so a real id that happens to start with `T` is preserved.
+   *
+   * Anything that does not resolve to a valid 8-char id — a numeric Allure TestOps id
+   * like `12345`, a JIRA key, a 6-digit TMS number — is rejected (returns null) so we
+   * never send an unmatchable id that would create duplicates.
+   *
+   * @param {string|number|null|undefined} value
+   * @returns {string|null} The bare 8-char id, or null when the value is not a valid id
+   */
+  normalizeTestId(value) {
+    if (value === null || value === undefined) return null;
+    const match = value.toString().trim().match(/^@?T?([\w\d]{8})$/);
+    return match ? match[1] : null;
+  }
+
   convertSteps(steps, depth = 0) {
     if (depth >= 10) return null;
 
@@ -482,8 +541,10 @@ class AllureReader {
           debug('Fetched code for test %s', t.title);
         }
 
+        // Don't override an id already taken from a @TmsLink — the link is the
+        // explicit, source-independent match key the client maintains.
         const testId = fetchIdFromCode(contents, { lang: this.getLanguage() });
-        if (testId) {
+        if (testId && !t.test_id) {
           t.test_id = testId;
           debug('Fetched test id %s for test %s', testId, t.title);
         }