@testomatio/reporter 2.3.5-beta-5-xml-import → 2.3.5-beta.8-xml-import

package/lib/junit-adapter/csharp.d.ts

@@ -1,4 +1,5 @@
 export default CSharpAdapter;
 declare class CSharpAdapter extends Adapter {
+    getFilePath(t: any): string;
 }
 import Adapter from './adapter.js';

package/lib/junit-adapter/csharp.js

@@ -7,53 +7,24 @@ const path_1 = __importDefault(require("path"));
 const adapter_js_1 = __importDefault(require("./adapter.js"));
 class CSharpAdapter extends adapter_js_1.default {
     formatTest(t) {
-        // Don't override example if it already exists from NUnit XML processing
-        // The xmlReader.js already extracts parameters correctly from <arguments>
-        if (!t.example) {
-            const title = t.title.replace(/\(.*?\)/, '').trim();
-            const exampleMatch = t.title.match(/\((.*?)\)/);
-            if (exampleMatch) {
-                // Keep as array for consistency with NUnit XML processing
-                t.example = exampleMatch[1].split(',').map(param => param.trim());
-            }
-            t.title = title.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_1.default.isAbsolute(filePath)) {
-                // Try to find project-relative path
-                const cwd = process.cwd().replace(/\\/g, '/');
-                if (filePath.startsWith(cwd)) {
-                    filePath = path_1.default.relative(cwd, filePath).replace(/\\/g, '/');
-                }
-            }
-            return filePath;
-        }
-        // Convert namespace path to file path
-        const fileName = namespaceToFileName(filePath);
+        const fileName = namespaceToFileName(t.file);
         return fileName;
     }
 }
 module.exports = 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_1.default.sep)}.cs`;
 }

package/lib/pipe/debug.js

@@ -18,7 +18,7 @@ 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: [],

package/lib/pipe/testomatio.js

@@ -23,7 +23,7 @@ if (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,7 +60,7 @@ class TestomatioPipe {
                 retry: constants_js_1.REPORTER_REQUEST_RETRIES.retriesPerRequest,
                 retryDelay: constants_js_1.REPORTER_REQUEST_RETRIES.retryTimeout,
                 httpMethodsToRetry: ['GET', 'PUT', 'HEAD', 'OPTIONS', 'DELETE', 'POST'],
-                shouldRetry: error => {
+                shouldRetry: (error) => {
                     if (!error.response)
                         return false;
                     switch (error.response?.status) {
@@ -73,8 +73,8 @@ class TestomatioPipe {
                             break;
                     }
                     return error.response?.status >= 401; // Retry on 401+ and 5xx
-                },
-            },
+                }
+            }
         });
         this.isEnabled = true;
         // do not finish this run (for parallel testing)
@@ -193,7 +193,7 @@ class TestomatioPipe {
                 method: 'PUT',
                 url: `/api/reporter/${this.runId}`,
                 data: runParams,
-                responseType: 'json',
+                responseType: 'json'
             });
             if (resp.data.artifacts)
                 (0, pipe_utils_js_1.setS3Credentials)(resp.data.artifacts);
@@ -206,7 +206,7 @@ class TestomatioPipe {
                 url: '/api/reporter',
                 data: runParams,
                 maxContentLength: Infinity,
-                responseType: 'json',
+                responseType: 'json'
             });
             this.runId = resp.data.uid;
             this.runUrl = `${this.url}/${resp.data.url.split('/').splice(3).join('/')}`;
@@ -259,17 +259,15 @@ class TestomatioPipe {
         this.#formatData(data);
         const json = json_cycle_1.default.stringify(data);
         debug('Adding test', json);
-        return this.client
-            .request({
+        return this.client.request({
             method: 'POST',
             url: `/api/reporter/${this.runId}/testrun`,
             data: json,
             headers: {
                 'Content-Type': 'application/json',
             },
-            maxContentLength: Infinity,
-        })
-            .catch(err => {
+            maxContentLength: Infinity
+        }).catch(err => {
             this.requestFailures++;
             this.notReportedTestsCount++;
             if (err.response) {
@@ -314,21 +312,19 @@ class TestomatioPipe {
         // get tests from batch and clear batch
         const testsToSend = this.batch.tests.splice(0);
         debug('📨 Batch upload', testsToSend.length, 'tests');
-        return this.client
-            .request({
+        return this.client.request({
             method: 'POST',
             url: `/api/reporter/${this.runId}/testrun`,
             data: {
                 api_key: this.apiKey,
                 tests: testsToSend,
-                batch_index: this.batch.batchIndex,
+                batch_index: this.batch.batchIndex
             },
             headers: {
                 'Content-Type': 'application/json',
             },
-            maxContentLength: Infinity,
-        })
-            .catch(err => {
+            maxContentLength: Infinity
+        }).catch(err => {
             this.requestFailures++;
             this.notReportedTestsCount += testsToSend.length;
             if (err.response) {
@@ -412,7 +408,7 @@ class TestomatioPipe {
                         status_event,
                         detach: params.detach,
                         tests: params.tests,
-                    },
+                    }
                 });
                 console.log(constants_js_1.APP_PREFIX, '✅ Testrun finished');
                 if (this.runUrl) {

package/lib/utils/utils.js

@@ -173,8 +173,6 @@ exports.fetchSourceCodeFromStackTrace = fetchSourceCodeFromStackTrace;
 exports.TEST_ID_REGEX = /@T([\w\d]{8})/;
 exports.SUITE_ID_REGEX = /@S([\w\d]{8})/;
 const fetchIdFromCode = (code, opts = {}) => {
-    if (!code)
-        return null;
     const comments = code
         .split('\n')
         .map(l => l.trim())
@@ -217,29 +215,10 @@ const fetchSourceCode = (contents, opts = {}) => {
                 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) {
+            if (lineIndex === -1)
+                lineIndex = lines.findIndex(l => l.includes(`public void ${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;
-                }
-            }
         }
         else {
             lineIndex = lines.findIndex(l => l.includes(title));
@@ -248,7 +227,7 @@ const fetchSourceCode = (contents, opts = {}) => {
     if (opts.prepend) {
         lineIndex -= opts.prepend;
     }
-    if (lineIndex !== -1 && lineIndex !== undefined) {
+    if (lineIndex) {
         const result = [];
         for (let i = lineIndex; i < lineIndex + limit; i++) {
             if (lines[i] === undefined)
@@ -291,14 +270,6 @@ const fetchSourceCode = (contents, opts = {}) => {
                     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]);
         }

package/lib/xmlReader.d.ts

@@ -77,13 +77,6 @@ declare class XmlReader {
         skipped_count: number;
         tests: any[];
     };
-    deduplicateTestsByFQN(tests: any): any[];
-    generateFQN(test: any): string;
-    generateNormalizedFQN(test: any): string;
-    extractAssemblyName(test: any): any;
-    extractNamespace(test: any): any;
-    extractClassName(test: any): any;
-    extractCsFileFromPath(test: any): any;
     calculateStats(): {};
     fetchSourceCode(): void;
     formatTests(): void;

package/lib/xmlReader.js

@@ -131,21 +131,7 @@ class XmlReader {
         const { result, total, passed, failed, inconclusive, skipped } = jsonSuite;
         reduceOptions.preferClassname = this.stats.language === 'python';
         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 })));
-        // Optional deduplication for complex NUnit scenarios - can be enabled via options
-        let finalTests = resultTests;
-        if (this.opts.enableNUnitDeduplication) {
-            finalTests = this.deduplicateTestsByFQN(resultTests);
-            debug('Tests after deduplication:', finalTests.length);
-            debug('Deduplicated tests:', finalTests.map(t => ({
-                title: t.title,
-                examples: t.examples,
-                example: t.example,
-                file: t.file,
-            })));
-        }
-        this.tests = this.tests.concat(finalTests);
+        this.tests = this.tests.concat(resultTests);
         return {
             status: result?.toLowerCase(),
             create_tests: true,
@@ -153,7 +139,7 @@ class XmlReader {
             passed_count: parseInt(passed, 10),
             failed_count: parseInt(failed, 10),
             skipped_count: parseInt(inconclusive + skipped, 10),
-            tests: finalTests,
+            tests: resultTests,
         };
     }
     processTRX(jsonSuite) {
@@ -289,171 +275,6 @@ class XmlReader {
             tests,
         };
     }
-    deduplicateTestsByFQN(tests) {
-        const fqnMap = new Map();
-        tests.forEach(test => {
-            const fqn = this.generateNormalizedFQN(test);
-            if (fqnMap.has(fqn)) {
-                const existingTest = fqnMap.get(fqn);
-                // For parameterized tests, merge as Examples
-                if (test.example && Array.isArray(test.example) && test.example.length > 0) {
-                    // Initialize examples array if it doesn't exist
-                    if (!existingTest.examples) {
-                        existingTest.examples = [];
-                        // Add the existing test's example as the first item if it has parameters
-                        if (existingTest.example && Array.isArray(existingTest.example) && existingTest.example.length > 0) {
-                            existingTest.examples.push({
-                                parameters: existingTest.example,
-                                status: existingTest.status,
-                                run_time: existingTest.run_time,
-                                message: existingTest.message,
-                                stack: existingTest.stack,
-                            });
-                            // Clear the main test's example since it's now in examples array
-                            delete existingTest.example;
-                        }
-                    }
-                    // Add this test's execution as an example
-                    existingTest.examples.push({
-                        parameters: test.example,
-                        status: test.status,
-                        run_time: test.run_time,
-                        message: test.message,
-                        stack: test.stack,
-                    });
-                    // Update the main test status to reflect the worst status
-                    if (test.status === 'failed' || existingTest.status === 'failed') {
-                        existingTest.status = 'failed';
-                    }
-                    else if (test.status === 'skipped' && existingTest.status !== 'failed') {
-                        existingTest.status = 'skipped';
-                    }
-                    // Update total run time
-                    existingTest.run_time = (existingTest.run_time || 0) + (test.run_time || 0);
-                }
-                else {
-                    // Merge test properties for non-parameterized tests, prioritizing Test Explorer structure
-                    if (test.test_id && !existingTest.test_id) {
-                        existingTest.test_id = test.test_id;
-                    }
-                    // Keep the most complete test data
-                    if (test.stack && !existingTest.stack) {
-                        existingTest.stack = test.stack;
-                    }
-                    if (test.message && !existingTest.message) {
-                        existingTest.message = test.message;
-                    }
-                }
-                // Prefer Test Explorer structure (longer, more complete suite_title)
-                if (test.suite_title && test.suite_title.length > existingTest.suite_title.length) {
-                    existingTest.suite_title = test.suite_title;
-                }
-                // Always use the source file path if available
-                if (test.file && test.file.endsWith('.cs')) {
-                    existingTest.file = test.file;
-                }
-                else if (!existingTest.file || !existingTest.file.endsWith('.cs')) {
-                    existingTest.file = this.extractCsFileFromPath(test);
-                }
-            }
-            else {
-                // Fix file path to use proper .cs file names from source paths
-                if (!test.file || !test.file.endsWith('.cs')) {
-                    test.file = this.extractCsFileFromPath(test);
-                }
-                fqnMap.set(fqn, test);
-            }
-        });
-        return Array.from(fqnMap.values());
-    }
-    generateFQN(test) {
-        // Generate Fully Qualified Name: Namespace + Class + Method (standard .NET FQN)
-        // Don't include assembly as it can vary between different test structures
-        const namespace = this.extractNamespace(test);
-        const className = this.extractClassName(test);
-        const methodName = test.title;
-        // Use the most complete namespace.class structure available
-        if (test.suite_title && test.suite_title.includes('.')) {
-            return `${test.suite_title}.${methodName}`;
-        }
-        return `${namespace}.${className}.${methodName}`;
-    }
-    generateNormalizedFQN(test) {
-        // Generate normalized FQN for deduplication by extracting the core namespace.class.method
-        // For parameterized tests, we want the SAME FQN so they merge into one test with multiple Examples
-        const fullClassName = test.suite_title || '';
-        const methodName = test.title;
-        // Extract the most specific namespace.class pattern
-        if (fullClassName.includes('.')) {
-            const parts = fullClassName.split('.');
-            if (parts.length >= 2) {
-                const className = parts[parts.length - 1];
-                // Look for common .NET namespace patterns and normalize them:
-                // TestProject.Tests.MyClass -> Tests.MyClass
-                // Tests.MyClass -> Tests.MyClass
-                // MyProject.SubNamespace.Tests.MyClass -> Tests.MyClass
-                let normalizedNamespace = '';
-                for (let i = parts.length - 2; i >= 0; i--) {
-                    const part = parts[i];
-                    // Build namespace from right to left, excluding project names
-                    if (part === 'Tests' || part.endsWith('Tests') || part.includes('Test')) {
-                        // Found a test namespace, use it as the normalized namespace
-                        normalizedNamespace = part;
-                        break;
-                    }
-                    else if (i === parts.length - 2) {
-                        // If no test namespace found, use the immediate parent as namespace
-                        normalizedNamespace = part;
-                    }
-                }
-                return `${normalizedNamespace}.${className}.${methodName}`;
-            }
-        }
-        // Fallback for simple class names
-        return `${fullClassName}.${methodName}`;
-    }
-    extractAssemblyName(test) {
-        // Extract assembly name from file path or use default
-        if (test.file) {
-            const parts = test.file.split(/[/\\]/);
-            return parts[0] || 'DefaultAssembly';
-        }
-        return 'DefaultAssembly';
-    }
-    extractNamespace(test) {
-        // Extract namespace from suite_title or classname
-        if (test.suite_title && test.suite_title.includes('.')) {
-            const parts = test.suite_title.split('.');
-            return parts.slice(0, -1).join('.');
-        }
-        return test.suite_title || 'DefaultNamespace';
-    }
-    extractClassName(test) {
-        // Extract class name from suite_title
-        if (test.suite_title && test.suite_title.includes('.')) {
-            const parts = test.suite_title.split('.');
-            return parts[parts.length - 1];
-        }
-        return test.suite_title || 'DefaultClass';
-    }
-    extractCsFileFromPath(test) {
-        // Extract .cs file name from source file path, not namespace
-        if (test.file) {
-            // Look for actual .cs file path patterns
-            const csFileMatch = test.file.match(/([^/\\]+\.cs)$/);
-            if (csFileMatch) {
-                return test.file;
-            }
-            // If no .cs extension, assume it's a namespace path and convert to likely file name
-            const className = this.extractClassName(test);
-            const pathParts = test.file.split(/[/\\]/);
-            pathParts[pathParts.length - 1] = `${className}.cs`;
-            return pathParts.join('/');
-        }
-        // Fallback to class name
-        const className = this.extractClassName(test);
-        return `${className}.cs`;
-    }
     calculateStats() {
         this.stats = {
             ...this.stats,
@@ -503,9 +324,7 @@ class XmlReader {
                     return;
                 }
                 const contents = fs_1.default.readFileSync(file).toString();
-                // Try original test name first (for parameterized tests), fallback to regular title
-                const titleForLookup = t.originalTestName ? t.originalTestName.replace(/\(.*?\)/, '').trim() : t.title;
-                t.code = (0, utils_js_1.fetchSourceCode)(contents, { ...t, title: titleForLookup, lang: this.stats.language });
+                t.code = (0, utils_js_1.fetchSourceCode)(contents, { ...t, lang: this.stats.language });
                 if (t.code)
                     debug('Fetched code for test %s', t.title);
                 t.test_id = (0, utils_js_1.fetchIdFromCode)(t.code, { lang: this.stats.language });
@@ -611,12 +430,7 @@ function reduceTestCases(prev, item) {
     testCases
         .filter(t => !!t)
         .forEach(testCaseItem => {
-        // Simple file extraction (version 2.1.1 approach) with fallback to enhanced extraction
-        let file = testCaseItem.file || item.filepath || item.fullname || item.package || '';
-        // If no file found with simple approach and we have enhanced extraction enabled, use it
-        if (!file && item.filepath) {
-            file = extractSourceFilePath(testCaseItem, item);
-        }
+        const file = testCaseItem.file || item.filepath || item.fullname || item.package || '';
         let stack = '';
         let message = '';
         if (testCaseItem.error)
@@ -640,31 +454,13 @@ 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);
-        }
+        const suiteTitle = preferClassname ? testCaseItem.classname : item.name || testCaseItem.classname;
         title ||= testCaseItem.name || testCaseItem.methodname || testCaseItem.classname;
         tags ||= [];
-        // Store original test name for enhanced parameter extraction
-        const originalTestName = testCaseItem.name || testCaseItem.methodname;
-        // Enhanced NUnit-style arguments from <arguments> element
-        if (testCaseItem.arguments && testCaseItem.arguments.arg) {
-            const args = Array.isArray(testCaseItem.arguments.arg)
-                ? testCaseItem.arguments.arg
-                : [testCaseItem.arguments.arg];
-            example = args; // Store as array instead of object
-            // Remove parameters from title for NUnit tests
-            title = (testCaseItem.methodname || title).replace(/\(.*?\)/, '').trim();
-        }
-        else {
-            // Simple parameter extraction (version 2.1.1 approach)
-            const exampleMatches = testCaseItem.name?.match(/\S\((.*?)\)/);
-            if (exampleMatches) {
-                example = { ...exampleMatches[1].split(',').map(v => v.trim().replace(/[^\w\s-]/g, '')) };
-                title = title.replace(/\(.*?\)/, '').trim();
-            }
+        const exampleMatches = testCaseItem.name?.match(/\S\((.*?)\)/);
+        if (exampleMatches) {
+            example = { ...exampleMatches[1].split(',').map(v => v.trim().replace(/[^\w\s-]/g, '')) };
+            title = title.replace(/\(.*?\)/, '').trim();
         }
         stack = `${testCaseItem['system-out'] || testCaseItem.output || testCaseItem.log || ''}\n\n${stack}\n\n${suiteOutput}\n\n${suiteErr}`.trim();
         if (!testId)
@@ -712,7 +508,6 @@ function reduceTestCases(prev, item) {
             run_time: parseFloat(testCaseItem.time || testCaseItem.duration) * 1000,
             status,
             title,
-            originalTestName, // Store original name for enhanced features
             root_suite_id: TESTOMATIO_SUITE,
             suite_title: suiteTitle,
             files,
@@ -721,102 +516,6 @@ function reduceTestCases(prev, item) {
     });
     return prev;
 }
-function extractSourceFilePath(testCaseItem, item) {
-    // Priority order for file path extraction to match Test Explorer structure:
-    // 1. filepath attribute (direct .cs file path from NUnit)
-    // 2. fullname (contains full project path)
-    // 3. file attribute from test case
-    // 4. package (fallback)
-    // NUnit provides filepath attribute with actual .cs file path - use this first
-    if (item.filepath) {
-        // Clean up Windows/Unix path separators and ensure proper format
-        let filePath = item.filepath.replace(/\\/g, '/');
-        // Make relative to current working directory if absolute
-        if (path_1.default.isAbsolute(item.filepath)) {
-            const cwd = process.cwd().replace(/\\/g, '/');
-            if (filePath.startsWith(cwd)) {
-                filePath = path_1.default.relative(cwd, item.filepath).replace(/\\/g, '/');
-            }
-            else {
-                // Try to extract relative path from common patterns
-                const commonPatterns = ['/Tests/', '/test/', '/src/', '/Test/'];
-                for (const pattern of commonPatterns) {
-                    const index = filePath.lastIndexOf(pattern);
-                    if (index !== -1) {
-                        filePath = filePath.substring(index + 1);
-                        break;
-                    }
-                }
-            }
-        }
-        return filePath;
-    }
-    if (testCaseItem.file) {
-        let filePath = testCaseItem.file.replace(/\\/g, '/');
-        // Make relative to current working directory if absolute
-        if (path_1.default.isAbsolute(testCaseItem.file)) {
-            const cwd = process.cwd().replace(/\\/g, '/');
-            if (filePath.startsWith(cwd)) {
-                filePath = path_1.default.relative(cwd, testCaseItem.file).replace(/\\/g, '/');
-            }
-            else {
-                // Try to extract relative path from common patterns
-                const commonPatterns = ['/Tests/', '/test/', '/src/', '/Test/'];
-                for (const pattern of commonPatterns) {
-                    const index = filePath.lastIndexOf(pattern);
-                    if (index !== -1) {
-                        filePath = filePath.substring(index + 1);
-                        break;
-                    }
-                }
-            }
-        }
-        return filePath;
-    }
-    if (item.fullname) {
-        // Extract actual file path from fullname if it contains path separators
-        const fullnameParts = item.fullname.split('.');
-        if (fullnameParts.length > 2) {
-            // For ParameterizedMethod, get the class name (not method name)
-            // Example: "NUnit_sample_test.Tests.SampleTests.TestBooleanValue" -> "Tests/SampleTests.cs"
-            let namespaceParts, className;
-            if (item.type === 'ParameterizedMethod') {
-                // For parameterized methods, the last part is the method name, second-to-last is class
-                namespaceParts = fullnameParts.slice(1, -2); // Skip project name and method name
-                className = fullnameParts[fullnameParts.length - 2]; // Get class name
-            }
-            else {
-                // For regular classes/fixtures
-                namespaceParts = fullnameParts.slice(1, -1); // Skip project name
-                className = fullnameParts[fullnameParts.length - 1];
-            }
-            return `${namespaceParts.join('/')}/${className}.cs`;
-        }
-    }
-    if (item.package)
-        return item.package.replace(/\\/g, '/');
-    // Fallback: construct from classname
-    if (testCaseItem.classname) {
-        const parts = testCaseItem.classname.split('.');
-        const className = parts[parts.length - 1];
-        const namespacePath = parts.slice(0, -1).join('/');
-        return `${namespacePath}/${className}.cs`;
-    }
-    return '';
-}
-function extractTestExplorerSuiteTitle(testCaseItem, item) {
-    // Extract suite title to match Test Explorer structure (Project/Namespace hierarchy)
-    // Priority: fullname > classname > name
-    if (item.fullname) {
-        // Use fullname to maintain Test Explorer structure
-        return item.fullname;
-    }
-    if (testCaseItem.classname) {
-        return testCaseItem.classname;
-    }
-    // Fallback to item name but prefer classname structure
-    return item.name || testCaseItem.classname || 'UnknownClass';
-}
 function processTestSuite(testsuite) {
     if (!testsuite)
         return [];
@@ -828,8 +527,7 @@ function processTestSuite(testsuite) {
     if (!Array.isArray(testsuite)) {
         suites = [testsuite];
     }
-    // Simple approach from version 2.1.1 with enhanced processing for complex scenarios
-    const subSuites = suites.filter(s => s['test-suite'] && !s['test-case']);
+    const subSuites = suites.filter(s => s['test-suite'] && !testsuite['test-case']);
     return [...subSuites.map(s => processTestSuite(s['test-suite'])), ...suites.reduce(reduceTestCases, [])].flat();
 }
 function fetchProperties(item) {

package/package.json

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