@testomatio/reporter 2.3.7-beta.4-stack-artifacts → 2.3.7-beta.6-stack-artifacts

package/lib/client.js

@@ -51,7 +51,9 @@ 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)
@@ -171,34 +173,9 @@ class Client {
          * @type {TestData}
          */
         const { rid, error = null, steps: originalSteps, title, suite_title, } = testData;
-        let steps = originalSteps;
+        const steps = originalSteps;
         const uploadedFiles = [];
         const stackArtifactsEnabled = (0, utils_js_1.transformEnvVarToBoolean)(process.env.TESTOMATIO_STACK_ARTIFACTS);
-        let formattedSteps;
-        if (stackArtifactsEnabled) {
-            const timestamp = +new Date;
-            formattedSteps = Array.isArray(steps) ? steps.map(step => (0, utils_js_1.formatStep)(step)).flat().join('\n') : '';
-            if (error?.stack?.length > 5000) {
-                uploadedFiles.push(this.uploader.uploadFileAsBuffer(Buffer.from(error.stack, 'utf8'), [this.runId, rid, `stack_${timestamp}.log`]));
-            }
-            if (formattedSteps?.length > 10000) {
-                uploadedFiles.push(this.uploader.uploadFileAsBuffer(Buffer.from(JSON.stringify(steps, null, 2), 'utf8'), [this.runId, rid, `steps_${timestamp}.json`]));
-            }
-        }
-        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 [];
-        }
         const { time = 0, example = null, files = [], filesBuffers = [], code = null, file, suite_id, test_id, timestamp, links, manuallyAttachedArtifacts, overwrite, tags, } = testData;
         let { message = '', meta = {} } = testData;
         meta = Object.entries(meta)
@@ -214,20 +191,25 @@ class Client {
             errorFormatted += this.formatError(error) || '';
             message = error?.message;
         }
+        let fullLogs = this.formatLogs({ error: errorFormatted, steps, logs: testData.logs });
         if (stackArtifactsEnabled) {
-            if (error?.stack?.length > 5000)
-                errorFormatted = `[Large stack saved as artifact]`;
-            if (formattedSteps?.length > 10000)
-                steps = null;
+            const timestamp = +new Date;
+            uploadedFiles.push(this.uploader.uploadFileAsBuffer(Buffer.from(stripColors(fullLogs), 'utf8'), [this.runId, rid, `logs_${timestamp}.log`]));
+            fullLogs = null;
         }
-        else {
-            formattedSteps = Array.isArray(steps) ? steps.map(step => (0, utils_js_1.formatStep)(step)).flat().join('\n') : '';
+        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 [];
         }
-        let fullLogs = this.formatLogs({ error: errorFormatted, steps, logs: testData.logs });
-        if (stackArtifactsEnabled && fullLogs.length > 5000) {
-            const timestamp = +new Date;
-            uploadedFiles.push(this.uploader.uploadFileAsBuffer(Buffer.from(fullLogs, 'utf8'), [this.runId, rid, `logs_${timestamp}.log`]));
-            fullLogs = fullLogs.slice(0, 5000) + '\n\n[Full logs saved as artifact]';
+        if (isTestShouldBeExculedFromReport(testData))
+            return [];
+        if (status === constants_js_1.STATUS.SKIPPED && process.env.TESTOMATIO_EXCLUDE_SKIPPED) {
+            debug('Skipping test from report', testData?.title);
+            return [];
         }
         if (manuallyAttachedArtifacts?.length)
             files.push(...manuallyAttachedArtifacts);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testomatio/reporter",
-  "version": "2.3.7-beta.4-stack-artifacts",
+  "version": "2.3.7-beta.6-stack-artifacts",
   "description": "Testomatio Reporter Client",
   "engines": {
     "node": ">=18"

package/src/client.js

@@ -19,9 +19,12 @@ import {
   transformEnvVarToBoolean
 } from './utils/utils.js';
 import { filesize as prettyBytes } from 'filesize';
+import { stripVTControlCharacters } from 'util';
 
 const debug = createDebugMessages('@testomatio/reporter:client');
 
+const stripColors = 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)
@@ -167,50 +170,12 @@ class Client {
       title,
       suite_title,
     } = testData;
-    let steps = originalSteps;
+    const steps = originalSteps;
 
     const uploadedFiles = [];
     const stackArtifactsEnabled = transformEnvVarToBoolean(process.env.TESTOMATIO_STACK_ARTIFACTS);
 
-    let formattedSteps;
-    if (stackArtifactsEnabled) {
-      const timestamp = +new Date;
-      formattedSteps = Array.isArray(steps) ? steps.map(step => formatStep(step)).flat().join('\n') : '';
-
-      if (error?.stack?.length > 5000) {
-        uploadedFiles.push(
-          this.uploader.uploadFileAsBuffer(
-            Buffer.from(error.stack, 'utf8'),
-            [this.runId, rid, `stack_${timestamp}.log`]
-          )
-        );
-      }
-      if (formattedSteps?.length > 10000) {
-        uploadedFiles.push(
-          this.uploader.uploadFileAsBuffer(
-            Buffer.from(JSON.stringify(steps, null, 2), 'utf8'),
-            [this.runId, rid, `steps_${timestamp}.json`]
-          )
-        );
-      }
-    }
-    if (!this.pipes || !this.pipes.length)
-      this.pipes = await 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 === STATUS.SKIPPED && process.env.TESTOMATIO_EXCLUDE_SKIPPED) {
-      debug('Skipping test from report', testData?.title);
-      return [];
-    }
-
+    
     const {
       time = 0,
       example = null,
@@ -243,24 +208,35 @@ class Client {
       message = error?.message;
     }
 
-    if (stackArtifactsEnabled) {
-      if (error?.stack?.length > 5000) errorFormatted = `[Large stack saved as artifact]`;
-      if (formattedSteps?.length > 10000) steps = null;
-    } else {
-      formattedSteps = Array.isArray(steps) ? steps.map(step => formatStep(step)).flat().join('\n') : '';
-    }
-
     let fullLogs = this.formatLogs({ error: errorFormatted, steps, logs: testData.logs });
 
-    if (stackArtifactsEnabled && fullLogs.length > 5000) {
+    if (stackArtifactsEnabled) {
       const timestamp = +new Date;
       uploadedFiles.push(
         this.uploader.uploadFileAsBuffer(
-          Buffer.from(fullLogs, 'utf8'),
+          Buffer.from(stripColors(fullLogs), 'utf8'),
           [this.runId, rid, `logs_${timestamp}.log`]
         )
       );
-      fullLogs = fullLogs.slice(0, 5000) + '\n\n[Full logs saved as artifact]';
+      fullLogs = null;
+    }
+
+    
+    if (!this.pipes || !this.pipes.length)
+      this.pipes = await 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 === STATUS.SKIPPED && process.env.TESTOMATIO_EXCLUDE_SKIPPED) {
+      debug('Skipping test from report', testData?.title);
+      return [];
     }
 
     if (manuallyAttachedArtifacts?.length) files.push(...manuallyAttachedArtifacts);