@testomatio/reporter 2.3.8-rc.1 → 2.3.8

package/src/junit-adapter/nunit-parser.js

@@ -217,6 +217,16 @@ export class NUnitXmlParser {
 
     const files = [];
 
+    // Extract attachments (NUnit format)
+    if (testCase.attachments) {
+      const attachments = Array.isArray(testCase.attachments.attachment)
+        ? testCase.attachments.attachment
+        : [testCase.attachments.attachment];
+
+      const attachmentFiles = attachments.filter(a => a && a.filePath).map(a => a.filePath);
+      files.push(...attachmentFiles);
+    }
+
     if (testCase.failure) {
       message = testCase.failure.message || '';
       stack = testCase.failure['stack-trace'] || testCase.failure['#text'] || '';
@@ -382,14 +392,14 @@ export class NUnitXmlParser {
       parameters.push(current.trim());
     }
 
-    // Clean up parameters - remove quotes if they wrap the entire parameter
+    // Clean up parameters - remove quotes if they wrap the entire parameter and filter empty ones
     return parameters.map(param => {
       param = param.trim();
       if ((param.startsWith('"') && param.endsWith('"')) || (param.startsWith("'") && param.endsWith("'"))) {
         return param.slice(1, -1);
       }
       return param;
-    });
+    }).filter(p => !!p);
   }
 
   /**

package/src/pipe/testomatio.js

@@ -163,7 +163,7 @@ class TestomatioPipe {
 
   /**
    * Creates a new run on Testomat.io
-   * @param {{isBatchEnabled?: boolean}} params
+   * @param {{isBatchEnabled?: boolean, kind?: string}} params
    * @returns Promise<void>
    */
   async createRun(params = {}) {
@@ -204,6 +204,7 @@ class TestomatioPipe {
         label: this.label,
         shared_run: this.sharedRun,
         shared_run_timeout: this.sharedRunTimeout,
+        kind: params.kind,
       }).filter(([, value]) => !!value),
     );
     debug(' >>>>>> Run params', JSON.stringify(runParams, null, 2));

package/src/utils/utils.js

@@ -390,17 +390,6 @@ const fetchSourceCode = (contents, opts = {}) => {
             break;
           // Stop at class declaration
           if (trimmed.includes(' class ') && trimmed.includes('public')) break;
-          // Stop at helper method calls (like ProcessBooleanValue, AddNumbers) - these are private methods
-          if (methodStartFound && trimmed.match(/^\s*\/\/\s*Helper methods for testing/)) break;
-        }
-      }
-
-      // For C# tests, stop if we encounter helper method calls in the method body
-      if (opts.lang === 'csharp' && methodStartFound && braceDepth > 0) {
-        const trimmed = lines[i].trim();
-        // Stop at comment indicating helper methods section
-        if (trimmed.match(/^\s*\/\/\s*Helper methods for testing/)) {
-          break;
         }
       }
 

package/src/xmlReader.js

@@ -80,7 +80,7 @@ class XmlReader {
 
     // Enhanced NUnit parsing - enabled by default for NUnit XML
     // Can be disabled via opts.enhancedNunit = false or TESTOMATIO_LEGACY_NUNIT=1
-    this.enhancedNunit = opts.enhancedNunit !== false && !transformEnvVarToBoolean(TESTOMATIO_LEGACY_NUNIT); // Default true, can be disabled
+    this.enhancedNunit = !transformEnvVarToBoolean(TESTOMATIO_LEGACY_NUNIT);
     this.groupParameterized = opts.groupParameterized !== false; // Default true, can be disabled
 
     // @ts-ignore
@@ -274,7 +274,9 @@ class XmlReader {
   _parseTRXTestDefinition(td) {
     const title = td.name.replace(/\(.*?\)/, '').trim();
     const exampleMatch = td.name.match(/\((.*?)\)/);
-    const example = exampleMatch ? { ...exampleMatch[1].split(',') } : null;
+    const example = exampleMatch ? {
+      ...exampleMatch[1].split(',').map(p => p.trim()).filter(p => p !== '')
+    } : null;
 
     const suite = td.TestMethod.className.split(', ')[0].split('.');
     const suite_title = suite.pop();
@@ -598,7 +600,7 @@ function reduceTestCases(prev, item) {
 
       const exampleMatches = testCaseItem.name?.match(/\S\((.*?)\)/);
       if (exampleMatches) {
-        example = { ...exampleMatches[1].split(',').map(v => v.trim().replace(/[^\w\s-]/g, '')) };
+        example = { ...exampleMatches[1].split(',').map(v => v.trim().replace(/[^\w\s-]/g, '')).filter(v => v !== '') };
         title = title.replace(/\(.*?\)/, '').trim();
       }