@testomatio/reporter 2.3.7-beta.10-stack-artifacts → 2.3.7-beta.11-fix-vite

package/lib/bin/startTest.js

@@ -35,7 +35,7 @@ while (i < args.length) {
         newArgs[0] = 'start';
     }
     else if (arg === '--finish') {
-        // Map --finish to finish command  
+        // Map --finish to finish command
         newArgs[0] = 'finish';
     }
     else {
@@ -46,8 +46,8 @@ while (i < args.length) {
 }
 // Execute the main CLI with mapped arguments
 const child = (0, node_child_process_1.spawn)(process.execPath, [cliPath, ...newArgs], {
-    stdio: 'inherit'
+    stdio: 'inherit',
 });
-child.on('exit', (code) => {
+child.on('exit', code => {
     process.exit(code);
 });

package/lib/client.js

@@ -51,9 +51,7 @@ const node_url_1 = require("node:url");
 const uploader_js_1 = require("./uploader.js");
 const utils_js_1 = require("./utils/utils.js");
 const filesize_1 = require("filesize");
-const util_1 = require("util");
 const debug = (0, debug_1.default)('@testomatio/reporter:client');
-const stripColors = util_1.stripVTControlCharacters || ((str) => str?.replace(/\x1b\[[0-9;]*m/g, '') || '');
 // removed __dirname usage, because:
 // 1. replaced with ESM syntax (import.meta.url), but it throws an error on tsc compilation;
 // 2. got error "__dirname already defined" in compiles js code (cjs dir)
@@ -160,6 +158,17 @@ class Client {
      * @returns {Promise<PipeResult[]>}
      */
     async addTestRun(status, testData) {
+        if (!this.pipes || !this.pipes.length)
+            this.pipes = await (0, index_js_1.pipesFactory)(this.paramsForPipesFactory || {}, this.pipeStore);
+        // all pipes disabled, skipping
+        if (!this.pipes?.filter(p => p.isEnabled).length)
+            return [];
+        if (isTestShouldBeExculedFromReport(testData))
+            return [];
+        if (status === constants_js_1.STATUS.SKIPPED && process.env.TESTOMATIO_EXCLUDE_SKIPPED) {
+            debug('Skipping test from report', testData?.title);
+            return []; // do not log skipped tests
+        }
         if (!testData)
             testData = {
                 title: 'Unknown test',
@@ -172,12 +181,9 @@ class Client {
         /**
          * @type {TestData}
          */
-        const { rid, error = null, steps: originalSteps, title, suite_title, } = testData;
-        const steps = originalSteps;
-        const uploadedFiles = [];
-        const stackArtifactsEnabled = (0, utils_js_1.transformEnvVarToBoolean)(process.env.TESTOMATIO_STACK_ARTIFACTS);
-        const { time = 0, example = null, files = [], filesBuffers = [], code = null, file, suite_id, test_id, timestamp, links, 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)
             .reduce((acc, [key, value]) => {
@@ -185,36 +191,19 @@ class Client {
                 acc[key] = value;
             return acc;
         }, {});
+        // Get links from storage using the test context
         const testContext = suite_title ? `${suite_title} ${title}` : title;
         let errorFormatted = '';
         if (error) {
             errorFormatted += this.formatError(error) || '';
             message = error?.message;
         }
-        let fullLogs = this.formatLogs({ error: errorFormatted, steps, logs: testData.logs });
-        if (stackArtifactsEnabled && status && (steps || testData.logs || error)) {
-            const timestamp = +new Date;
-            if (fullLogs && fullLogs.trim().length > 0) {
-                uploadedFiles.push(this.uploader.uploadFileAsBuffer(Buffer.from(stripColors(fullLogs), 'utf8'), [this.runId, rid, `logs_${timestamp}.txt`]));
-            }
-            fullLogs = '';
-        }
-        if (!this.pipes || !this.pipes.length)
-            this.pipes = await (0, index_js_1.pipesFactory)(this.paramsForPipesFactory || {}, this.pipeStore);
-        if (!this.pipes?.filter(p => p.isEnabled).length) {
-            if (uploadedFiles.length > 0) {
-                await Promise.all(uploadedFiles);
-            }
-            return [];
-        }
-        if (isTestShouldBeExculedFromReport(testData))
-            return [];
-        if (status === constants_js_1.STATUS.SKIPPED && process.env.TESTOMATIO_EXCLUDE_SKIPPED) {
-            debug('Skipping test from report', testData?.title);
-            return [];
-        }
+        // Attach logs
+        const fullLogs = this.formatLogs({ error: errorFormatted, steps, logs: testData.logs });
+        // add artifacts
         if (manuallyAttachedArtifacts?.length)
             files.push(...manuallyAttachedArtifacts);
+        const uploadedFiles = [];
         for (let f of files) {
             if (!f)
                 continue; // f === null
@@ -296,7 +285,7 @@ class Client {
                 const uploadedArtifacts = this.uploader.successfulUploads.map(file => ({
                     relativePath: file.path.replace(process.cwd(), ''),
                     link: file.link,
-                    sizePretty: file.size == null ? 'unknown' : (0, filesize_1.filesize)(file.size, { round: 0 }).toString(),
+                    sizePretty: (0, filesize_1.filesize)(file.size, { round: 0 }).toString(),
                 }));
                 uploadedArtifacts.forEach(upload => {
                     debug(`🟢Uploaded artifact`, `${upload.relativePath},`, 'size:', `${upload.sizePretty},`, 'link:', `${upload.link}`);
@@ -306,7 +295,7 @@ class Client {
                 console.log(constants_js_1.APP_PREFIX, `🗄️ ${this.uploader.failedUploads.length} artifacts 🔴${picocolors_1.default.bold('failed')} to upload`);
                 const failedUploads = this.uploader.failedUploads.map(file => ({
                     relativePath: file.path.replace(process.cwd(), ''),
-                    sizePretty: file.size == null ? 'unknown' : (0, filesize_1.filesize)(file.size, { round: 0 }).toString(),
+                    sizePretty: (0, filesize_1.filesize)(file.size, { round: 0 }).toString(),
                 }));
                 const pathPadding = Math.max(...failedUploads.map(upload => upload.relativePath.length)) + 1;
                 failedUploads.forEach(upload => {

package/lib/pipe/testomatio.js

@@ -156,7 +156,7 @@ class TestomatioPipe {
         if (!this.isEnabled)
             return;
         if (this.batch.isEnabled && this.isEnabled)
-            this.batch.intervalFunction = setInterval(this.#batchUpload, this.batch.intervalTime);
+            this.batch.intervalFunction = setInterval(() => this.#batchUpload(), this.batch.intervalTime);
         let buildUrl = process.env.BUILD_URL || process.env.CI_JOB_URL || process.env.CIRCLE_BUILD_URL;
         // GitHub Actions Url
         if (!buildUrl && process.env.GITHUB_RUN_ID) {
@@ -420,7 +420,7 @@ class TestomatioPipe {
             if (this.runUrl && this.proceed) {
                 const notFinishedMessage = picocolors_1.default.yellow(picocolors_1.default.bold('Run was not finished because of $TESTOMATIO_PROCEED'));
                 console.log(constants_js_1.APP_PREFIX, `📊 ${notFinishedMessage}. Report URL: ${picocolors_1.default.magenta(this.runUrl)}`);
-                console.log(constants_js_1.APP_PREFIX, `🛬 Run to finish it: TESTOMATIO_RUN=${this.runId} npx start-test-run --finish`);
+                console.log(constants_js_1.APP_PREFIX, `🛬 Run to finish it: TESTOMATIO_RUN=${this.runId} npx @testomatio/reporter finish`);
             }
             if (this.hasUnmatchedTests) {
                 console.log('');

package/lib/reporter.d.ts

@@ -1,10 +1,3 @@
-export { Client };
-export const STATUS: {
-    PASSED: string;
-    FAILED: string;
-    SKIPPED: string;
-    FINISHED: string;
-};
 export const artifact: (data: string | {
     path: string;
     type: string;
@@ -87,7 +80,7 @@ export const label: (key: string, value?: string | null) => void;
 export const linkTest: (...testIds: string[]) => void;
 export const linkJira: (...jiraIds: string[]) => void;
 declare namespace _default {
-    export let testomatioLogger: {
+    let testomatioLogger: {
         "__#13@#originalUserLogger": {
             assert(condition?: boolean, ...data: any[]): void;
             assert(value: any, message?: string, ...optionalParams: any[]): void;
@@ -155,13 +148,13 @@ declare namespace _default {
         }): void;
         prettyObjects: boolean;
     };
-    export let artifact: (data: string | {
+    let artifact: (data: string | {
         path: string;
         type: string;
         name: string;
     }, context?: any) => void;
-    export let log: (...args: any[]) => void;
-    export let logger: {
+    let log: (...args: any[]) => void;
+    let logger: {
         "__#13@#originalUserLogger": {
             assert(condition?: boolean, ...data: any[]): void;
             assert(value: any, message?: string, ...optionalParams: any[]): void;
@@ -229,15 +222,13 @@ declare namespace _default {
         }): void;
         prettyObjects: boolean;
     };
-    export let meta: (keyValue: {
+    let meta: (keyValue: {
         [key: string]: string;
     } | string, value?: string | null) => void;
-    export let step: (message: string) => void;
-    export let label: (key: string, value?: string | null) => void;
-    export let linkTest: (...testIds: string[]) => void;
-    export let linkJira: (...jiraIds: string[]) => void;
-    export { Client as TestomatioClient };
-    export { STATUS };
+    let step: (message: string) => 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;
@@ -246,4 +237,3 @@ export type LoggerService = typeof import("./services/index.js").services.logger
 export type MetaFunction = typeof import("./reporter-functions.js").default.keyValue;
 export type StepFunction = typeof import("./reporter-functions.js").default.step;
 export type LabelFunction = typeof import("./reporter-functions.js").default.label;
-import Client from './client.js';

package/lib/reporter.js

@@ -1,48 +1,13 @@
 "use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.linkJira = exports.linkTest = exports.label = exports.step = exports.meta = exports.logger = exports.log = exports.artifact = exports.STATUS = exports.Client = void 0;
-const client_js_1 = __importDefault(require("./client.js"));
-exports.Client = client_js_1.default;
-const TestomatioConstants = __importStar(require("./constants.js"));
+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");
 const reporter_functions_js_1 = __importDefault(require("./reporter-functions.js"));
-exports.STATUS = TestomatioConstants.STATUS;
 exports.artifact = reporter_functions_js_1.default.artifact;
 exports.log = reporter_functions_js_1.default.log;
 exports.logger = index_js_1.services.logger;
@@ -72,6 +37,6 @@ module.exports = {
     label: reporter_functions_js_1.default.label,
     linkTest: reporter_functions_js_1.default.linkTest,
     linkJira: reporter_functions_js_1.default.linkJira,
-    TestomatioClient: client_js_1.default,
-    STATUS: exports.STATUS,
+    // TestomatClient,
+    // TRConstants,
 };