@testomatio/reporter 2.3.5-beta-5-xml-import → 2.3.5-beta-6-xml-import

package/README.md

@@ -13,7 +13,7 @@ Testomat.io Reporter (this npm package) supports:
 - 🔎 [Stack traces](./docs/stacktrace.md) and error messages
 - 🐙 [GitHub](./docs/pipes/github.md), [GitLab](./docs/pipes/gitlab.md) & [Bitbucket](./docs/pipes/bitbucket.md) integration
 - 🚅 Realtime reports
-- 🗃️ Other test frameworks supported via [JUNit XML](./docs/junit.md)
+- 🗃️ Other test frameworks supported via [JUnit XML](./docs/junit.md) with [XML import configuration](./docs/xml-imports.md)
 - 🚶‍♀️ Steps _(work in progress)_
 - 📄 [Logger](./docs/logger.md) _(work in progress, supports Jest for now)_
 - ☁️ Custom properties and metadata _(work in progress)_

package/lib/xmlReader.d.ts

@@ -13,6 +13,8 @@ declare class XmlReader {
     runId: any;
     adapter: import("./junit-adapter/adapter.js").default;
     opts: {};
+    disableSourceCodeFetching: any;
+    suiteOrganization: any;
     store: {};
     pipesPromise: Promise<any[]>;
     parser: XMLParser;

package/lib/xmlReader.js

@@ -20,7 +20,7 @@ const uploader_js_1 = require("./uploader.js");
 const debug = (0, debug_1.default)('@testomatio/reporter:xml');
 const ridRunId = (0, crypto_1.randomUUID)();
 const TESTOMATIO_URL = process.env.TESTOMATIO_URL || 'https://app.testomat.io';
-const { TESTOMATIO_RUNGROUP_TITLE, TESTOMATIO_SUITE, TESTOMATIO_MAX_STACK_TRACE, TESTOMATIO_TITLE, TESTOMATIO_ENV, TESTOMATIO_RUN, TESTOMATIO_MARK_DETACHED, } = process.env;
+const { TESTOMATIO_RUNGROUP_TITLE, TESTOMATIO_SUITE, TESTOMATIO_MAX_STACK_TRACE, TESTOMATIO_TITLE, TESTOMATIO_ENV, TESTOMATIO_RUN, TESTOMATIO_MARK_DETACHED, TESTOMATIO_DISABLE_SOURCE_CODE, } = process.env;
 const options = {
     ignoreDeclaration: true,
     ignoreAttributes: false,
@@ -47,6 +47,10 @@ class XmlReader {
         if (!this.adapter)
             throw new Error('XML adapter for this format not found');
         this.opts = opts || {};
+        // Check if source code fetching should be disabled
+        this.disableSourceCodeFetching = opts.disableSourceCodeFetching || TESTOMATIO_DISABLE_SOURCE_CODE;
+        // Control suite organization strategy: 'classname' (default) or 'fullpath'
+        this.suiteOrganization = opts.suiteOrganization || process.env.TESTOMATIO_SUITE_ORGANIZATION || 'classname';
         this.store = {};
         this.pipesPromise = (0, index_js_1.pipesFactory)(opts, this.store);
         this.parser = new fast_xml_parser_1.XMLParser(options);
@@ -58,6 +62,15 @@ class XmlReader {
         const packageJsonPath = path_1.default.resolve(__dirname, '..', 'package.json');
         this.version = JSON.parse(fs_1.default.readFileSync(packageJsonPath).toString()).version;
         console.log(constants_js_1.APP_PREFIX, `Testomatio Reporter v${this.version}`);
+        if (this.disableSourceCodeFetching) {
+            console.log(constants_js_1.APP_PREFIX, '🚫 Source code fetching is disabled');
+        }
+        if (this.suiteOrganization === 'fullpath') {
+            console.log(constants_js_1.APP_PREFIX, '📁 Using fullpath suite organization (may create nested structure)');
+        }
+        else {
+            console.log(constants_js_1.APP_PREFIX, '📋 Using classname suite organization (avoids duplicates)');
+        }
     }
     connectAdapter() {
         if (this.opts.javaTests) {
@@ -130,6 +143,7 @@ class XmlReader {
     processNUnit(jsonSuite) {
         const { result, total, passed, failed, inconclusive, skipped } = jsonSuite;
         reduceOptions.preferClassname = this.stats.language === 'python';
+        reduceOptions.suiteOrganization = this.suiteOrganization;
         const resultTests = processTestSuite(jsonSuite['test-suite']);
         debug('Raw tests extracted from NUnit XML:', resultTests.length);
         debug('Raw tests:', resultTests.map(t => ({ title: t.title, example: t.example, file: t.file })));
@@ -477,6 +491,11 @@ class XmlReader {
         return this.stats;
     }
     fetchSourceCode() {
+        // Skip source code fetching if disabled
+        if (this.disableSourceCodeFetching) {
+            debug('Source code fetching is disabled');
+            return;
+        }
         this.tests.forEach(t => {
             try {
                 const file = this.adapter.getFilePath(t);
@@ -640,11 +659,8 @@ function reduceTestCases(prev, item) {
         // SpecFlow config
         let { title, tags, testId } = fetchProperties(isParametrized ? item : testCaseItem);
         let example = null;
-        // Simple suite title extraction (version 2.1.1 approach) with fallback to enhanced
-        let suiteTitle = preferClassname ? testCaseItem.classname : item.name || testCaseItem.classname;
-        if (!suiteTitle && item.fullname) {
-            suiteTitle = extractTestExplorerSuiteTitle(testCaseItem, item);
-        }
+        // Smart suite title extraction to avoid duplicates
+        const suiteTitle = getSuiteTitle(testCaseItem, item, isParametrized, reduceOptions.suiteOrganization);
         title ||= testCaseItem.name || testCaseItem.methodname || testCaseItem.classname;
         tags ||= [];
         // Store original test name for enhanced parameter extraction
@@ -832,6 +848,34 @@ function processTestSuite(testsuite) {
     const subSuites = suites.filter(s => s['test-suite'] && !s['test-case']);
     return [...subSuites.map(s => processTestSuite(s['test-suite'])), ...suites.reduce(reduceTestCases, [])].flat();
 }
+function getSuiteTitle(testCaseItem, item, isParametrized, suiteOrganization = 'classname') {
+    let suiteTitle;
+    if (suiteOrganization === 'fullpath') {
+        // Use full namespace path (old behavior that creates detailed structure)
+        if (item.fullname) {
+            return item.fullname;
+        }
+        suiteTitle = testCaseItem.classname || item.name;
+    }
+    else {
+        // Use classname approach (default - avoids duplicates)
+        if (isParametrized) {
+            // For parameterized tests, use the class name to group them
+            suiteTitle = item.name || testCaseItem.classname;
+        }
+        else {
+            // For regular tests, prefer classname over fullname to avoid long paths
+            suiteTitle = testCaseItem.classname || item.name;
+        }
+        // If still no suite title and we have fullname, extract just the class name
+        if (!suiteTitle && item.fullname) {
+            const fullnameParts = item.fullname.split('.');
+            suiteTitle = fullnameParts[fullnameParts.length - 1]; // Just the class name
+        }
+    }
+    // Fallback
+    return suiteTitle || 'UnknownClass';
+}
 function fetchProperties(item) {
     const tags = [];
     let title = '';

package/package.json

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

package/src/xmlReader.js

@@ -35,6 +35,7 @@ const {
   TESTOMATIO_ENV,
   TESTOMATIO_RUN,
   TESTOMATIO_MARK_DETACHED,
+  TESTOMATIO_DISABLE_SOURCE_CODE,
 } = process.env;
 
 const options = {
@@ -66,6 +67,10 @@ class XmlReader {
     if (!this.adapter) throw new Error('XML adapter for this format not found');
 
     this.opts = opts || {};
+    // Check if source code fetching should be disabled
+    this.disableSourceCodeFetching = opts.disableSourceCodeFetching || TESTOMATIO_DISABLE_SOURCE_CODE;
+    // Control suite organization strategy: 'classname' (default) or 'fullpath'
+    this.suiteOrganization = opts.suiteOrganization || process.env.TESTOMATIO_SUITE_ORGANIZATION || 'classname';
     this.store = {};
     this.pipesPromise = pipesFactory(opts, this.store);
 
@@ -79,6 +84,16 @@ class XmlReader {
     const packageJsonPath = path.resolve(__dirname, '..', 'package.json');
     this.version = JSON.parse(fs.readFileSync(packageJsonPath).toString()).version;
     console.log(APP_PREFIX, `Testomatio Reporter v${this.version}`);
+
+    if (this.disableSourceCodeFetching) {
+      console.log(APP_PREFIX, '🚫 Source code fetching is disabled');
+    }
+
+    if (this.suiteOrganization === 'fullpath') {
+      console.log(APP_PREFIX, '📁 Using fullpath suite organization (may create nested structure)');
+    } else {
+      console.log(APP_PREFIX, '📋 Using classname suite organization (avoids duplicates)');
+    }
   }
 
   connectAdapter() {
@@ -160,6 +175,7 @@ class XmlReader {
     const { result, total, passed, failed, inconclusive, skipped } = jsonSuite;
 
     reduceOptions.preferClassname = this.stats.language === 'python';
+    reduceOptions.suiteOrganization = this.suiteOrganization;
     const resultTests = processTestSuite(jsonSuite['test-suite']);
 
     debug('Raw tests extracted from NUnit XML:', resultTests.length);
@@ -551,6 +567,12 @@ class XmlReader {
   }
 
   fetchSourceCode() {
+    // Skip source code fetching if disabled
+    if (this.disableSourceCodeFetching) {
+      debug('Source code fetching is disabled');
+      return;
+    }
+
     this.tests.forEach(t => {
       try {
         const file = this.adapter.getFilePath(t);
@@ -728,11 +750,8 @@ function reduceTestCases(prev, item) {
       let { title, tags, testId } = fetchProperties(isParametrized ? item : testCaseItem);
       let example = null;
 
-      // Simple suite title extraction (version 2.1.1 approach) with fallback to enhanced
-      let suiteTitle = preferClassname ? testCaseItem.classname : item.name || testCaseItem.classname;
-      if (!suiteTitle && item.fullname) {
-        suiteTitle = extractTestExplorerSuiteTitle(testCaseItem, item);
-      }
+      // Smart suite title extraction to avoid duplicates
+      const suiteTitle = getSuiteTitle(testCaseItem, item, isParametrized, reduceOptions.suiteOrganization);
 
       title ||= testCaseItem.name || testCaseItem.methodname || testCaseItem.classname;
       tags ||= [];
@@ -942,6 +961,36 @@ function processTestSuite(testsuite) {
   return [...subSuites.map(s => processTestSuite(s['test-suite'])), ...suites.reduce(reduceTestCases, [])].flat();
 }
 
+function getSuiteTitle(testCaseItem, item, isParametrized, suiteOrganization = 'classname') {
+  let suiteTitle;
+
+  if (suiteOrganization === 'fullpath') {
+    // Use full namespace path (old behavior that creates detailed structure)
+    if (item.fullname) {
+      return item.fullname;
+    }
+    suiteTitle = testCaseItem.classname || item.name;
+  } else {
+    // Use classname approach (default - avoids duplicates)
+    if (isParametrized) {
+      // For parameterized tests, use the class name to group them
+      suiteTitle = item.name || testCaseItem.classname;
+    } else {
+      // For regular tests, prefer classname over fullname to avoid long paths
+      suiteTitle = testCaseItem.classname || item.name;
+    }
+
+    // If still no suite title and we have fullname, extract just the class name
+    if (!suiteTitle && item.fullname) {
+      const fullnameParts = item.fullname.split('.');
+      suiteTitle = fullnameParts[fullnameParts.length - 1]; // Just the class name
+    }
+  }
+
+  // Fallback
+  return suiteTitle || 'UnknownClass';
+}
+
 function fetchProperties(item) {
   const tags = [];
   let title = '';