@testomatio/reporter 2.3.8 → 2.3.9

package/lib/xmlReader.js

@@ -221,9 +221,14 @@ class XmlReader {
     _parseTRXTestDefinition(td) {
         const title = td.name.replace(/\(.*?\)/, '').trim();
         const exampleMatch = td.name.match(/\((.*?)\)/);
-        const example = exampleMatch ? {
-            ...exampleMatch[1].split(',').map(p => p.trim()).filter(p => p !== '')
-        } : null;
+        const example = exampleMatch
+            ? {
+                ...exampleMatch[1]
+                    .split(',')
+                    .map(p => p.trim())
+                    .filter(p => p !== ''),
+            }
+            : null;
         const suite = td.TestMethod.className.split(', ')[0].split('.');
         const suite_title = suite.pop();
         // Convert namespace to file path for C#
@@ -518,7 +523,12 @@ function reduceTestCases(prev, item) {
         tags ||= [];
         const exampleMatches = testCaseItem.name?.match(/\S\((.*?)\)/);
         if (exampleMatches) {
-            example = { ...exampleMatches[1].split(',').map(v => v.trim().replace(/[^\w\s-]/g, '')).filter(v => v !== '') };
+            example = {
+                ...exampleMatches[1]
+                    .split(',')
+                    .map(v => v.trim().replace(/[^\w\s-]/g, ''))
+                    .filter(v => v !== ''),
+            };
             title = title.replace(/\(.*?\)/, '').trim();
         }
         stack = `${testCaseItem['system-out'] || testCaseItem.output || testCaseItem.log || ''}\n\n${stack}\n\n${suiteOutput}\n\n${suiteErr}`.trim();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testomatio/reporter",
-  "version": "2.3.8",
+  "version": "2.3.9",
   "description": "Testomatio Reporter Client",
   "engines": {
     "node": ">=18"
@@ -106,7 +106,7 @@
     "vitest": "^1.6.0"
   },
   "bin": {
-    "@testomatio/reporter": "./lib/bin/cli.js",
+    "testomatio/reporter": "./lib/bin/cli.js",
     "report-xml": "./lib/bin/reportXml.js",
     "start-test-run": "./lib/bin/startTest.js",
     "upload-artifacts": "./lib/bin/uploadArtifacts.js"

package/src/adapter/codecept.js

@@ -17,12 +17,15 @@ if (!global.codeceptjs) {
 // @ts-ignore
 const { event, recorder, codecept, output } = global.codeceptjs;
 
-const [, MAJOR_VERSION, MINOR_VERSION] = codecept.version().match(/(\d+)\.(\d+)/).map(Number);
+const [, MAJOR_VERSION, MINOR_VERSION] = codecept
+  .version()
+  .match(/(\d+)\.(\d+)/)
+  .map(Number);
 
 // Constants for hook execution order
 const HOOK_EXECUTION_ORDER = {
   PRE_TEST: ['BeforeSuiteHook', 'BeforeHook'],
-  POST_TEST: ['AfterHook', 'AfterSuiteHook']
+  POST_TEST: ['AfterHook', 'AfterSuiteHook'],
 };
 
 // codeceptjs workers are self-contained
@@ -35,7 +38,9 @@ if (MAJOR_VERSION < 3) {
 }
 
 if (MAJOR_VERSION === 3 && MINOR_VERSION < 7) {
-  console.log('🔴 CodeceptJS 3.7+ is supported, please upgrade CodeceptJS or use 1.6 version of `@testomatio/reporter`');
+  console.log(
+    '🔴 CodeceptJS 3.7+ is supported, please upgrade CodeceptJS or use 1.6 version of `@testomatio/reporter`',
+  );
 }
 
 function CodeceptReporter(config) {
@@ -57,18 +62,18 @@ function CodeceptReporter(config) {
     say: output.say,
   };
 
-  output.debug = function(msg) {
+  output.debug = function (msg) {
     originalOutput.debug(msg);
     dataStorage.putData('log', repeat(this?.stepShift || 0) + pc.cyan(msg.toString()));
   };
 
-  output.say = function(message, color = 'cyan') {
+  output.say = function (message, color = 'cyan') {
     originalOutput.say(message, color);
     const sayMsg = repeat(this?.stepShift || 0) + `  ${pc.bold(pc[color](message))}`;
     dataStorage.putData('log', sayMsg);
   };
 
-  output.log = function(msg) {
+  output.log = function (msg) {
     originalOutput.log(msg);
     dataStorage.putData('log', repeat(this?.stepShift || 0) + pc.gray(msg));
   };
@@ -108,7 +113,7 @@ function CodeceptReporter(config) {
   });
 
   // Hook event listeners
-  event.dispatcher.on(event.hook.started, (hook) => {
+  event.dispatcher.on(event.hook.started, hook => {
     output.stepShift = 2;
     currentHook = hook.name;
     let title = hook.hookName;
@@ -126,7 +131,6 @@ function CodeceptReporter(config) {
     services.setContext(null);
   });
 
-
   // mark as failed all tests inside the failed hook
   event.dispatcher.on(event.hook.failed, hook => {
     if (hook.name !== 'BeforeSuiteHook') return;
@@ -148,7 +152,6 @@ function CodeceptReporter(config) {
     }
   });
 
-
   event.dispatcher.on(event.suite.before, suite => {
     dataStorage.setContext(suite.fullTitle());
   });
@@ -167,7 +170,7 @@ function CodeceptReporter(config) {
     testTimeMap[test.uid] = Date.now();
   });
 
-  event.dispatcher.on(event.all.result, async (result) => {
+  event.dispatcher.on(event.all.result, async result => {
     debug('waiting for all tests to be reported');
     // all tests were reported and we can upload videos
     await Promise.all(reportTestPromises);
@@ -327,7 +330,7 @@ function captureHookStep(step, currentHook, hookSteps) {
     status: step.status,
     startTime,
     endTime,
-    helperMethod: step.helperMethod
+    helperMethod: step.helperMethod,
   });
   hookSteps.set(currentHook, hookStepsArray);
 }
@@ -422,13 +425,12 @@ function processTestSteps(steps, hierarchy) {
   }
 }
 
-
 function createSectionStep(metaStep) {
   return {
     category: 'user',
     title: metaStep.toString(), // Use built-in toString method
     duration: metaStep.duration || 0, // Use built-in duration
-    steps: []
+    steps: [],
   };
 }
 
@@ -439,7 +441,7 @@ function createHookSection(hookName, steps) {
     category: 'hook',
     title: formatHookName(hookName),
     duration: 0,
-    steps: []
+    steps: [],
   };
 
   for (const step of steps) {
@@ -457,7 +459,6 @@ function formatHookName(hookName) {
   return hookName.replace(/Hook$/, '');
 }
 
-
 // Format CodeceptJS step using its built-in methods
 function formatCodeceptStep(step) {
   if (!step) return null;
@@ -469,14 +470,14 @@ function formatCodeceptStep(step) {
   const formattedStep = {
     category,
     title,
-    duration
+    duration,
   };
 
   // Add error if step failed
   if (step.status === 'failed' && step.err) {
     formattedStep.error = {
       message: step.err.message || 'Step failed',
-      stack: step.err.stack || ''
+      stack: step.err.stack || '',
     };
   }
 
@@ -500,10 +501,9 @@ function formatHookStep(step) {
   return {
     category: 'hook',
     title,
-    duration: step.duration || 0
+    duration: step.duration || 0,
   };
 }
 
-
 export { CodeceptReporter };
 export default CodeceptReporter;

package/src/adapter/mocha.js

@@ -84,7 +84,7 @@ function MochaReporter(runner, opts) {
     const artifacts = services.artifacts.get(test.fullTitle());
     const keyValues = services.keyValues.get(test.fullTitle());
     const links = services.links.get(test.fullTitle());
-    
+
     client.addTestRun(STATUS.SKIPPED, {
       title: getTestName(test),
       suite_title: getSuiteTitle(test),

package/src/adapter/playwright.js

@@ -254,7 +254,7 @@ function generateTmpFilepath(filename = '') {
  */
 function extractTags(test) {
   const tagsSet = new Set();
-  
+
   // Extract tags from test title (@tag format)
   const titleTagsMatch = test.title.match(/@\w+/g);
   if (titleTagsMatch) {
@@ -262,7 +262,7 @@ function extractTags(test) {
       tagsSet.add(tag.replace('@', '').toLowerCase());
     });
   }
-  
+
   // Extract tags from test.tags (Playwright built-in tags)
   if (test.tags && Array.isArray(test.tags)) {
     test.tags.forEach(tag => {

package/src/bin/cli.js

@@ -36,7 +36,7 @@ program
   .command('start')
   .description('Start a new run and return its ID')
   .option('--kind <type>', 'Specify run type: automated, manual, or mixed')
-  .action(async (opts) => {
+  .action(async opts => {
     cleanLatestRunId();
 
     console.log('Starting a new Run on Testomat.io...');

package/src/client.js

@@ -1,30 +1,18 @@
 import createDebugMessages from 'debug';
-import createCallsiteRecord from 'callsite-record';
-import { minimatch } from 'minimatch';
 import fs from 'fs';
 import pc from 'picocolors';
-import { randomUUID } from 'crypto';
 import { APP_PREFIX, STATUS } from './constants.js';
 import { pipesFactory } from './pipe/index.js';
 import { glob } from 'glob';
-import path, { sep } from 'path';
+import path from 'path';
 import { fileURLToPath } from 'node:url';
 import { S3Uploader } from './uploader.js';
-import {
-  formatStep,
-  truncate,
-  readLatestRunId,
-  storeRunId,
-  validateSuiteId,
-  transformEnvVarToBoolean
-} from './utils/utils.js';
+import { readLatestRunId, storeRunId, validateSuiteId, transformEnvVarToBoolean } from './utils/utils.js';
 import { filesize as prettyBytes } from 'filesize';
-import { stripVTControlCharacters } from 'util';
+import { formatLogs, formatError, stripColors } from './utils/log-formatter.js';
 
 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)
@@ -163,19 +151,12 @@ class Client {
     /**
      * @type {TestData}
      */
-    const {
-      rid,
-      error = null,
-      steps: originalSteps,
-      title,
-      suite_title,
-    } = testData;
+    const { rid, error = null, steps: originalSteps, title, suite_title } = testData;
     let steps = originalSteps;
 
     const uploadedFiles = [];
     const stackArtifactsEnabled = transformEnvVarToBoolean(process.env.TESTOMATIO_STACK_ARTIFACTS);
 
-
     const {
       time = 0,
       example = null,
@@ -204,24 +185,24 @@ class Client {
 
     let errorFormatted = '';
     if (error) {
-      errorFormatted += this.formatError(error) || '';
+      errorFormatted += formatError(error) || '';
       message = error?.message;
     }
 
-    let fullLogs = this.formatLogs({ error: errorFormatted, steps, logs: testData.logs });
+    let fullLogs = formatLogs({ error: errorFormatted, steps, logs: testData.logs });
 
     if (stackArtifactsEnabled && fullLogs?.trim()?.length > 0) {
       uploadedFiles.push(
-        this.uploader.uploadFileAsBuffer(
-          Buffer.from(stripColors(fullLogs), 'utf8'),
-          [this.runId, rid, `logs_${+new Date}.log`]
-        )
+        this.uploader.uploadFileAsBuffer(Buffer.from(stripColors(fullLogs), 'utf8'), [
+          this.runId,
+          rid,
+          `logs_${+new Date()}.log`,
+        ]),
       );
       fullLogs = '';
       steps = null;
     }
 
-
     if (!this.pipes || !this.pipes.length)
       this.pipes = await pipesFactory(this.paramsForPipesFactory || {}, this.pipeStore);
 
@@ -232,7 +213,7 @@ class Client {
       return [];
     }
 
-    if (isTestShouldBeExculedFromReport(testData)) return [];
+    if (isTestShouldBeExcludedFromReport(testData)) return [];
 
     if (status === STATUS.SKIPPED && process.env.TESTOMATIO_EXCLUDE_SKIPPED) {
       debug('Skipping test from report', testData?.title);
@@ -408,84 +389,6 @@ class Client {
 
     return this.queue;
   }
-
-  /**
-   * Returns the formatted stack including the stack trace, steps, and logs.
-   * @returns {string}
-   */
-  formatLogs({ error, steps, logs }) {
-    error = error?.trim();
-    logs = logs?.trim().split('\n').map(l => truncate(l)).join('\n');
-
-    if (Array.isArray(steps)) {
-      steps = steps
-        .map(step => formatStep(step))
-        .flat()
-        .join('\n');
-    }
-
-    let testLogs = '';
-    if (steps) testLogs += `${pc.bold(pc.blue('################[ Steps ]################'))}\n${steps}\n\n`;
-    if (logs) testLogs += `${pc.bold(pc.gray('################[ Logs ]################'))}\n${logs}\n\n`;
-    if (error) testLogs += `${pc.bold(pc.red('################[ Failure ]################'))}\n${error}`;
-    return testLogs;
-  }
-
-  formatError(error, message) {
-    if (!message) message = error.message;
-    if (error.inspect) message = error.inspect() || '';
-
-    let stack = '';
-    if (error.name) stack += `${pc.red(error.name)}`;
-    if (error.operator) stack += ` (${pc.red(error.operator)})`;
-    // add new line if something was added to stack
-    if (stack) stack += ': ';
-
-    stack += `${message}\n`;
-
-    if (error.diff) {
-      // diff for vitest
-      stack += error.diff;
-      stack += '\n\n';
-    } else if (error.actual && error.expected && error.actual !== error.expected) {
-      // diffs for mocha, cypress, codeceptjs style
-      stack += `\n\n${pc.bold(pc.green('+ expected'))} ${pc.bold(pc.red('- actual'))}`;
-      stack += `\n${pc.green(`+ ${error.expected.toString().split('\n').join('\n+ ')}`)}`;
-      stack += `\n${pc.red(`- ${error.actual.toString().split('\n').join('\n- ')}`)}`;
-      stack += '\n\n';
-    }
-
-    const customFilter = process.env.TESTOMATIO_STACK_IGNORE;
-
-    try {
-      let hasFrame = false;
-      const record = createCallsiteRecord({
-        forError: error,
-        isCallsiteFrame: frame => {
-          if (customFilter && minimatch(frame.fileName, customFilter)) return false;
-          if (hasFrame) return false;
-          if (isNotInternalFrame(frame)) hasFrame = true;
-          return hasFrame;
-        },
-      });
-      // @ts-ignore
-      if (record && !record.filename.startsWith('http')) {
-        stack += record.renderSync({ stackFilter: isNotInternalFrame });
-      }
-      return stack;
-    } catch (e) {
-      console.log(e);
-    }
-  }
-}
-
-function isNotInternalFrame(frame) {
-  return (
-    frame.getFileName() &&
-    frame.getFileName().includes(sep) &&
-    !frame.getFileName().includes('node_modules') &&
-    !frame.getFileName().includes('internal')
-  );
 }
 
 /**
@@ -493,7 +396,7 @@ function isNotInternalFrame(frame) {
  * @param {TestData} testData
  * @returns boolean
  */
-function isTestShouldBeExculedFromReport(testData) {
+function isTestShouldBeExcludedFromReport(testData) {
   // const fileName = path.basename(test.location?.file || '');
   const globExcludeFilesPattern = process.env.TESTOMATIO_EXCLUDE_FILES_FROM_REPORT_GLOB_PATTERN;
   if (!globExcludeFilesPattern) return false;
@@ -503,12 +406,12 @@ function isTestShouldBeExculedFromReport(testData) {
     return false;
   }
 
-  const excludeParretnsList = globExcludeFilesPattern.split(';');
+  const excludePatternsList = globExcludeFilesPattern.split(';');
 
   // as scanning files is time consuming operation, just save the result in variable to avoid multiple scans
   if (!listOfTestFilesToExcludeFromReport) {
     // list of files with relative paths
-    listOfTestFilesToExcludeFromReport = glob.sync(excludeParretnsList, { ignore: '**/node_modules/**' });
+    listOfTestFilesToExcludeFromReport = glob.sync(excludePatternsList, { ignore: '**/node_modules/**' });
     debug('Tests from next files will not be reported:', listOfTestFilesToExcludeFromReport);
   }
 

package/src/junit-adapter/csharp.js

@@ -8,7 +8,10 @@ class CSharpAdapter extends Adapter {
       const exampleMatch = t.title.match(/\((.*?)\)/);
       if (exampleMatch) {
         // Extract parameters as object with numeric keys for API
-        const params = exampleMatch[1].split(',').map(param => param.trim()).filter(param => param !== '');
+        const params = exampleMatch[1]
+          .split(',')
+          .map(param => param.trim())
+          .filter(param => param !== '');
         t.example = {};
         params.forEach((param, index) => {
           t.example[index] = param;

package/src/junit-adapter/nunit-parser.js

@@ -393,13 +393,15 @@ export class NUnitXmlParser {
     }
 
     // Clean up parameters - remove quotes if they wrap the entire parameter and filter empty ones
-    return parameters.map(param => {
-      param = param.trim();
-      if ((param.startsWith('"') && param.endsWith('"')) || (param.startsWith("'") && param.endsWith("'"))) {
-        return param.slice(1, -1);
-      }
-      return param;
-    }).filter(p => !!p);
+    return parameters
+      .map(param => {
+        param = param.trim();
+        if ((param.startsWith('"') && param.endsWith('"')) || (param.startsWith("'") && param.endsWith("'"))) {
+          return param.slice(1, -1);
+        }
+        return param;
+      })
+      .filter(p => !!p);
   }
 
   /**

package/src/pipe/bitbucket.js

@@ -44,8 +44,8 @@ export class BitbucketPipe {
       baseURL: 'https://api.bitbucket.org/2.0',
       headers: {
         'Content-Type': 'application/json',
-        'Authorization': `Bearer ${this.token}`
-      }
+        Authorization: `Bearer ${this.token}`,
+      },
     });
 
     debug('Bitbucket Pipe: Enabled');
@@ -186,7 +186,7 @@ export class BitbucketPipe {
       const addCommentResponse = await this.client.request({
         method: 'POST',
         url: commentsRequestURL,
-        data: { content: { raw: body } }
+        data: { content: { raw: body } },
       });
 
       const commentID = addCommentResponse.data.id;
@@ -221,7 +221,7 @@ async function deletePreviousReport(client, commentsRequestURL, hiddenCommentDat
   try {
     const response = await client.request({
       method: 'GET',
-      url: commentsRequestURL
+      url: commentsRequestURL,
     });
     comments = response.data.values;
   } catch (e) {
@@ -238,7 +238,7 @@ async function deletePreviousReport(client, commentsRequestURL, hiddenCommentDat
         const deleteCommentURL = `${commentsRequestURL}/${comment.id}`;
         await client.request({
           method: 'DELETE',
-          url: deleteCommentURL
+          url: deleteCommentURL,
         });
       } catch (e) {
         console.warn(`Can't delete previously added comment with testomat.io report. Ignored.`);

package/src/pipe/debug.js

@@ -93,8 +93,7 @@ export class DebugPipe {
       const logData = { action: 'addTest', testId: data };
       if (this.store.runId) logData.runId = this.store.runId;
       this.logToFile(logData);
-    }
-    else this.batch.tests.push(data);
+    } else this.batch.tests.push(data);
 
     if (!this.batch.intervalFunction) await this.batchUpload();
   }

package/src/pipe/gitlab.js

@@ -49,7 +49,7 @@ class GitLabPipe {
       baseURL: 'https://gitlab.com/api/v4',
       headers: {
         'Content-Type': 'application/json',
-      }
+      },
     });
 
     debug('GitLab Pipe: Enabled');
@@ -176,7 +176,7 @@ class GitLabPipe {
         method: 'POST',
         url: commentsRequestURL,
         params: { access_token: this.token },
-        data: { body }
+        data: { body },
       });
 
       const commentID = addCommentResponse.data.id;
@@ -212,7 +212,7 @@ async function deletePreviousReport(client, commentsRequestURL, hiddenCommentDat
     const response = await client.request({
       method: 'GET',
       url: commentsRequestURL,
-      params: { access_token: token }
+      params: { access_token: token },
     });
     comments = response.data;
   } catch (e) {
@@ -230,7 +230,7 @@ async function deletePreviousReport(client, commentsRequestURL, hiddenCommentDat
         await client.request({
           method: 'DELETE',
           url: deleteCommentURL,
-          params: { access_token: token }
+          params: { access_token: token },
         });
       } catch (e) {
         console.warn(`Can't delete previously added comment with testomat.io report. Ignore.`);