@testomatio/reporter 2.3.0-beta.1-links → 2.3.0-beta.2-xml-import

package/lib/adapter/codecept.js

@@ -26,8 +26,6 @@ const HOOK_EXECUTION_ORDER = {
     PRE_TEST: ['BeforeSuiteHook', 'BeforeHook'],
     POST_TEST: ['AfterHook', 'AfterSuiteHook']
 };
-// codeceptjs workers are self-contained
-data_storage_js_1.dataStorage.isFileStorage = false;
 const DATA_REGEXP = /[|\s]+?(\{".*\}|\[.*\])/;
 if (MAJOR_VERSION < 3) {
     console.log('🔴 This reporter works with CodeceptJS 3+, please update your tests');

package/lib/adapter/cucumber/current.js

@@ -105,7 +105,6 @@ class CucumberReporter extends cucumber_1.Formatter {
         const logs = index_js_1.services.logger.getLogs(testTitle).join('\n');
         const artifacts = index_js_1.services.artifacts.get(testTitle);
         const keyValues = index_js_1.services.keyValues.get(testTitle);
-        const links = index_js_1.services.links.get(testTitle);
         this.client.addTestRun(status, {
             // error: testCaseAttempt.worstTestStepResult.message,
             message,
@@ -115,7 +114,6 @@ class CucumberReporter extends cucumber_1.Formatter {
                 .trim(),
             example: { ...example },
             logs,
-            links,
             manuallyAttachedArtifacts: artifacts,
             meta: keyValues,
             title: scenario,

package/lib/adapter/jest.js

@@ -57,7 +57,6 @@ class JestReporter {
             const logs = getTestLogs(result);
             const artifacts = index_js_1.services.artifacts.get(result.fullName);
             const keyValues = index_js_1.services.keyValues.get(result.fullName);
-            const links = index_js_1.services.links.get(result.fullName);
             const deducedStatus = status === 'pending' ? 'skipped' : status;
             // In jest if test is not matched with test name pattern it is considered as skipped.
             // So adding a check if it is skipped for real or because of test pattern
@@ -70,7 +69,6 @@ class JestReporter {
                     title,
                     time: duration,
                     logs,
-                    links,
                     manuallyAttachedArtifacts: artifacts,
                     meta: keyValues,
                 });

package/lib/adapter/playwright.js

@@ -55,7 +55,6 @@ class PlaywrightReporter {
         }
         const manuallyAttachedArtifacts = index_js_1.services.artifacts.get(fullTestTitle);
         const testMeta = index_js_1.services.keyValues.get(fullTestTitle);
-        const links = index_js_1.services.links.get(fullTestTitle);
         const rid = test.id || test.testId || (0, uuid_1.v4)();
         /**
          * @type {{
@@ -92,7 +91,6 @@ class PlaywrightReporter {
             steps: steps.length ? steps : undefined,
             time: duration,
             logs,
-            links,
             manuallyAttachedArtifacts,
             meta: {
                 browser: project.browser,

package/lib/adapter/webdriver.js

@@ -77,10 +77,14 @@ class WebdriverReporter extends reporter_1.default {
     onTestEnd(test) {
         test.suite = test.parent;
         const logs = getTestLogs(test.fullTitle);
-        test.artifacts = index_js_1.services.artifacts.get(test.fullTitle);
-        test.meta = index_js_1.services.keyValues.get(test.fullTitle);
-        test.links = index_js_1.services.links.get(test.fullTitle);
+        // TODO: FIX: artifacts for some reason leads to empty report on Testomat.io
+        // ^ not reproduced anymore (Jul 2025)
+        // but still be under investigation
+        const artifacts = index_js_1.services.artifacts.get(test.fullTitle);
+        const keyValues = index_js_1.services.keyValues.get(test.fullTitle);
         test.logs = logs;
+        test.artifacts = artifacts;
+        test.meta = keyValues;
         this._addTestPromises.push(this.addTest(test));
     }
     // wdio-cucumber does not trigger onTestEnd hook, thus, using this one
@@ -92,7 +96,7 @@ class WebdriverReporter extends reporter_1.default {
     async addTest(test) {
         if (!this.client)
             return;
-        const { title, _duration: duration, state, error, output, links, artifacts, meta, logs } = test;
+        const { title, _duration: duration, state, error, output } = test;
         const testId = (0, utils_js_1.getTestomatIdFromTestTitle)(title);
         const screenshotEndpoint = '/session/:sessionId/screenshot';
         const screenshotsBuffers = output
@@ -100,11 +104,10 @@ class WebdriverReporter extends reporter_1.default {
             .map(el => Buffer.from(el.result.value, 'base64'));
         await this.client.addTestRun(state, {
             rid: test.uid || '',
-            manuallyAttachedArtifacts: artifacts,
+            manuallyAttachedArtifacts: test.artifacts,
             error,
-            logs,
-            meta,
-            links,
+            logs: test.logs,
+            meta: test.meta,
             title,
             test_id: testId,
             time: duration,

package/lib/client.js

@@ -50,6 +50,7 @@ const path_1 = __importStar(require("path"));
 const node_url_1 = require("node:url");
 const uploader_js_1 = require("./uploader.js");
 const utils_js_1 = require("./utils/utils.js");
+const links_js_1 = require("./services/links.js");
 const filesize_1 = require("filesize");
 const debug = (0, debug_1.default)('@testomatio/reporter:client');
 // removed __dirname usage, because:
@@ -181,11 +182,40 @@ class Client {
         /**
          * @type {TestData}
          */
-        const { rid, error = null, time = 0, example = null, files = [], filesBuffers = [], steps, code = null, title, file, suite_title, suite_id, test_id, timestamp, links, manuallyAttachedArtifacts, overwrite, } = testData;
+        const { rid, error = null, time = 0, example = null, files = [], filesBuffers = [], steps, code = null, title, file, suite_title, suite_id, test_id, timestamp, manuallyAttachedArtifacts, overwrite, } = testData;
         let { message = '', meta = {} } = testData;
         // stringify meta values and limit keys and values length to 255
         meta = Object.entries(meta)
             .filter(([, value]) => value !== null && value !== undefined)
+            .map(([key, value]) => {
+            try {
+                if (typeof value === 'object') {
+                    value = JSON.stringify(value);
+                }
+                else if (typeof value !== 'string') {
+                    try {
+                        value = value.toString();
+                    }
+                    catch (err) {
+                        console.warn(constants_js_1.APP_PREFIX, `Can't convert meta value to string`, err);
+                    }
+                }
+                if (value?.length > 255) {
+                    value = value.substring(0, 255);
+                    debug(constants_js_1.APP_PREFIX, `Meta info value "${value}" is too long, trimmed to 255 characters`);
+                }
+                if (key?.length > 255) {
+                    const newKey = key.substring(0, 255);
+                    debug(constants_js_1.APP_PREFIX, `Meta info key "${key}" is too long, trimmed to 255 characters`);
+                    return [newKey, value];
+                }
+                return [key, value];
+            }
+            catch (err) {
+                debug(constants_js_1.APP_PREFIX, `Error while processing meta info key ${key}`, err);
+                return [null, null];
+            }
+        })
             .reduce((acc, [key, value]) => {
             if (key)
                 acc[key] = value;
@@ -193,6 +223,7 @@ class Client {
         }, {});
         // Get links from storage using the test context
         const testContext = suite_title ? `${suite_title} ${title}` : title;
+        const links = links_js_1.linkStorage.get(testContext) || [];
         let errorFormatted = '';
         if (error) {
             errorFormatted += this.formatError(error) || '';

package/lib/data-storage.js

@@ -143,7 +143,7 @@ class DataStorage {
             if (global?.testomatioDataStore[dataType]) {
                 const testData = global.testomatioDataStore[dataType][context];
                 if (testData)
-                    debug('<=', dataType, 'global', context, testData);
+                    debug(`"${dataType}" data for constext "${context}":`, testData.join(', '));
                 return testData || [];
             }
             // debug(`No ${this.dataType} data for context ${context} in <global> storage`);
@@ -165,7 +165,7 @@ class DataStorage {
             if (fs_1.default.existsSync(filepath)) {
                 const testDataAsText = fs_1.default.readFileSync(filepath, 'utf-8');
                 if (testDataAsText)
-                    debug('<=', dataType, 'file', context, testDataAsText);
+                    debug(`"${dataType}" data for context "${context}":`, testDataAsText);
                 const testDataArr = testDataAsText?.split(os_1.default.EOL) || [];
                 return testDataArr;
             }
@@ -184,7 +184,7 @@ class DataStorage {
      * @param {*} context
      */
     #putDataToGlobalVar(dataType, data, context) {
-        debug('=>', dataType, 'global', context, data);
+        debug('Saving data to global variable for ', context, ':', data);
         if (!global.testomatioDataStore)
             global.testomatioDataStore = {};
         if (!global.testomatioDataStore?.[dataType])
@@ -208,7 +208,7 @@ class DataStorage {
         const filepath = (0, path_1.join)(dataDirPath, filename);
         if (!fs_1.default.existsSync(dataDirPath))
             utils_js_1.fileSystem.createDir(dataDirPath);
-        debug('=>', dataType, 'file', context, data);
+        debug(`Saving data to file for context "${context}" to ${filepath}. Data: ${JSON.stringify(data)}`);
         // append new line if file already exists (in this case its definitely includes some data)
         if (fs_1.default.existsSync(filepath)) {
             fs_1.default.appendFileSync(filepath, os_1.default.EOL + data, 'utf-8');
@@ -222,7 +222,7 @@ function stringToMD5Hash(str) {
     const md5 = crypto_1.default.createHash('md5');
     md5.update(str);
     const hash = md5.digest('hex');
-    return `${process.env.runId || 'run'}_${hash}`;
+    return hash;
 }
 exports.dataStorage = DataStorage.getInstance();
 // TODO: consider using fs promises instead of writeSync/appendFileSync to

package/lib/reporter-functions.d.ts

@@ -5,7 +5,6 @@ declare namespace _default {
     export { setKeyValue as keyValue };
     export { setLabel as label };
     export { linkTest };
-    export { linkJira };
 }
 export default _default;
 /**
@@ -41,21 +40,15 @@ declare function setKeyValue(keyValue: {
     [key: string]: string;
 } | string, value?: string | null): void;
 /**
- * Add a single label to the test report
+ * Add label(s) to the test report
  * @param {string} key - label key (e.g. 'severity', 'feature', or just 'smoke' for labels without values)
- * @param {string|null} [value=null] - optional label value (e.g. 'high', 'login')
+ * @param {string|string[]|null} [value=null] - optional label value(s) (e.g. 'high', 'login') or array of values
  * @returns {void}
  */
-declare function setLabel(key: string, value?: string | null): void;
+declare function setLabel(key: string, value?: string | string[] | null): void;
 /**
  * Add link(s) to the test report
  * @param {...string} testIds - test IDs to link
  * @returns {void}
  */
 declare function linkTest(...testIds: string[]): void;
-/**
- * Add JIRA issue link(s) to the test report
- * @param {...string} jiraIds - JIRA issue IDs to link
- * @returns {void}
- */
-declare function linkJira(...jiraIds: string[]): void;

package/lib/reporter-functions.js

@@ -50,14 +50,15 @@ function setKeyValue(keyValue, value = null) {
     index_js_1.services.keyValues.put(keyValue);
 }
 /**
- * Add a single label to the test report
+ * Add label(s) to the test report
  * @param {string} key - label key (e.g. 'severity', 'feature', or just 'smoke' for labels without values)
- * @param {string|null} [value=null] - optional label value (e.g. 'high', 'login')
+ * @param {string|string[]|null} [value=null] - optional label value(s) (e.g. 'high', 'login') or array of values
  * @returns {void}
  */
 function setLabel(key, value = null) {
     if (Array.isArray(value)) {
-        return value.forEach(label => setLabel(key, label));
+        value.forEach(val => setLabel(key, val));
+        return;
     }
     const labelObject = value !== null && value !== undefined && value !== ''
         ? { label: `${key}:${value}` }
@@ -73,15 +74,6 @@ function linkTest(...testIds) {
     const links = testIds.map(testId => ({ test: testId }));
     index_js_1.services.links.put(links);
 }
-/**
- * Add JIRA issue link(s) to the test report
- * @param {...string} jiraIds - JIRA issue IDs to link
- * @returns {void}
- */
-function linkJira(...jiraIds) {
-    const links = jiraIds.map(jiraId => ({ jira: jiraId }));
-    index_js_1.services.links.put(links);
-}
 module.exports = {
     artifact: saveArtifact,
     log: logMessage,
@@ -89,5 +81,4 @@ module.exports = {
     keyValue: setKeyValue,
     label: setLabel,
     linkTest,
-    linkJira,
 };

package/lib/reporter.d.ts

@@ -5,7 +5,7 @@ export const artifact: (data: string | {
 }, context?: any) => void;
 export const log: (...args: any[]) => void;
 export const logger: {
-    "__#13@#originalUserLogger": {
+    "__#14@#originalUserLogger": {
         assert(condition?: boolean, ...data: any[]): void;
         assert(value: any, message?: string, ...optionalParams: any[]): void;
         clear(): void;
@@ -50,13 +50,13 @@ export const logger: {
         profile(label?: string): void;
         profileEnd(label?: string): void;
     };
-    "__#13@#userLoggerWithOverridenMethods": any;
+    "__#14@#userLoggerWithOverridenMethods": any;
     logLevel: string;
     step(strings: any, ...values: any[]): void;
     getLogs(context: string): string[];
-    "__#13@#stringifyLogs"(...args: any[]): string;
+    "__#14@#stringifyLogs"(...args: any[]): string;
     _templateLiteralLog(strings: any, ...args: any[]): void;
-    "__#13@#logWrapper"(argsArray: any, level: any): void;
+    "__#14@#logWrapper"(argsArray: any, level: any): void;
     assert(...args: any[]): void;
     debug(...args: any[]): void;
     error(...args: any[]): void;
@@ -76,11 +76,11 @@ export const meta: (keyValue: {
     [key: string]: string;
 } | string, value?: string | null) => void;
 export const step: (message: string) => void;
-export const label: (key: string, value?: string | null) => void;
+export const label: (key: string, value?: string | string[] | null) => void;
 export const linkTest: (...testIds: string[]) => void;
 declare namespace _default {
     let testomatioLogger: {
-        "__#13@#originalUserLogger": {
+        "__#14@#originalUserLogger": {
             assert(condition?: boolean, ...data: any[]): void;
             assert(value: any, message?: string, ...optionalParams: any[]): void;
             clear(): void;
@@ -125,13 +125,13 @@ declare namespace _default {
             profile(label?: string): void;
             profileEnd(label?: string): void;
         };
-        "__#13@#userLoggerWithOverridenMethods": any;
+        "__#14@#userLoggerWithOverridenMethods": any;
         logLevel: string;
         step(strings: any, ...values: any[]): void;
         getLogs(context: string): string[];
-        "__#13@#stringifyLogs"(...args: any[]): string;
+        "__#14@#stringifyLogs"(...args: any[]): string;
         _templateLiteralLog(strings: any, ...args: any[]): void;
-        "__#13@#logWrapper"(argsArray: any, level: any): void;
+        "__#14@#logWrapper"(argsArray: any, level: any): void;
         assert(...args: any[]): void;
         debug(...args: any[]): void;
         error(...args: any[]): void;
@@ -154,7 +154,7 @@ declare namespace _default {
     }, context?: any) => void;
     let log: (...args: any[]) => void;
     let logger: {
-        "__#13@#originalUserLogger": {
+        "__#14@#originalUserLogger": {
             assert(condition?: boolean, ...data: any[]): void;
             assert(value: any, message?: string, ...optionalParams: any[]): void;
             clear(): void;
@@ -199,13 +199,13 @@ declare namespace _default {
             profile(label?: string): void;
             profileEnd(label?: string): void;
         };
-        "__#13@#userLoggerWithOverridenMethods": any;
+        "__#14@#userLoggerWithOverridenMethods": any;
         logLevel: string;
         step(strings: any, ...values: any[]): void;
         getLogs(context: string): string[];
-        "__#13@#stringifyLogs"(...args: any[]): string;
+        "__#14@#stringifyLogs"(...args: any[]): string;
         _templateLiteralLog(strings: any, ...args: any[]): void;
-        "__#13@#logWrapper"(argsArray: any, level: any): void;
+        "__#14@#logWrapper"(argsArray: any, level: any): void;
         assert(...args: any[]): void;
         debug(...args: any[]): void;
         error(...args: any[]): void;
@@ -225,7 +225,7 @@ declare namespace _default {
         [key: string]: string;
     } | string, value?: string | null) => void;
     let step: (message: string) => void;
-    let label: (key: string, value?: string | null) => void;
+    let label: (key: string, value?: string | string[] | null) => void;
     let linkTest: (...testIds: string[]) => void;
 }
 export default _default;

package/lib/services/artifacts.d.ts

@@ -3,7 +3,7 @@ export const artifactStorage: ArtifactStorage;
  * Artifact storage is supposed to store file paths
  */
 declare class ArtifactStorage {
-    static "__#14@#instance": any;
+    static "__#15@#instance": any;
     /**
      * Singleton
      * @returns {ArtifactStorage}

package/lib/services/key-values.d.ts

@@ -1,6 +1,6 @@
 export const keyValueStorage: KeyValueStorage;
 declare class KeyValueStorage {
-    static "__#15@#instance": any;
+    static "__#16@#instance": any;
     /**
      *
      * @returns {KeyValueStorage}

package/lib/services/labels.d.ts

@@ -0,0 +1,22 @@
+export const labelStorage: LabelStorage;
+declare class LabelStorage {
+    static "__#19@#instance": any;
+    /**
+     *
+     * @returns {LabelStorage}
+     */
+    static getInstance(): LabelStorage;
+    /**
+     * Stores labels array and passes it to reporter
+     * @param {string[]} labels - array of label strings
+     * @param {*} context - full test title
+     */
+    put(labels: string[], context?: any): void;
+    /**
+     * Returns labels array for the test
+     * @param {*} context testId or test context from test runner
+     * @returns {string[]} labels array, e.g. ['smoke', 'severity:high', 'feature:user_account']
+     */
+    get(context?: any): string[];
+}
+export {};

package/lib/services/labels.js

@@ -0,0 +1,62 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.labelStorage = void 0;
+const debug_1 = __importDefault(require("debug"));
+const data_storage_js_1 = require("../data-storage.js");
+const debug = (0, debug_1.default)('@testomatio/reporter:services-labels');
+class LabelStorage {
+    static #instance;
+    /**
+     *
+     * @returns {LabelStorage}
+     */
+    static getInstance() {
+        if (!this.#instance) {
+            this.#instance = new LabelStorage();
+        }
+        return this.#instance;
+    }
+    /**
+     * Stores labels array and passes it to reporter
+     * @param {string[]} labels - array of label strings
+     * @param {*} context - full test title
+     */
+    put(labels, context = null) {
+        if (!labels || !Array.isArray(labels))
+            return;
+        data_storage_js_1.dataStorage.putData('links', labels, context);
+    }
+    /**
+     * Returns labels array for the test
+     * @param {*} context testId or test context from test runner
+     * @returns {string[]} labels array, e.g. ['smoke', 'severity:high', 'feature:user_account']
+     */
+    get(context = null) {
+        const labelsList = data_storage_js_1.dataStorage.getData('links', context);
+        if (!labelsList || !labelsList?.length)
+            return [];
+        const allLabels = [];
+        for (const labels of labelsList) {
+            if (Array.isArray(labels)) {
+                allLabels.push(...labels);
+            }
+            else if (typeof labels === 'string') {
+                try {
+                    const parsedLabels = JSON.parse(labels);
+                    if (Array.isArray(parsedLabels)) {
+                        allLabels.push(...parsedLabels);
+                    }
+                }
+                catch (e) {
+                    debug(`Error parsing labels for test ${context}`, labels);
+                }
+            }
+        }
+        // Remove duplicates
+        return [...new Set(allLabels)];
+    }
+}
+exports.labelStorage = LabelStorage.getInstance();

package/lib/services/links.d.ts

@@ -1,6 +1,6 @@
 export const linkStorage: LinkStorage;
 declare class LinkStorage {
-    static "__#16@#instance": any;
+    static "__#13@#instance": any;
     /**
      *
      * @returns {LinkStorage}

package/lib/services/logger.d.ts

@@ -5,7 +5,7 @@ export const logger: Logger;
  * Supports different syntaxes to satisfy any user preferences.
  */
 declare class Logger {
-    static "__#13@#instance": any;
+    static "__#14@#instance": any;
     /**
      *
      * @returns {Logger}

package/lib/xmlReader.d.ts

@@ -77,6 +77,13 @@ declare class XmlReader {
         skipped_count: number;
         tests: any[];
     };
+    deduplicateTestsByFQN(tests: any): any[];
+    generateFQN(test: any): string;
+    generateNormalizedFQN(test: any): string;
+    extractAssemblyName(test: any): any;
+    extractNamespace(test: any): any;
+    extractClassName(test: any): any;
+    extractCsFileFromPath(test: any): any;
     calculateStats(): {};
     fetchSourceCode(): void;
     formatTests(): void;