@testomatio/reporter 2.3.2-beta.3-xml-import → 2.3.2

package/lib/adapter/playwright.d.ts

@@ -11,4 +11,10 @@ declare class PlaywrightReporter {
     onEnd(result: any): Promise<void>;
     #private;
 }
+/**
+ * Extracts and normalizes tags from test title, test options, and suite level
+ * @param {*} test - testInfo object from Playwright
+ * @returns {string[]} - array of normalized tags
+ */
+export function extractTags(test: any): string[];
 import TestomatioClient from '../client.js';

package/lib/adapter/playwright.js

@@ -3,6 +3,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.extractTags = extractTags;
 const picocolors_1 = __importDefault(require("picocolors"));
 const crypto_1 = __importDefault(require("crypto"));
 const os_1 = __importDefault(require("os"));
@@ -253,3 +254,5 @@ function getTestContextName(test) {
     return `${test._requireFile || ''}_${test.title}`;
 }
 module.exports = PlaywrightReporter;
+
+module.exports.extractTags = extractTags;

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/utils/utils.d.ts

@@ -2,7 +2,7 @@ export function getPackageVersion(): any;
 export const TEST_ID_REGEX: RegExp;
 export const SUITE_ID_REGEX: RegExp;
 export function ansiRegExp(): RegExp;
-export function cleanLatestRunId(): void;
+export function cleanLatestRunId(): any;
 export function isSameTest(test: any, t: any): boolean;
 export function fetchSourceCode(contents: any, opts?: {}): string;
 export function fetchSourceCodeFromStackTrace(stack?: string): string;
@@ -47,7 +47,7 @@ export function removeColorCodes(input: any): any;
  * @returns {String|null} testInfo as one string
  */
 export function specificTestInfo(test: any): string | null;
-export function storeRunId(runId: any): void;
+export function storeRunId(runId: any): any;
 export namespace testRunnerHelper {
     function getNameOfCurrentlyRunningTest(): any;
 }

package/lib/utils/utils.js

@@ -172,8 +172,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())
@@ -216,29 +214,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));
@@ -247,7 +226,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)
@@ -290,14 +269,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]);
         }
@@ -419,7 +390,14 @@ function storeRunId(runId) {
     if (!runId || runId === 'undefined')
         return;
     const filePath = path_1.default.join(os_1.default.tmpdir(), `testomatio.latest.run`);
-    fs_1.default.writeFileSync(filePath, runId);
+    try {
+        fs_1.default.writeFileSync(filePath, runId);
+    }
+    catch (e) {
+        if (e.code === 'ENOENT')
+            return null;
+        debug('Could not store latest run ID file: ', e.message);
+    }
 }
 /**
  *
@@ -438,7 +416,6 @@ function readLatestRunId() {
         return fs_1.default.readFileSync(filePath)?.toString()?.trim() ?? null;
     }
     catch (e) {
-        console.warn('Could not read latest run ID from file: ', e);
         return null;
     }
 }
@@ -452,6 +429,8 @@ function cleanLatestRunId() {
         debug(`Cleaned latest run ID (${runId}) file`, filePath);
     }
     catch (e) {
+        if (e.code === 'ENOENT')
+            return null;
         console.warn('Could not clean latest run ID file: ', e);
     }
 }

package/lib/xmlReader.d.ts

@@ -46,7 +46,7 @@ declare class XmlReader {
         passed_count: number;
         skipped_count: number;
         status: string;
-        tests: any;
+        tests: any[];
         tests_count: number;
     };
     processNUnit(jsonSuite: any): {
@@ -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,18 +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 })));
-        // Deduplicate tests based on FQN (Assembly + Namespace + Class + Method)
-        const deduplicatedTests = this.deduplicateTestsByFQN(resultTests);
-        debug('Tests after deduplication:', deduplicatedTests.length);
-        debug('Deduplicated tests:', deduplicatedTests.map(t => ({
-            title: t.title,
-            examples: t.examples,
-            example: t.example,
-            file: t.file,
-        })));
-        this.tests = this.tests.concat(deduplicatedTests);
+        this.tests = this.tests.concat(resultTests);
         return {
             status: result?.toLowerCase(),
             create_tests: true,
@@ -150,7 +139,7 @@ class XmlReader {
             passed_count: parseInt(passed, 10),
             failed_count: parseInt(failed, 10),
             skipped_count: parseInt(inconclusive + skipped, 10),
-            tests: deduplicatedTests,
+            tests: resultTests,
         };
     }
     processTRX(jsonSuite) {
@@ -286,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,
@@ -496,13 +320,11 @@ class XmlReader {
                         this.stats.language = 'csharp';
                 }
                 if (!fs_1.default.existsSync(file)) {
-                    debug('Failed to open file with the source code: %s', file);
+                    debug('Failed to open file with the source code', file);
                     return;
                 }
                 const contents = fs_1.default.readFileSync(file).toString();
-                // Use original test name for source code lookup, not humanized title
-                const originalTitle = t.originalTestName ? t.originalTestName.replace(/\(.*?\)/, '').trim() : t.title;
-                t.code = (0, utils_js_1.fetchSourceCode)(contents, { ...t, title: originalTitle, 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 });
@@ -608,8 +430,7 @@ function reduceTestCases(prev, item) {
     testCases
         .filter(t => !!t)
         .forEach(testCaseItem => {
-        // Use consistent Test Explorer structure: prioritize fullname for file path
-        const file = extractSourceFilePath(testCaseItem, item);
+        const file = testCaseItem.file || item.filepath || item.fullname || item.package || '';
         let stack = '';
         let message = '';
         if (testCaseItem.error)
@@ -629,33 +450,17 @@ function reduceTestCases(prev, item) {
         if (!message)
             message = stack.trim().split('\n')[0];
         const isParametrized = item.type === 'ParameterizedMethod';
+        const preferClassname = reduceOptions.preferClassname || isParametrized;
         // SpecFlow config
         let { title, tags, testId } = fetchProperties(isParametrized ? item : testCaseItem);
         let example = null;
-        // Use consistent Test Explorer structure for suite title
-        const 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 parameter extraction
-        const originalTestName = testCaseItem.name || testCaseItem.methodname;
-        // Handle 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 {
-            // Fallback to parsing parameters from test name (SpecFlow, etc.)
-            const exampleMatches = originalTestName?.match(/\((.*?)\)$/);
-            if (exampleMatches) {
-                // Extract and store parameters as Examples
-                const parameterValues = exampleMatches[1].split(',').map(v => v.trim().replace(/['"]/g, ''));
-                example = parameterValues;
-                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)
@@ -703,7 +508,6 @@ function reduceTestCases(prev, item) {
             run_time: parseFloat(testCaseItem.time || testCaseItem.duration) * 1000,
             status,
             title,
-            originalTestName, // Store original name for parameter-aware FQN generation
             root_suite_id: TESTOMATIO_SUITE,
             suite_title: suiteTitle,
             files,
@@ -712,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 [];
@@ -819,20 +527,8 @@ function processTestSuite(testsuite) {
     if (!Array.isArray(testsuite)) {
         suites = [testsuite];
     }
-    let allResults = [];
-    for (const suite of suites) {
-        // Process child test suites recursively (TestFixture, ParameterizedMethod, etc.)
-        if (suite['test-suite']) {
-            const childResults = processTestSuite(suite['test-suite']);
-            allResults = allResults.concat(childResults);
-        }
-        // Process direct test cases in this suite
-        if (suite['test-case'] || suite.testcase) {
-            const leafResults = reduceTestCases([], suite);
-            allResults = allResults.concat(leafResults);
-        }
-    }
-    return allResults;
+    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) {
     const tags = [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testomatio/reporter",
-  "version": "2.3.2-beta.3-xml-import",
+  "version": "2.3.2",
   "description": "Testomatio Reporter Client",
   "engines": {
     "node": ">=18"
@@ -54,7 +54,7 @@
     "lint": "eslint src",
     "lint:fix": "eslint src --fix",
     "format": "npm run lint:fix && npm run pretty:fix",
-    "test": "mocha tests/unit/**/*_test.js",
+    "test": "mocha 'tests/unit/**/*_test.js'",
     "test:playwright": "mocha tests/adapter/playwright.test.js",
     "test:codecept": "mocha tests/adapter/codecept.test.js tests/adapter/codecept_comprehensive.test.js tests/adapter/codecept_steps_sections.test.js",
     "test:frameworks": "npm run test:playwright && npm run test:codecept",

package/src/adapter/playwright.js

@@ -283,3 +283,4 @@ function getTestContextName(test) {
 }
 
 export default PlaywrightReporter;
+export { extractTags };

package/src/junit-adapter/csharp.js

@@ -3,45 +3,18 @@ import Adapter from './adapter.js';
 
 class CSharpAdapter extends Adapter {
   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.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;
   }
 }
@@ -49,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/utils/utils.js

@@ -139,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())
@@ -182,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));
     }
@@ -217,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;
@@ -242,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]);
     }
@@ -380,7 +350,12 @@ const testRunnerHelper = {
 function storeRunId(runId) {
   if (!runId || runId === 'undefined') return;
   const filePath = path.join(os.tmpdir(), `testomatio.latest.run`);
-  fs.writeFileSync(filePath, runId);
+  try {
+    fs.writeFileSync(filePath, runId);
+  } catch (e) {
+    if (e.code === 'ENOENT') return null;
+    debug('Could not store latest run ID file: ', e.message);
+  }
 }
 
 /**
@@ -399,7 +374,6 @@ function readLatestRunId() {
 
     return fs.readFileSync(filePath)?.toString()?.trim() ?? null;
   } catch (e) {
-    console.warn('Could not read latest run ID from file: ', e);
     return null;
   }
 }
@@ -413,6 +387,7 @@ function cleanLatestRunId() {
     }
     debug(`Cleaned latest run ID (${runId}) file`, filePath);
   } catch (e) {
+    if (e.code === 'ENOENT') return null;
     console.warn('Could not clean latest run ID file: ', e);
   }
 }

package/src/xmlReader.js

@@ -162,27 +162,7 @@ class XmlReader {
     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 })),
-    );
-
-    // Deduplicate tests based on FQN (Assembly + Namespace + Class + Method)
-    const deduplicatedTests = this.deduplicateTestsByFQN(resultTests);
-
-    debug('Tests after deduplication:', deduplicatedTests.length);
-    debug(
-      'Deduplicated tests:',
-      deduplicatedTests.map(t => ({
-        title: t.title,
-        examples: t.examples,
-        example: t.example,
-        file: t.file,
-      })),
-    );
-
-    this.tests = this.tests.concat(deduplicatedTests);
+    this.tests = this.tests.concat(resultTests);
 
     return {
       status: result?.toLowerCase(),
@@ -191,7 +171,7 @@ class XmlReader {
       passed_count: parseInt(passed, 10),
       failed_count: parseInt(failed, 10),
       skipped_count: parseInt(inconclusive + skipped, 10),
-      tests: deduplicatedTests,
+      tests: resultTests,
     };
   }
 
@@ -339,194 +319,6 @@ class XmlReader {
     };
   }
 
-  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,
@@ -565,18 +357,12 @@ class XmlReader {
         }
 
         if (!fs.existsSync(file)) {
-          debug('Failed to open file with the source code: %s', file);
+          debug('Failed to open file with the source code', file);
           return;
         }
-
         const contents = fs.readFileSync(file).toString();
-
-        // Use original test name for source code lookup, not humanized title
-        const originalTitle = t.originalTestName ? t.originalTestName.replace(/\(.*?\)/, '').trim() : t.title;
-
-        t.code = fetchSourceCode(contents, { ...t, title: originalTitle, lang: this.stats.language });
+        t.code = fetchSourceCode(contents, { ...t, lang: this.stats.language });
         if (t.code) debug('Fetched code for test %s', t.title);
-
         t.test_id = fetchIdFromCode(t.code, { lang: this.stats.language });
         if (t.test_id) debug('Fetched test id %s for test %s', t.test_id, t.title);
       } catch (err) {
@@ -699,8 +485,7 @@ function reduceTestCases(prev, item) {
   testCases
     .filter(t => !!t)
     .forEach(testCaseItem => {
-      // Use consistent Test Explorer structure: prioritize fullname for file path
-      const file = extractSourceFilePath(testCaseItem, item);
+      const file = testCaseItem.file || item.filepath || item.fullname || item.package || '';
 
       let stack = '';
       let message = '';
@@ -715,37 +500,20 @@ function reduceTestCases(prev, item) {
       if (!message) message = stack.trim().split('\n')[0];
 
       const isParametrized = item.type === 'ParameterizedMethod';
+      const preferClassname = reduceOptions.preferClassname || isParametrized;
 
       // SpecFlow config
       let { title, tags, testId } = fetchProperties(isParametrized ? item : testCaseItem);
       let example = null;
-
-      // Use consistent Test Explorer structure for suite title
-      const 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 parameter extraction
-      const originalTestName = testCaseItem.name || testCaseItem.methodname;
-
-      // Handle 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 {
-        // Fallback to parsing parameters from test name (SpecFlow, etc.)
-        const exampleMatches = originalTestName?.match(/\((.*?)\)$/);
-        if (exampleMatches) {
-          // Extract and store parameters as Examples
-          const parameterValues = exampleMatches[1].split(',').map(v => v.trim().replace(/['"]/g, ''));
-          example = parameterValues;
-          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 = `${
@@ -800,7 +568,6 @@ function reduceTestCases(prev, item) {
         run_time: parseFloat(testCaseItem.time || testCaseItem.duration) * 1000,
         status,
         title,
-        originalTestName, // Store original name for parameter-aware FQN generation
         root_suite_id: TESTOMATIO_SUITE,
         suite_title: suiteTitle,
         files,
@@ -810,113 +577,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.isAbsolute(item.filepath)) {
-      const cwd = process.cwd().replace(/\\/g, '/');
-      if (filePath.startsWith(cwd)) {
-        filePath = path.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.isAbsolute(testCaseItem.file)) {
-      const cwd = process.cwd().replace(/\\/g, '/');
-      if (filePath.startsWith(cwd)) {
-        filePath = path.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 [];
   if (testsuite.testsuite) return processTestSuite(testsuite.testsuite);
@@ -927,23 +587,9 @@ function processTestSuite(testsuite) {
     suites = [testsuite];
   }
 
-  let allResults = [];
-
-  for (const suite of suites) {
-    // Process child test suites recursively (TestFixture, ParameterizedMethod, etc.)
-    if (suite['test-suite']) {
-      const childResults = processTestSuite(suite['test-suite']);
-      allResults = allResults.concat(childResults);
-    }
-
-    // Process direct test cases in this suite
-    if (suite['test-case'] || suite.testcase) {
-      const leafResults = reduceTestCases([], suite);
-      allResults = allResults.concat(leafResults);
-    }
-  }
+  const subSuites = suites.filter(s => s['test-suite'] && !testsuite['test-case']);
 
-  return allResults;
+  return [...subSuites.map(s => processTestSuite(s['test-suite'])), ...suites.reduce(reduceTestCases, [])].flat();
 }
 
 function fetchProperties(item) {