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

package/lib/data-storage.d.ts

@@ -1,6 +1,6 @@
 export const dataStorage: DataStorage;
 declare class DataStorage {
-    static #instance: any;
+    static "__#12@#instance": any;
     /**
      *
      * @returns {DataStorage}

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

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

package/lib/junit-adapter/csharp.js

@@ -7,24 +7,53 @@ const path_1 = __importDefault(require("path"));
 const adapter_js_1 = __importDefault(require("./adapter.js"));
 class CSharpAdapter extends adapter_js_1.default {
     formatTest(t) {
-        const title = t.title.replace(/\(.*?\)/, '').trim();
-        const example = t.title.match(/\((.*?)\)/);
-        if (example)
-            t.example = { ...example[1].split(',') };
+        // 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 suite = t.suite_title.split('.');
         t.suite_title = suite.pop();
         t.file = namespaceToFileName(t.file);
-        t.title = title.trim();
         return t;
     }
     getFilePath(t) {
-        const fileName = namespaceToFileName(t.file);
+        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);
         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(path_1.default.sep)}.cs`;
+    return `${fileParts.join('/')}.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,
+                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,
+            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)
@@ -171,7 +171,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);
@@ -184,7 +184,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('/')}`;
@@ -241,17 +241,15 @@ class TestomatioPipe {
         }
         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) {
@@ -296,21 +294,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) {
@@ -398,7 +394,7 @@ class TestomatioPipe {
                         status_event,
                         detach: params.detach,
                         tests: params.tests,
-                    },
+                    }
                 });
                 if (this.runUrl) {
                     console.log(constants_js_1.APP_PREFIX, '📊 Report Saved. Report URL:', picocolors_1.default.magenta(this.runUrl));

package/lib/reporter.d.ts

@@ -5,7 +5,7 @@ export const artifact: (data: string | {
 }, context?: any) => void;
 export const log: (...args: any[]) => void;
 export const logger: {
-    #originalUserLogger: {
+    "__#13@#originalUserLogger": {
         assert(condition?: boolean, ...data: any[]): void;
         assert(value: any, message?: string, ...optionalParams: any[]): void;
         clear(): void;
@@ -50,13 +50,13 @@ export const logger: {
         profile(label?: string): void;
         profileEnd(label?: string): void;
     };
-    #userLoggerWithOverridenMethods: any;
+    "__#13@#userLoggerWithOverridenMethods": any;
     logLevel: string;
     step(strings: any, ...values: any[]): void;
     getLogs(context: string): string[];
-    #stringifyLogs(...args: any[]): string;
+    "__#13@#stringifyLogs"(...args: any[]): string;
     _templateLiteralLog(strings: any, ...args: any[]): void;
-    #logWrapper(argsArray: any, level: any): void;
+    "__#13@#logWrapper"(argsArray: any, level: any): void;
     assert(...args: any[]): void;
     debug(...args: any[]): void;
     error(...args: any[]): void;
@@ -81,7 +81,7 @@ export const linkTest: (...testIds: string[]) => void;
 export const linkJira: (...jiraIds: string[]) => void;
 declare namespace _default {
     let testomatioLogger: {
-        #originalUserLogger: {
+        "__#13@#originalUserLogger": {
             assert(condition?: boolean, ...data: any[]): void;
             assert(value: any, message?: string, ...optionalParams: any[]): void;
             clear(): void;
@@ -126,13 +126,13 @@ declare namespace _default {
             profile(label?: string): void;
             profileEnd(label?: string): void;
         };
-        #userLoggerWithOverridenMethods: any;
+        "__#13@#userLoggerWithOverridenMethods": any;
         logLevel: string;
         step(strings: any, ...values: any[]): void;
         getLogs(context: string): string[];
-        #stringifyLogs(...args: any[]): string;
+        "__#13@#stringifyLogs"(...args: any[]): string;
         _templateLiteralLog(strings: any, ...args: any[]): void;
-        #logWrapper(argsArray: any, level: any): void;
+        "__#13@#logWrapper"(argsArray: any, level: any): void;
         assert(...args: any[]): void;
         debug(...args: any[]): void;
         error(...args: any[]): void;
@@ -155,7 +155,7 @@ declare namespace _default {
     }, context?: any) => void;
     let log: (...args: any[]) => void;
     let logger: {
-        #originalUserLogger: {
+        "__#13@#originalUserLogger": {
             assert(condition?: boolean, ...data: any[]): void;
             assert(value: any, message?: string, ...optionalParams: any[]): void;
             clear(): void;
@@ -200,13 +200,13 @@ declare namespace _default {
             profile(label?: string): void;
             profileEnd(label?: string): void;
         };
-        #userLoggerWithOverridenMethods: any;
+        "__#13@#userLoggerWithOverridenMethods": any;
         logLevel: string;
         step(strings: any, ...values: any[]): void;
         getLogs(context: string): string[];
-        #stringifyLogs(...args: any[]): string;
+        "__#13@#stringifyLogs"(...args: any[]): string;
         _templateLiteralLog(strings: any, ...args: any[]): void;
-        #logWrapper(argsArray: any, level: any): void;
+        "__#13@#logWrapper"(argsArray: any, level: any): void;
         assert(...args: any[]): void;
         debug(...args: any[]): void;
         error(...args: any[]): void;

package/lib/services/artifacts.d.ts

@@ -3,7 +3,7 @@ export const artifactStorage: ArtifactStorage;
  * Artifact storage is supposed to store file paths
  */
 declare class ArtifactStorage {
-    static #instance: any;
+    static "__#14@#instance": any;
     /**
      * Singleton
      * @returns {ArtifactStorage}

package/lib/services/key-values.d.ts

@@ -1,6 +1,6 @@
 export const keyValueStorage: KeyValueStorage;
 declare class KeyValueStorage {
-    static #instance: any;
+    static "__#15@#instance": any;
     /**
      *
      * @returns {KeyValueStorage}

package/lib/services/links.d.ts

@@ -1,6 +1,6 @@
 export const linkStorage: LinkStorage;
 declare class LinkStorage {
-    static #instance: any;
+    static "__#16@#instance": any;
     /**
      *
      * @returns {LinkStorage}

package/lib/services/logger.d.ts

@@ -5,7 +5,7 @@ export const logger: Logger;
  * Supports different syntaxes to satisfy any user preferences.
  */
 declare class Logger {
-    static #instance: any;
+    static "__#13@#instance": any;
     /**
      *
      * @returns {Logger}

package/lib/utils/utils.js

@@ -172,6 +172,8 @@ 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())
@@ -214,10 +216,29 @@ const fetchSourceCode = (contents, opts = {}) => {
                 lineIndex = lines.findIndex(l => l.includes(`${title}(`));
         }
         else if (opts.lang === 'csharp') {
-            if (lineIndex === -1)
-                lineIndex = lines.findIndex(l => l.includes(`public void ${title}`));
-            if (lineIndex === -1)
+            // 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;
+                }
+            }
         }
         else {
             lineIndex = lines.findIndex(l => l.includes(title));
@@ -226,7 +247,7 @@ const fetchSourceCode = (contents, opts = {}) => {
     if (opts.prepend) {
         lineIndex -= opts.prepend;
     }
-    if (lineIndex) {
+    if (lineIndex !== -1 && lineIndex !== undefined) {
         const result = [];
         for (let i = lineIndex; i < lineIndex + limit; i++) {
             if (lines[i] === undefined)
@@ -269,6 +290,14 @@ 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

@@ -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,6 +77,13 @@ 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;