@testomatio/reporter 2.3.0-beta.5-links → 2.3.0

package/lib/adapter/codecept.js

@@ -26,6 +26,8 @@ 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,6 +105,7 @@ 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,
@@ -114,6 +115,7 @@ class CucumberReporter extends cucumber_1.Formatter {
                 .trim(),
             example: { ...example },
             logs,
+            links,
             manuallyAttachedArtifacts: artifacts,
             meta: keyValues,
             title: scenario,

package/lib/adapter/jest.js

@@ -57,6 +57,7 @@ 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
@@ -69,6 +70,7 @@ class JestReporter {
                     title,
                     time: duration,
                     logs,
+                    links,
                     manuallyAttachedArtifacts: artifacts,
                     meta: keyValues,
                 });

package/lib/adapter/playwright.js

@@ -57,6 +57,7 @@ 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 {{
@@ -94,6 +95,7 @@ class PlaywrightReporter {
             steps: steps.length ? steps : undefined,
             time: duration,
             logs,
+            links,
             manuallyAttachedArtifacts,
             meta: {
                 browser: project.browser,

package/lib/adapter/webdriver.js

@@ -78,12 +78,10 @@ class WebdriverReporter extends reporter_1.default {
     onTestEnd(test) {
         test.suite = test.parent;
         const logs = getTestLogs(test.fullTitle);
-        // 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.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);
         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
@@ -95,7 +93,7 @@ class WebdriverReporter extends reporter_1.default {
     async addTest(test) {
         if (!this.client)
             return;
-        const { title, _duration: duration, state, error, output } = test;
+        const { title, _duration: duration, state, error, output, links, artifacts, meta, logs } = test;
         const testId = (0, utils_js_1.getTestomatIdFromTestTitle)(title);
         const screenshotEndpoint = '/session/:sessionId/screenshot';
         const screenshotsBuffers = output
@@ -106,8 +104,9 @@ class WebdriverReporter extends reporter_1.default {
             rid,
             manuallyAttachedArtifacts: test.artifacts,
             error,
-            logs: test.logs,
-            meta: test.meta,
+            logs,
+            meta,
+            links,
             title,
             test_id: testId,
             time: duration,

package/lib/client.js

@@ -50,7 +50,6 @@ 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:
@@ -182,40 +181,11 @@ 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, manuallyAttachedArtifacts, overwrite, tags, } = 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, tags, } = 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;
@@ -223,7 +193,6 @@ 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}" data for constext "${context}":`, testData.join(', '));
+                    debug('<=', dataType, 'global', context, testData);
                 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}" data for context "${context}":`, testDataAsText);
+                    debug('<=', dataType, 'file', 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('Saving data to global variable for ', context, ':', data);
+        debug('=>', dataType, 'global', 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(`Saving data to file for context "${context}" to ${filepath}. Data: ${JSON.stringify(data)}`);
+        debug('=>', dataType, 'file', context, 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 hash;
+    return `${process.env.runId || 'run'}_${hash}`;
 }
 exports.dataStorage = DataStorage.getInstance();
 // TODO: consider using fs promises instead of writeSync/appendFileSync to

package/lib/reporter-functions.d.ts

@@ -5,6 +5,7 @@ declare namespace _default {
     export { setKeyValue as keyValue };
     export { setLabel as label };
     export { linkTest };
+    export { linkJira };
 }
 export default _default;
 /**
@@ -40,15 +41,21 @@ declare function setKeyValue(keyValue: {
     [key: string]: string;
 } | string, value?: string | null): void;
 /**
- * Add label(s) to the test report
+ * Add a single label to the test report
  * @param {string} key - label key (e.g. 'severity', 'feature', or just 'smoke' for labels without values)
- * @param {string|string[]|null} [value=null] - optional label value(s) (e.g. 'high', 'login') or array of values
+ * @param {string|null} [value=null] - optional label value (e.g. 'high', 'login')
  * @returns {void}
  */
-declare function setLabel(key: string, value?: string | string[] | null): void;
+declare function setLabel(key: string, value?: 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,15 +50,14 @@ function setKeyValue(keyValue, value = null) {
     index_js_1.services.keyValues.put(keyValue);
 }
 /**
- * Add label(s) to the test report
+ * Add a single label to the test report
  * @param {string} key - label key (e.g. 'severity', 'feature', or just 'smoke' for labels without values)
- * @param {string|string[]|null} [value=null] - optional label value(s) (e.g. 'high', 'login') or array of values
+ * @param {string|null} [value=null] - optional label value (e.g. 'high', 'login')
  * @returns {void}
  */
 function setLabel(key, value = null) {
     if (Array.isArray(value)) {
-        value.forEach(val => setLabel(key, val));
-        return;
+        return value.forEach(label => setLabel(key, label));
     }
     const labelObject = value !== null && value !== undefined && value !== ''
         ? { label: `${key}:${value}` }
@@ -74,6 +73,15 @@ 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,
@@ -81,4 +89,5 @@ 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: {
-    "__#14@#originalUserLogger": {
+    "__#13@#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;
     };
-    "__#14@#userLoggerWithOverridenMethods": any;
+    "__#13@#userLoggerWithOverridenMethods": any;
     logLevel: string;
     step(strings: any, ...values: any[]): void;
     getLogs(context: string): string[];
-    "__#14@#stringifyLogs"(...args: any[]): string;
+    "__#13@#stringifyLogs"(...args: any[]): string;
     _templateLiteralLog(strings: any, ...args: any[]): void;
-    "__#14@#logWrapper"(argsArray: any, level: any): void;
+    "__#13@#logWrapper"(argsArray: any, level: any): void;
     assert(...args: any[]): void;
     debug(...args: any[]): void;
     error(...args: any[]): void;
@@ -76,11 +76,12 @@ 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 | string[] | null) => void;
+export const label: (key: string, value?: string | null) => void;
 export const linkTest: (...testIds: string[]) => void;
+export const linkJira: (...jiraIds: string[]) => void;
 declare namespace _default {
     let testomatioLogger: {
-        "__#14@#originalUserLogger": {
+        "__#13@#originalUserLogger": {
             assert(condition?: boolean, ...data: any[]): void;
             assert(value: any, message?: string, ...optionalParams: any[]): void;
             clear(): void;
@@ -125,13 +126,13 @@ declare namespace _default {
             profile(label?: string): void;
             profileEnd(label?: string): void;
         };
-        "__#14@#userLoggerWithOverridenMethods": any;
+        "__#13@#userLoggerWithOverridenMethods": any;
         logLevel: string;
         step(strings: any, ...values: any[]): void;
         getLogs(context: string): string[];
-        "__#14@#stringifyLogs"(...args: any[]): string;
+        "__#13@#stringifyLogs"(...args: any[]): string;
         _templateLiteralLog(strings: any, ...args: any[]): void;
-        "__#14@#logWrapper"(argsArray: any, level: any): void;
+        "__#13@#logWrapper"(argsArray: any, level: any): void;
         assert(...args: any[]): void;
         debug(...args: any[]): void;
         error(...args: any[]): void;
@@ -154,7 +155,7 @@ declare namespace _default {
     }, context?: any) => void;
     let log: (...args: any[]) => void;
     let logger: {
-        "__#14@#originalUserLogger": {
+        "__#13@#originalUserLogger": {
             assert(condition?: boolean, ...data: any[]): void;
             assert(value: any, message?: string, ...optionalParams: any[]): void;
             clear(): void;
@@ -199,13 +200,13 @@ declare namespace _default {
             profile(label?: string): void;
             profileEnd(label?: string): void;
         };
-        "__#14@#userLoggerWithOverridenMethods": any;
+        "__#13@#userLoggerWithOverridenMethods": any;
         logLevel: string;
         step(strings: any, ...values: any[]): void;
         getLogs(context: string): string[];
-        "__#14@#stringifyLogs"(...args: any[]): string;
+        "__#13@#stringifyLogs"(...args: any[]): string;
         _templateLiteralLog(strings: any, ...args: any[]): void;
-        "__#14@#logWrapper"(argsArray: any, level: any): void;
+        "__#13@#logWrapper"(argsArray: any, level: any): void;
         assert(...args: any[]): void;
         debug(...args: any[]): void;
         error(...args: any[]): void;
@@ -225,8 +226,9 @@ declare namespace _default {
         [key: string]: string;
     } | string, value?: string | null) => void;
     let step: (message: string) => void;
-    let label: (key: string, value?: string | string[] | null) => void;
+    let label: (key: string, value?: string | null) => void;
     let linkTest: (...testIds: string[]) => void;
+    let linkJira: (...jiraIds: string[]) => void;
 }
 export default _default;
 export type ArtifactFunction = typeof import("./reporter-functions.js").default.artifact;

package/lib/reporter.js

@@ -3,7 +3,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.linkTest = exports.label = exports.step = exports.meta = exports.logger = exports.log = exports.artifact = void 0;
+exports.linkJira = exports.linkTest = exports.label = exports.step = exports.meta = exports.logger = exports.log = exports.artifact = void 0;
 // import TestomatClient from './client.js';
 // import * as TRConstants from './constants.js';
 const index_js_1 = require("./services/index.js");
@@ -15,6 +15,7 @@ exports.meta = reporter_functions_js_1.default.keyValue;
 exports.step = reporter_functions_js_1.default.step;
 exports.label = reporter_functions_js_1.default.label;
 exports.linkTest = reporter_functions_js_1.default.linkTest;
+exports.linkJira = reporter_functions_js_1.default.linkJira;
 /**
  * @typedef {typeof import('./reporter-functions.js').default.artifact} ArtifactFunction
  * @typedef {typeof import('./reporter-functions.js').default.log} LogFunction
@@ -35,6 +36,7 @@ module.exports = {
     step: reporter_functions_js_1.default.step,
     label: reporter_functions_js_1.default.label,
     linkTest: reporter_functions_js_1.default.linkTest,
+    linkJira: reporter_functions_js_1.default.linkJira,
     // TestomatClient,
     // TRConstants,
 };

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 "__#15@#instance": any;
+    static "__#14@#instance": any;
     /**
      * Singleton
      * @returns {ArtifactStorage}

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

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

package/lib/services/labels.d.ts

@@ -1,22 +0,0 @@
-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

@@ -1,62 +0,0 @@
-"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 "__#13@#instance": any;
+    static "__#16@#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 "__#14@#instance": any;
+    static "__#13@#instance": any;
     /**
      *
      * @returns {Logger}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testomatio/reporter",
-  "version": "2.3.0-beta.5-links",
+  "version": "2.3.0",
   "description": "Testomatio Reporter Client",
   "engines": {
     "node": ">=18"

package/src/adapter/codecept.js

@@ -25,6 +25,9 @@ const HOOK_EXECUTION_ORDER = {
   POST_TEST: ['AfterHook', 'AfterSuiteHook']
 };
 
+// codeceptjs workers are self-contained
+dataStorage.isFileStorage = false;
+
 const DATA_REGEXP = /[|\s]+?(\{".*\}|\[.*\])/;
 
 if (MAJOR_VERSION < 3) {

package/src/adapter/cucumber/current.js

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

package/src/adapter/jest.js

@@ -59,6 +59,7 @@ export class JestReporter {
       const logs = getTestLogs(result);
       const artifacts = services.artifacts.get(result.fullName);
       const keyValues = services.keyValues.get(result.fullName);
+      const links = 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.
@@ -72,6 +73,7 @@ export class JestReporter {
           title,
           time: duration,
           logs,
+          links,
           manuallyAttachedArtifacts: artifacts,
           meta: keyValues,
         });

package/src/adapter/playwright.js

@@ -61,6 +61,7 @@ class PlaywrightReporter {
     }
     const manuallyAttachedArtifacts = services.artifacts.get(fullTestTitle);
     const testMeta = services.keyValues.get(fullTestTitle);
+    const links = services.links.get(fullTestTitle);
     const rid = test.id || test.testId || uuidv4();
 
     /**
@@ -99,6 +100,7 @@ class PlaywrightReporter {
       steps: steps.length ? steps : undefined,
       time: duration,
       logs,
+      links,
       manuallyAttachedArtifacts,
       meta: {
         browser: project.browser,

package/src/adapter/webdriver.js

@@ -53,12 +53,11 @@ class WebdriverReporter extends WDIOReporter {
   onTestEnd(test) {
     test.suite = test.parent;
     const logs = getTestLogs(test.fullTitle);
-    // still be under investigation
-    const artifacts = services.artifacts.get(test.fullTitle);
-    const keyValues = services.keyValues.get(test.fullTitle);
+
+    test.artifacts = services.artifacts.get(test.fullTitle);
+    test.meta = services.keyValues.get(test.fullTitle);
+    test.links = services.links.get(test.fullTitle);
     test.logs = logs;
-    test.artifacts = artifacts;
-    test.meta = keyValues;
 
     this._addTestPromises.push(this.addTest(test));
   }
@@ -73,7 +72,7 @@ class WebdriverReporter extends WDIOReporter {
   async addTest(test) {
     if (!this.client) return;
 
-    const { title, _duration: duration, state, error, output } = test;
+    const { title, _duration: duration, state, error, output, links, artifacts, meta, logs } = test;
 
     const testId = getTestomatIdFromTestTitle(title);
 
@@ -88,8 +87,9 @@ class WebdriverReporter extends WDIOReporter {
       rid,
       manuallyAttachedArtifacts: test.artifacts,
       error,
-      logs: test.logs,
-      meta: test.meta,
+      logs,
+      meta,
+      links,
       title,
       test_id: testId,
       time: duration,

package/src/client.js

@@ -11,7 +11,6 @@ import path, { sep } from 'path';
 import { fileURLToPath } from 'node:url';
 import { S3Uploader } from './uploader.js';
 import { formatStep, readLatestRunId, storeRunId, validateSuiteId } from './utils/utils.js';
-import { linkStorage } from './services/links.js';
 import { filesize as prettyBytes } from 'filesize';
 
 const debug = createDebugMessages('@testomatio/reporter:client');
@@ -182,6 +181,7 @@ class Client {
       suite_id,
       test_id,
       timestamp,
+      links,
       manuallyAttachedArtifacts,
       overwrite,
       tags,
@@ -191,35 +191,6 @@ class Client {
     // 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(APP_PREFIX, `Can't convert meta value to string`, err);
-            }
-          }
-
-          if (value?.length > 255) {
-            value = value.substring(0, 255);
-            debug(APP_PREFIX, `Meta info value "${value}" is too long, trimmed to 255 characters`);
-          }
-
-          if (key?.length > 255) {
-            const newKey = key.substring(0, 255);
-            debug(APP_PREFIX, `Meta info key "${key}" is too long, trimmed to 255 characters`);
-            return [newKey, value];
-          }
-
-          return [key, value];
-        } catch (err) {
-          debug(APP_PREFIX, `Error while processing meta info key ${key}`, err);
-          return [null, null];
-        }
-      })
       .reduce((acc, [key, value]) => {
         if (key) acc[key] = value;
         return acc;
@@ -227,7 +198,6 @@ class Client {
 
     // Get links from storage using the test context
     const testContext = suite_title ? `${suite_title} ${title}` : title;
-    const links = linkStorage.get(testContext) || [];
 
     let errorFormatted = '';
     if (error) {

package/src/data-storage.js

@@ -116,7 +116,7 @@ class DataStorage {
     try {
       if (global?.testomatioDataStore[dataType]) {
         const testData = global.testomatioDataStore[dataType][context];
-        if (testData) debug(`"${dataType}" data for constext "${context}":`, testData.join(', '));
+        if (testData) debug('<=', dataType, 'global', context, testData);
         return testData || [];
       }
       // debug(`No ${this.dataType} data for context ${context} in <global> storage`);
@@ -137,7 +137,7 @@ class DataStorage {
       const filepath = join(dataDirPath, `${dataType}_${context}`);
       if (fs.existsSync(filepath)) {
         const testDataAsText = fs.readFileSync(filepath, 'utf-8');
-        if (testDataAsText) debug(`"${dataType}" data for context "${context}":`, testDataAsText);
+        if (testDataAsText) debug('<=', dataType, 'file', context, testDataAsText);
         const testDataArr = testDataAsText?.split(os.EOL) || [];
         return testDataArr;
       }
@@ -156,7 +156,7 @@ class DataStorage {
    * @param {*} context
    */
   #putDataToGlobalVar(dataType, data, context) {
-    debug('Saving data to global variable for ', context, ':', data);
+    debug('=>', dataType, 'global', context, data);
     if (!global.testomatioDataStore) global.testomatioDataStore = {};
     if (!global.testomatioDataStore?.[dataType]) global.testomatioDataStore[dataType] = {};
 
@@ -177,7 +177,7 @@ class DataStorage {
     const filename = `${dataType}_${context}`;
     const filepath = join(dataDirPath, filename);
     if (!fs.existsSync(dataDirPath)) fileSystem.createDir(dataDirPath);
-    debug(`Saving data to file for context "${context}" to ${filepath}. Data: ${JSON.stringify(data)}`);
+    debug('=>', dataType, 'file', context, data);
 
     // append new line if file already exists (in this case its definitely includes some data)
     if (fs.existsSync(filepath)) {
@@ -192,8 +192,7 @@ function stringToMD5Hash(str) {
   const md5 = crypto.createHash('md5');
   md5.update(str);
   const hash = md5.digest('hex');
-
-  return hash;
+  return `${process.env.runId || 'run'}_${hash}`;
 }
 
 export const dataStorage = DataStorage.getInstance();

package/src/reporter-functions.js

@@ -53,24 +53,21 @@ function setKeyValue(keyValue, value = null) {
 }
 
 /**
- * Add label(s) to the test report
+ * Add a single label to the test report
  * @param {string} key - label key (e.g. 'severity', 'feature', or just 'smoke' for labels without values)
- * @param {string|string[]|null} [value=null] - optional label value(s) (e.g. 'high', 'login') or array of values
+ * @param {string|null} [value=null] - optional label value (e.g. 'high', 'login')
  * @returns {void}
  */
 function setLabel(key, value = null) {
   if (Array.isArray(value)) {
-    value.forEach(val => setLabel(key, val));
-    return;
+    return value.forEach(label => setLabel(key, label));
   }
-
-  const labelObject = value !== null && value !== undefined && value !== '' 
-    ? { label: `${key}:${value}` } 
+  const labelObject = value !== null && value !== undefined && value !== ''
+    ? { label: `${key}:${value}` }
     : { label: key };
   services.links.put([labelObject]);
 }
 
-
 /**
  * Add link(s) to the test report
  * @param {...string} testIds - test IDs to link
@@ -81,6 +78,16 @@ function linkTest(...testIds) {
   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 }));
+  services.links.put(links);
+}
+
 export default {
   artifact: saveArtifact,
   log: logMessage,
@@ -88,4 +95,5 @@ export default {
   keyValue: setKeyValue,
   label: setLabel,
   linkTest,
+  linkJira,
 };

package/src/reporter.js

@@ -10,6 +10,7 @@ export const meta = reporterFunctions.keyValue;
 export const step = reporterFunctions.step;
 export const label = reporterFunctions.label;
 export const linkTest = reporterFunctions.linkTest;
+export const linkJira = reporterFunctions.linkJira;
 
 /**
  * @typedef {typeof import('./reporter-functions.js').default.artifact} ArtifactFunction
@@ -32,6 +33,7 @@ export default {
   step: reporterFunctions.step,
   label: reporterFunctions.label,
   linkTest: reporterFunctions.linkTest,
+  linkJira: reporterFunctions.linkJira,
 
   // TestomatClient,
   // TRConstants,

package/src/services/labels.js

@@ -1,59 +1 @@
-import createDebugMessages from 'debug';
-import { dataStorage } from '../data-storage.js';
 
-const debug = createDebugMessages('@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;
-    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 = 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)];
-  }
-}
-
-export const labelStorage = LabelStorage.getInstance();