@testomatio/reporter 2.3.7-beta.3-xml-import → 2.3.7-beta.5-stack-artifacts

package/lib/xmlReader.js

@@ -11,7 +11,6 @@ const fast_xml_parser_1 = require("fast-xml-parser");
 const constants_js_1 = require("./constants.js");
 const crypto_1 = require("crypto");
 const url_1 = require("url");
-const nunit_parser_js_1 = require("./junit-adapter/nunit-parser.js");
 const utils_js_1 = require("./utils/utils.js");
 const index_js_1 = require("./pipe/index.js");
 const index_js_2 = __importDefault(require("./junit-adapter/index.js"));
@@ -55,9 +54,6 @@ class XmlReader {
         this.stats = {};
         this.stats.language = opts.lang?.toLowerCase();
         this.uploader = new uploader_js_1.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_1.default.resolve(__dirname, '..', 'package.json');
         this.version = JSON.parse(fs_1.default.readFileSync(packageJsonPath).toString()).version;
@@ -106,8 +102,7 @@ class XmlReader {
         return this.processJUnit(jsonSuite);
     }
     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);
         const hasFailures = resultTests.filter(t => t.status === 'failed').length > 0;
@@ -133,13 +128,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';
         const resultTests = processTestSuite(jsonSuite['test-suite']);
@@ -154,58 +142,65 @@ class XmlReader {
             tests: resultTests,
         };
     }
-    /**
-     * 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 nunit_parser_js_1.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(`${constants_js_1.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);
-        // Parse test definitions
-        const tests = defs.map(td => this._parseTRXTestDefinition(td));
-        // Parse test results
+        const tests = defs.map(td => {
+            const title = td.name.replace(/\(.*?\)/, '').trim();
+            let example = td.name.match(/\((.*?)\)/);
+            if (example)
+                example = { ...example[1].split(',') };
+            const suite = td.TestMethod.className.split(', ')[0].split('.');
+            const suite_title = suite.pop();
+            return {
+                title,
+                example,
+                file: suite.join('/'),
+                description: td.Description,
+                suite_title,
+                id: td.Execution.id,
+            };
+        }) || [];
         let result = jsonSuite?.TestRun?.Results?.UnitTestResult;
         if (!Array.isArray(result))
             result = [result].filter(d => !!d);
-        const results = result.map(td => this._parseTRXTestResult(td, tests));
+        const results = result.map(td => ({
+            id: td.executionId,
+            // seconds are used in junit reports, but ms are used by testomatio
+            run_time: parseFloat(td.duration) * 1000,
+            status: td.outcome,
+            stack: td.Output.StdOut,
+            files: td?.ResultFiles?.ResultFile?.map(rf => rf.path),
+        }));
+        results.forEach(r => {
+            const test = tests.find(t => t.id === r.id) || {};
+            r.suite_title = test.suite_title;
+            r.title = test.title?.trim();
+            if (test.code)
+                r.code = test.code;
+            if (test.description)
+                r.description = test.description;
+            if (test.example)
+                r.example = test.example;
+            if (test.file)
+                r.file = test.file;
+            r.create = true;
+            r.overwrite = true;
+            if (r.status === 'Passed')
+                r.status = constants_js_1.STATUS.PASSED;
+            if (r.status === 'Failed')
+                r.status = constants_js_1.STATUS.FAILED;
+            if (r.status === 'Skipped')
+                r.status = constants_js_1.STATUS.SKIPPED;
+            delete r.id;
+        });
         debug(results);
         const counters = jsonSuite?.TestRun?.ResultSummary?.Counters || {};
         const failed_count = parseInt(counters.failed, 10) + parseInt(counters.error, 10);
-        const status = failed_count > 0 ? constants_js_1.STATUS.FAILED : constants_js_1.STATUS.PASSED.toString();
+        let status = constants_js_1.STATUS.PASSED.toString();
+        if (failed_count > 0)
+            status = constants_js_1.STATUS.FAILED;
         this.tests = results.filter(t => !!t.title);
         return {
             status,
@@ -217,57 +212,6 @@ class XmlReader {
             tests: results,
         };
     }
-    _parseTRXTestDefinition(td) {
-        const title = td.name.replace(/\(.*?\)/, '').trim();
-        const exampleMatch = td.name.match(/\((.*?)\)/);
-        const example = exampleMatch ? { ...exampleMatch[1].split(',') } : null;
-        const suite = td.TestMethod.className.split(', ')[0].split('.');
-        const suite_title = suite.pop();
-        // Convert namespace to file path for C#
-        const file = `${suite.join('/')}.cs`;
-        return {
-            title, // Base name without parameters for test import
-            example, // Parameters object for parameterized tests
-            file, // File path with .cs extension
-            description: td.Description,
-            suite_title,
-            id: td.Execution.id,
-        };
-    }
-    _parseTRXTestResult(td, tests) {
-        const test = tests.find(t => t.id === td.executionId) || {};
-        const result = {
-            suite_title: test.suite_title,
-            title: test.title?.trim(),
-            file: test.file,
-            description: test.description,
-            code: test.code,
-            run_time: parseFloat(td.duration) * 1000,
-            stack: td.Output?.StdOut || '',
-            files: td?.ResultFiles?.ResultFile?.map(rf => rf.path),
-            create: true,
-            overwrite: true,
-        };
-        // Add example for parameterized tests
-        if (test.example) {
-            result.example = test.example;
-        }
-        // Map TRX status to Testomat.io status
-        result.status = this._mapTRXStatus(td.outcome);
-        return result;
-    }
-    _mapTRXStatus(outcome) {
-        switch (outcome) {
-            case 'Passed':
-                return constants_js_1.STATUS.PASSED;
-            case 'Failed':
-                return constants_js_1.STATUS.FAILED;
-            case 'Skipped':
-                return constants_js_1.STATUS.SKIPPED;
-            default:
-                return constants_js_1.STATUS.PASSED;
-        }
-    }
     processXUnit(assemblies) {
         const tests = [];
         assemblies = Array.isArray(assemblies.assembly) ? assemblies.assembly : [assemblies.assembly];

package/package.json

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

package/src/bin/cli.js

@@ -158,7 +158,7 @@ program
   .option('--lang <lang>', 'Language used (python, ruby, java)')
   .option('--timelimit <time>', 'default time limit in seconds to kill a stuck process')
   .action(async (pattern, opts) => {
-    if (!pattern.endsWith('.xml') && !pattern.includes('*')) {
+    if (!pattern.endsWith('.xml')) {
       pattern += '.xml';
     }
     let { javaTests, lang } = opts;

package/src/bin/reportXml.js

@@ -23,7 +23,7 @@ program
   .option('--timelimit <time>', 'default time limit in seconds to kill a stuck process')
   .option('--env-file <envfile>', 'Load environment variables from env file')
   .action(async (pattern, opts) => {
-    if (!pattern.endsWith('.xml') && !pattern.includes('*')) {
+    if (!pattern.endsWith('.xml')) {
       pattern += '.xml';
     }
     let { javaTests, lang } = opts;
@@ -34,10 +34,7 @@ program
     }
     lang = lang?.toLowerCase();
     if (javaTests === true || (lang === 'java' && !javaTests)) javaTests = 'src/test/java';
-    const runReader = new XmlReader({
-      javaTests,
-      lang,
-    });
+    const runReader = new XmlReader({ javaTests, lang });
     const files = glob.sync(pattern, { cwd: opts.dir || process.cwd() });
     if (!files.length) {
       console.log(APP_PREFIX, `Report can't be created. No XML files found 😥`);

package/src/client.js

@@ -37,8 +37,9 @@ class Client {
     this.runId = '';
     this.queue = Promise.resolve();
 
-    // Get package.json path - use a simple approach that works in both environments
-    const pathToPackageJSON = path.join(process.cwd(), 'package.json');
+    // @ts-ignore this line will be removed in compiled code, because __dirname is defined in commonjs
+    const __dirname = path.dirname(fileURLToPath(import.meta.url));
+    const pathToPackageJSON = path.join(__dirname, '../package.json');
     try {
       this.version = JSON.parse(fs.readFileSync(pathToPackageJSON).toString()).version;
       console.log(APP_PREFIX, `Testomatio Reporter v${this.version}`);
@@ -386,11 +387,7 @@ class Client {
    */
   formatLogs({ error, steps, logs }) {
     error = error?.trim();
-    logs = logs
-      ?.trim()
-      .split('\n')
-      .map(l => truncate(l))
-      .join('\n');
+    logs = logs?.trim().split('\n').map(l => truncate(l)).join('\n');
 
     if (Array.isArray(steps)) {
       steps = steps

package/src/junit-adapter/csharp.js

@@ -3,50 +3,18 @@ import Adapter from './adapter.js';
 
 class CSharpAdapter extends Adapter {
   formatTest(t) {
-    // Extract example from title if not already present
-    if (!t.example) {
-      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());
-        t.example = {};
-        params.forEach((param, index) => {
-          t.example[index] = param;
-        });
-      }
-    }
-
-    // Remove parameters from title to avoid duplicates in Test Suite
-    // The example field will be used for grouping on import
-    t.title = t.title.replace(/\(.*?\)/, '').trim();
-
+    const title = t.title.replace(/\(.*?\)/, '').trim();
+    const example = t.title.match(/\((.*?)\)/);
+    if (example) t.example = { ...example[1].split(',') };
     const suite = t.suite_title.split('.');
     t.suite_title = suite.pop();
     t.file = namespaceToFileName(t.file);
+    t.title = title.trim();
     return t;
   }
 
   getFilePath(t) {
-    if (!t.file) return null;
-
-    // Normalize path separators for cross-platform compatibility
-    let filePath = t.file.replace(/\\/g, '/');
-
-    // If file already has .cs extension, use it directly
-    if (filePath.endsWith('.cs')) {
-      // Make relative path if it's absolute
-      if (path.isAbsolute(filePath)) {
-        // Try to find project-relative path
-        const cwd = process.cwd().replace(/\\/g, '/');
-        if (filePath.startsWith(cwd)) {
-          filePath = path.relative(cwd, filePath).replace(/\\/g, '/');
-        }
-      }
-      return filePath;
-    }
-
-    // Convert namespace path to file path
-    const fileName = namespaceToFileName(filePath);
+    const fileName = namespaceToFileName(t.file);
     return fileName;
   }
 }
@@ -54,14 +22,7 @@ class CSharpAdapter extends Adapter {
 export default CSharpAdapter;
 
 function namespaceToFileName(fileName) {
-  if (!fileName) return '';
-
-  // If already a .cs file path, clean it up
-  if (fileName.endsWith('.cs')) {
-    return fileName.replace(/\\/g, '/');
-  }
-
   const fileParts = fileName.split('.');
   fileParts[fileParts.length - 1] = fileParts[fileParts.length - 1]?.replace(/\$.*/, '');
-  return `${fileParts.join('/')}.cs`;
+  return `${fileParts.join(path.sep)}.cs`;
 }

package/src/pipe/debug.js

@@ -15,7 +15,7 @@ export class DebugPipe {
     this.isEnabled = !!process.env.TESTOMATIO_DEBUG || !!process.env.DEBUG;
     if (this.isEnabled) {
       this.batch = {
-        isEnabled: this.params.isBatchEnabled ?? (process.env.TESTOMATIO_DISABLE_BATCH_UPLOAD ? false : true),
+        isEnabled: this.params.isBatchEnabled ?? !process.env.TESTOMATIO_DISABLE_BATCH_UPLOAD ?? true,
         intervalFunction: null,
         intervalTime: 5000,
         tests: [],
@@ -93,7 +93,8 @@ 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/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,7 +459,7 @@ class TestomatioPipe {
             status_event,
             detach: params.detach,
             tests: params.tests,
-          },
+          }
         });
 
         if (this.runUrl) {
@@ -522,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/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);