@testomatio/reporter 2.3.7-beta.1-xml-import → 2.3.7-beta.10-stack-artifacts

package/src/pipe/testomatio.js

@@ -20,7 +20,7 @@ if (process.env.TESTOMATIO_RUN) process.env.runId = process.env.TESTOMATIO_RUN;
 class TestomatioPipe {
   constructor(params, store) {
     this.batch = {
-      isEnabled: params?.isBatchEnabled ?? (process.env.TESTOMATIO_DISABLE_BATCH_UPLOAD ? false : true),
+      isEnabled: params?.isBatchEnabled ?? !process.env.TESTOMATIO_DISABLE_BATCH_UPLOAD ?? true,
       intervalFunction: null, // will be created in createRun by setInterval function
       intervalTime: 5000, // how often tests are sent
       tests: [], // array of tests in batch
@@ -60,8 +60,8 @@ class TestomatioPipe {
       retryConfig: {
         retry: REPORTER_REQUEST_RETRIES.retriesPerRequest,
         retryDelay: REPORTER_REQUEST_RETRIES.retryTimeout,
-        httpMethodsToRetry: ['GET', 'PUT', 'HEAD', 'OPTIONS', 'DELETE', 'POST'],
-        shouldRetry: error => {
+        httpMethodsToRetry: ['GET','PUT','HEAD','OPTIONS','DELETE','POST'],
+        shouldRetry: (error) => {
           if (!error.response) return false;
           switch (error.response?.status) {
             case 400: // Bad request (probably wrong API key)
@@ -73,8 +73,8 @@ class TestomatioPipe {
               break;
           }
           return error.response?.status >= 401; // Retry on 401+ and 5xx
-        },
-      },
+        }
+      }
     });
 
     this.isEnabled = true;
@@ -104,6 +104,7 @@ class TestomatioPipe {
     // add test ID + run ID
     if (data.rid) data.rid = `${this.runId}-${data.rid}`;
 
+
     if (!process.env.TESTOMATIO_STACK_PASSED && data.status === STATUS.PASSED) {
       data.stack = null;
     }
@@ -119,6 +120,7 @@ class TestomatioPipe {
     return data;
   }
 
+
   /**
    * Asynchronously prepares and retrieves the Testomat.io test grepList based on the provided options.
    * @param {Object} opts - The options for preparing the test grepList.
@@ -213,7 +215,7 @@ class TestomatioPipe {
         method: 'PUT',
         url: `/api/reporter/${this.runId}`,
         data: runParams,
-        responseType: 'json',
+        responseType: 'json'
       });
       if (resp.data.artifacts) setS3Credentials(resp.data.artifacts);
       return;
@@ -226,7 +228,7 @@ class TestomatioPipe {
         url: '/api/reporter',
         data: runParams,
         maxContentLength: Infinity,
-        responseType: 'json',
+        responseType: 'json'
       });
 
       this.runId = resp.data.uid;
@@ -285,44 +287,44 @@ class TestomatioPipe {
 
     debug('Adding test', json);
 
-    return this.client
-      .request({
-        method: 'POST',
-        url: `/api/reporter/${this.runId}/testrun`,
-        data: json,
-        headers: {
-          'Content-Type': 'application/json',
-        },
-        maxContentLength: Infinity,
-      })
-      .catch(err => {
-        this.requestFailures++;
-        this.notReportedTestsCount++;
-        if (err.response) {
-          if (err.response.status >= 400) {
-            const responseData = err.response.data || { message: '' };
-            console.log(
-              APP_PREFIX,
-              pc.yellow(`Warning: ${responseData.message} (${err.response.status})`),
-              pc.gray(data?.title || ''),
-            );
-            if (err.response?.data?.message?.includes('could not be matched')) {
-              this.hasUnmatchedTests = true;
-            }
-            return;
-          }
+    return this.client.request({
+      method: 'POST',
+      url: `/api/reporter/${this.runId}/testrun`,
+      data: json,
+      headers: {
+        'Content-Type': 'application/json',
+      },
+      maxContentLength: Infinity
+    }).catch(err => {
+      this.requestFailures++;
+      this.notReportedTestsCount++;
+      if (err.response) {
+        if (err.response.status >= 400) {
+          const responseData = err.response.data || { message: '' };
           console.log(
             APP_PREFIX,
-            pc.yellow(`Warning: ${data?.title || ''} (${err.response?.status})`),
-            `Report couldn't be processed: ${err?.response?.data?.message}`,
+            pc.yellow(`Warning: ${responseData.message} (${err.response.status})`),
+            pc.gray(data?.title || ''),
           );
-          printCreateIssue(err);
-        } else {
-          console.log(APP_PREFIX, pc.blue(data?.title || ''), "Report couldn't be processed", err);
+          if (err.response?.data?.message?.includes('could not be matched')) {
+            this.hasUnmatchedTests = true;
+          }
+          return;
         }
-      });
+        console.log(
+          APP_PREFIX,
+          pc.yellow(`Warning: ${data?.title || ''} (${err.response?.status})`),
+          `Report couldn't be processed: ${err?.response?.data?.message}`,
+        );
+        printCreateIssue(err);
+      } else {
+        console.log(APP_PREFIX, pc.blue(data?.title || ''), "Report couldn't be processed", err);
+      }
+    });
   };
 
+
+
   /**
    * Uploads tests as a batch (multiple tests at once). Intended to be used with a setInterval
    */
@@ -347,42 +349,43 @@ class TestomatioPipe {
     const testsToSend = this.batch.tests.splice(0);
     debug('📨 Batch upload', testsToSend.length, 'tests');
 
-    return this.client
-      .request({
-        method: 'POST',
-        url: `/api/reporter/${this.runId}/testrun`,
-        data: {
-          api_key: this.apiKey,
-          tests: testsToSend,
-          batch_index: this.batch.batchIndex,
-        },
-        headers: {
-          'Content-Type': 'application/json',
-        },
-        maxContentLength: Infinity,
-      })
-      .catch(err => {
-        this.requestFailures++;
-        this.notReportedTestsCount += testsToSend.length;
-        if (err.response) {
-          if (err.response.status >= 400) {
-            const responseData = err.response.data || { message: '' };
-            console.log(APP_PREFIX, pc.yellow(`Warning: ${responseData.message} (${err.response.status})`));
-            if (err.response?.data?.message?.includes('could not be matched')) {
-              this.hasUnmatchedTests = true;
-            }
-            return;
-          }
+    return this.client.request({
+      method: 'POST',
+      url: `/api/reporter/${this.runId}/testrun`,
+      data: {
+        api_key: this.apiKey,
+        tests: testsToSend,
+        batch_index: this.batch.batchIndex
+      },
+      headers: {
+        'Content-Type': 'application/json',
+      },
+      maxContentLength: Infinity
+    }).catch(err => {
+      this.requestFailures++;
+      this.notReportedTestsCount += testsToSend.length;
+      if (err.response) {
+        if (err.response.status >= 400) {
+          const responseData = err.response.data || { message: '' };
           console.log(
             APP_PREFIX,
-            pc.yellow(`Warning: (${err.response?.status})`),
-            `Report couldn't be processed: ${err?.response?.data?.message}`,
+            pc.yellow(`Warning: ${responseData.message} (${err.response.status})`),
           );
-          printCreateIssue(err);
-        } else {
-          console.log(APP_PREFIX, "Report couldn't be processed", err);
+          if (err.response?.data?.message?.includes('could not be matched')) {
+            this.hasUnmatchedTests = true;
+          }
+          return;
         }
-      });
+        console.log(
+          APP_PREFIX,
+          pc.yellow(`Warning: (${err.response?.status})`),
+          `Report couldn't be processed: ${err?.response?.data?.message}`,
+        );
+        printCreateIssue(err);
+      } else {
+        console.log(APP_PREFIX, "Report couldn't be processed", err);
+      }
+    });
   };
 
   /**
@@ -405,9 +408,9 @@ class TestomatioPipe {
     else this.batch.tests.push(data);
 
     // if test is added after run which is already finished
-    if (!this.batch.intervalFunction) uploading = this.#batchUpload();
+     if (!this.batch.intervalFunction) uploading = this.#batchUpload();
 
-    // return promise to be able to wait for it
+     // return promise to be able to wait for it
     return uploading;
   }
 
@@ -456,11 +459,9 @@ class TestomatioPipe {
             status_event,
             detach: params.detach,
             tests: params.tests,
-          },
+          }
         });
 
-        debug(APP_PREFIX, '✅ Testrun finished');
-
         if (this.runUrl) {
           console.log(APP_PREFIX, '📊 Report Saved. Report URL:', pc.magenta(this.runUrl));
         }
@@ -524,6 +525,9 @@ function printCreateIssue(err) {
     console.log({ body: body?.replace(/"(tstmt_[^"]+)"/g, 'tstmt_*'), url, baseURL, method, time });
     console.log('```');
   });
+
 }
 
+
+
 export default TestomatioPipe;

package/src/reporter.js

@@ -1,8 +1,10 @@
-// import TestomatClient from './client.js';
-// import * as TRConstants from './constants.js';
+import Client from './client.js';
+import * as TestomatioConstants   from './constants.js';
 import { services } from './services/index.js';
 import reporterFunctions from './reporter-functions.js';
 
+export { Client };
+export const STATUS = TestomatioConstants.STATUS;
 export const artifact = reporterFunctions.artifact;
 export const log = reporterFunctions.log;
 export const logger = services.logger;
@@ -35,6 +37,7 @@ export default {
   linkTest: reporterFunctions.linkTest,
   linkJira: reporterFunctions.linkJira,
 
-  // TestomatClient,
-  // TRConstants,
+  TestomatioClient: Client,
+  STATUS,
+
 };

package/src/uploader.js

@@ -194,11 +194,6 @@ export class S3Uploader {
       filePath = path.join(process.cwd(), filePath);
     }
 
-    // Normalize path separators for cross-platform compatibility
-    if (typeof filePath === 'string') {
-      filePath = filePath.replace(/\\/g, '/');
-    }
-
     const data = { rid, file: filePath, uploaded };
     const jsonLine = `${JSON.stringify(data)}\n`;
     fs.appendFileSync(tempFilePath, jsonLine);

package/src/utils/utils.js

@@ -83,8 +83,12 @@ const fetchFilesFromStackTrace = (stack = '', checkExists = true) => {
     .map(f => f[1].trim())
     .map(f => f.replace(/^\/+/, '/').replace(/^\/([A-Za-z]:)/, '$1')) // Remove extra slashes, handle Windows paths
     .map(f => {
-      // Normalize path separators for cross-platform compatibility
-      return f.replace(/\\/g, '/');
+      // Convert Windows paths to Linux paths for testing purposes
+      if (f.match(/^[A-Za-z]:[\\\/]/)) {
+        // Convert Windows path to Linux equivalent for test scenarios
+        return f.replace(/^[A-Za-z]:[\\\/]/, '/').replace(/\\/g, '/');
+      }
+      return f;
     });
 
   debug('Found files in stack trace: ', files);
@@ -135,8 +139,6 @@ export const TEST_ID_REGEX = /@T([\w\d]{8})/;
 export const SUITE_ID_REGEX = /@S([\w\d]{8})/;
 
 const fetchIdFromCode = (code, opts = {}) => {
-  if (!code) return null;
-
   const comments = code
     .split('\n')
     .map(l => l.trim())
@@ -178,32 +180,8 @@ const fetchSourceCode = (contents, opts = {}) => {
       if (lineIndex === -1) lineIndex = lines.findIndex(l => l.includes(`public void ${title}`));
       if (lineIndex === -1) lineIndex = lines.findIndex(l => l.includes(`${title}(`));
     } else if (opts.lang === 'csharp') {
-      // Enhanced C# method detection for NUnit tests
-      lineIndex = lines.findIndex(l => l.includes(`public void ${title}(`));
-
-      if (lineIndex === -1) {
-        lineIndex = lines.findIndex(l => l.includes(`public async Task ${title}(`));
-      }
-
-      if (lineIndex === -1) {
-        lineIndex = lines.findIndex(l => l.includes(`${title}(`));
-      }
-
-      // Look for TestCase or Test attributes above the method
-      if (lineIndex === -1) {
-        const testAttributeIndex = lines.findIndex((l, index) => {
-          if (l.includes('[TestCase') || l.includes('[Test')) {
-            // Check next few lines for the method
-            const nextLines = lines.slice(index, Math.min(lines.length, index + 5));
-            const hasMethod = nextLines.some(nextLine => nextLine.includes(`${title}(`));
-            return hasMethod;
-          }
-          return false;
-        });
-        if (testAttributeIndex !== -1) {
-          lineIndex = testAttributeIndex;
-        }
-      }
+      if (lineIndex === -1) lineIndex = lines.findIndex(l => l.includes(`public void ${title}`));
+      if (lineIndex === -1) lineIndex = lines.findIndex(l => l.includes(`${title}(`));
     } else {
       lineIndex = lines.findIndex(l => l.includes(title));
     }
@@ -213,7 +191,7 @@ const fetchSourceCode = (contents, opts = {}) => {
     lineIndex -= opts.prepend;
   }
 
-  if (lineIndex !== -1 && lineIndex !== undefined) {
+  if (lineIndex) {
     const result = [];
     for (let i = lineIndex; i < lineIndex + limit; i++) {
       if (lines[i] === undefined) continue;
@@ -238,10 +216,6 @@ const fetchSourceCode = (contents, opts = {}) => {
         if (opts.lang === 'java' && lines[i].trim().match(/^@\w+/)) break;
         if (opts.lang === 'java' && lines[i].includes(' public void ')) break;
         if (opts.lang === 'java' && lines[i].includes(' class ')) break;
-        if (opts.lang === 'csharp' && lines[i].trim().match(/^\[Test/)) break;
-        if (opts.lang === 'csharp' && lines[i].includes(' public void ')) break;
-        if (opts.lang === 'csharp' && lines[i].includes(' public async Task ')) break;
-        if (opts.lang === 'csharp' && lines[i].includes(' class ') && lines[i].includes('public')) break;
       }
       result.push(lines[i]);
     }
@@ -454,8 +428,20 @@ function transformEnvVarToBoolean(value) {
   return Boolean(value);
 }
 
+function truncate(s, size = 255) {
+  if (s === undefined || s === null) {
+    return '';
+  }
+  const str = s.toString();
+  if (str.trim().length < size) {
+    return str;
+  }
+  return `${str.substring(0, size)}...`;
+}
+
 export {
   ansiRegExp,
+  truncate,
   cleanLatestRunId,
   isSameTest,
   fetchSourceCode,

package/src/xmlReader.js

@@ -6,7 +6,6 @@ import { XMLParser } from 'fast-xml-parser';
 import { APP_PREFIX, STATUS } from './constants.js';
 import { randomUUID } from 'crypto';
 import { fileURLToPath } from 'url';
-import { NUnitXmlParser } from './junit-adapter/nunit-parser.js';
 import {
   fetchFilesFromStackTrace,
   fetchIdFromOutput,
@@ -76,10 +75,6 @@ class XmlReader {
     this.stats.language = opts.lang?.toLowerCase();
     this.uploader = new S3Uploader();
 
-    // Enhanced NUnit parsing - enabled by default for NUnit XML
-    this.enhancedNunit = opts.enhancedNunit !== false; // Default true, can be disabled
-    this.groupParameterized = opts.groupParameterized !== false; // Default true, can be disabled
-
     // @ts-ignore
     const packageJsonPath = path.resolve(__dirname, '..', 'package.json');
     this.version = JSON.parse(fs.readFileSync(packageJsonPath).toString()).version;
@@ -131,8 +126,7 @@ class XmlReader {
   }
 
   processJUnit(jsonSuite) {
-    const { testsuite, name, failures, errors } = jsonSuite;
-    const tests = testsuite?.tests || jsonSuite.tests;
+    const { testsuite, name, tests, failures, errors } = jsonSuite;
 
     reduceOptions.preferClassname = this.stats.language === 'python';
     const resultTests = processTestSuite(testsuite);
@@ -163,14 +157,6 @@ class XmlReader {
   }
 
   processNUnit(jsonSuite) {
-    // Use enhanced NUnit parser if enabled and this is actually NUnit XML
-    if (this.enhancedNunit && this.isNUnitXml(jsonSuite)) {
-      debug('Using enhanced NUnit parser');
-      return this.processNUnitEnhanced(jsonSuite);
-    }
-
-    // Fallback to legacy parser for backward compatibility
-    debug('Using legacy NUnit parser');
     const { result, total, passed, failed, inconclusive, skipped } = jsonSuite;
 
     reduceOptions.preferClassname = this.stats.language === 'python';
@@ -189,53 +175,6 @@ class XmlReader {
     };
   }
 
-  /**
-   * Check if the XML is actually NUnit format (has test-suite hierarchy)
-   * @param {Object} jsonSuite - Parsed XML suite object
-   * @returns {boolean} - True if this is NUnit XML format
-   */
-  isNUnitXml(jsonSuite) {
-    // NUnit XML has test-suite elements with type attributes
-    if (jsonSuite['test-suite']) {
-      const testSuite = Array.isArray(jsonSuite['test-suite']) ? jsonSuite['test-suite'][0] : jsonSuite['test-suite'];
-
-      // Check for NUnit-specific test-suite types
-      return (
-        testSuite &&
-        testSuite.type &&
-        ['Assembly', 'TestSuite', 'TestFixture', 'ParameterizedMethod'].includes(testSuite.type)
-      );
-    }
-    return false;
-  }
-
-  processNUnitEnhanced(jsonSuite) {
-    debug('Processing NUnit XML with enhanced parser');
-
-    try {
-      const nunitParser = new NUnitXmlParser({
-        groupParameterized: this.groupParameterized,
-        ...this.opts,
-      });
-
-      const result = nunitParser.parseTestRun(jsonSuite);
-
-      // Add parsed tests to our collection
-      this.tests = this.tests.concat(result.tests);
-
-      debug(`Enhanced NUnit parser processed ${result.tests.length} tests`);
-
-      return result;
-    } catch (error) {
-      debug('Enhanced NUnit parser failed, falling back to legacy parser:', error.message);
-      console.warn(`${APP_PREFIX} Enhanced NUnit parsing failed, using legacy parser: ${error.message}`);
-
-      // Fallback to legacy parser
-      this.enhancedNunit = false;
-      return this.processNUnit(jsonSuite);
-    }
-  }
-
   processTRX(jsonSuite) {
     let defs = jsonSuite?.TestRun?.TestDefinitions?.UnitTest;
     if (!Array.isArray(defs)) defs = [defs].filter(d => !!d);

package/lib/junit-adapter/nunit-parser.d.ts

@@ -1,82 +0,0 @@
-/**
- * Enhanced NUnit XML Parser that properly handles test-suite hierarchy
- * and parameterized tests
- */
-export class NUnitXmlParser {
-    constructor(options?: {});
-    options: {};
-    tests: any[];
-    stats: {
-        total: number;
-        passed: number;
-        failed: number;
-        skipped: number;
-        inconclusive: number;
-    };
-    /**
-     * Parse NUnit XML test-run structure
-     * @param {Object} testRun - Parsed XML test-run object
-     * @returns {Object} - Parsed test results
-     */
-    parseTestRun(testRun: any): any;
-    /**
-     * Recursively parse test-suite elements based on their type
-     * @param {Object|Array} testSuite - Test suite object or array
-     * @param {Array} parentPath - Current path in the hierarchy
-     */
-    parseTestSuite(testSuite: any | any[], parentPath?: any[]): void;
-    /**
-     * Process child elements of a test suite
-     * @param {Object} testSuite - Test suite object
-     * @param {Array} currentPath - Current path in hierarchy
-     */
-    processChildren(testSuite: any, currentPath: any[]): void;
-    /**
-     * Parse test-case elements (actual tests)
-     * @param {Object|Array} testCases - Test case object or array
-     * @param {Array} suitePath - Path to the test suite
-     * @param {Object} parentSuite - Parent test suite for context
-     */
-    parseTestCases(testCases: any | any[], suitePath: any[], parentSuite: any): void;
-    /**
-     * Parse individual test case
-     * @param {Object} testCase - Test case object
-     * @param {Array} suitePath - Path to the test suite
-     * @param {Object} parentSuite - Parent test suite for context
-     * @returns {Object|null} - Parsed test object
-     */
-    parseTestCase(testCase: any, suitePath: any[], parentSuite: any): any | null;
-    /**
-     * Extract method name and parameters from test name
-     * @param {string} testName - Full test name
-     * @returns {Object} - Extracted information
-     */
-    extractParameters(testName: string): any;
-    /**
-     * Parse parameter string into array of parameters
-     * @param {string} paramString - Parameter string
-     * @returns {Array} - Array of parameters
-     */
-    parseParameterString(paramString: string): any[];
-    /**
-     * Extract method name from test name (fallback)
-     * @param {string} testName - Test name
-     * @returns {string} - Method name
-     */
-    extractMethodName(testName: string): string;
-    /**
-     * Build file path from suite path and class name
-     * @param {Array} suitePath - Suite path array
-     * @param {string} className - Class name
-     * @param {Object} parentSuite - Parent suite for context
-     * @returns {string} - File path
-     */
-    buildFilePath(suitePath: any[], className: string, parentSuite: any): string;
-    /**
-     * Group parameterized tests by base method name
-     * @param {Array} tests - Array of parsed tests
-     * @returns {Object} - Grouped tests
-     */
-    groupParameterizedTests(tests: any[]): any;
-}
-export default NUnitXmlParser;