@testomatio/reporter 2.3.7-beta.1-xml-import → 2.3.7-beta.3-xml-import

package/lib/adapter/codecept.js

@@ -110,6 +110,25 @@ function CodeceptReporter(config) {
         output.stepShift = 2;
         index_js_1.services.setContext(null);
     });
+    // mark as failed all tests inside the failed hook
+    event.dispatcher.on(event.hook.failed, hook => {
+        if (hook.name !== 'BeforeSuiteHook')
+            return;
+        const suite = hook.runnable.parent;
+        if (!suite)
+            return;
+        const error = hook?.ctx?.currentTest?.err;
+        for (const test of suite.tests) {
+            client.addTestRun('failed', {
+                ...stripExampleFromTitle(test.title),
+                rid: test.uid,
+                test_id: (0, utils_js_1.getTestomatIdFromTestTitle)(test.title),
+                suite_title: stripTagsFromTitle(suite.title),
+                error,
+                time: hook?.runnable?.duration,
+            });
+        }
+    });
     event.dispatcher.on(event.suite.before, suite => {
         data_storage_js_1.dataStorage.setContext(suite.fullTitle());
     });
@@ -379,7 +398,7 @@ function formatCodeceptStep(step) {
     if (!step)
         return null;
     const category = step.constructor.name === 'HelperStep' ? 'framework' : 'user';
-    const title = step.toString(); // Use built-in toString
+    const title = (0, utils_js_1.truncate)(step); // Use built-in toString
     const duration = step.duration || 0; // Use built-in duration
     const formattedStep = {
         category,
@@ -403,10 +422,11 @@ function formatHookStep(step) {
     if (step.actor && step.name) {
         title = `${step.actor} ${step.name}`;
         if (step.args && step.args.length > 0) {
-            const argsStr = step.args.map(arg => JSON.stringify(arg)).join(', ');
+            const argsStr = step.args.map(arg => (0, utils_js_1.truncate)(JSON.stringify(arg))).join(', ');
             title += ` ${argsStr}`;
         }
     }
+    title = (0, utils_js_1.truncate)(title);
     return {
         category: 'hook',
         title,

package/lib/bin/cli.js

@@ -145,7 +145,7 @@ program
     .option('--lang <lang>', 'Language used (python, ruby, java)')
     .option('--timelimit <time>', 'default time limit in seconds to kill a stuck process')
     .action(async (pattern, opts) => {
-    if (!pattern.endsWith('.xml')) {
+    if (!pattern.endsWith('.xml') && !pattern.includes('*')) {
         pattern += '.xml';
     }
     let { javaTests, lang } = opts;

package/lib/bin/reportXml.js

@@ -25,7 +25,7 @@ program
     .option('--timelimit <time>', 'default time limit in seconds to kill a stuck process')
     .option('--env-file <envfile>', 'Load environment variables from env file')
     .action(async (pattern, opts) => {
-    if (!pattern.endsWith('.xml')) {
+    if (!pattern.endsWith('.xml') && !pattern.includes('*')) {
         pattern += '.xml';
     }
     let { javaTests, lang } = opts;

package/lib/bin/startTest.js

@@ -35,7 +35,7 @@ while (i < args.length) {
         newArgs[0] = 'start';
     }
     else if (arg === '--finish') {
-        // Map --finish to finish command  
+        // Map --finish to finish command
         newArgs[0] = 'finish';
     }
     else {
@@ -46,8 +46,8 @@ while (i < args.length) {
 }
 // Execute the main CLI with mapped arguments
 const child = (0, node_child_process_1.spawn)(process.execPath, [cliPath, ...newArgs], {
-    stdio: 'inherit'
+    stdio: 'inherit',
 });
-child.on('exit', (code) => {
+child.on('exit', code => {
     process.exit(code);
 });

package/lib/client.js

@@ -70,8 +70,8 @@ class Client {
         this.pipeStore = {};
         this.runId = '';
         this.queue = Promise.resolve();
-        // @ts-ignore this line will be removed in compiled code, because __dirname is defined in commonjs
-        const pathToPackageJSON = path_1.default.join(__dirname, '../package.json');
+        // Get package.json path - use a simple approach that works in both environments
+        const pathToPackageJSON = path_1.default.join(process.cwd(), 'package.json');
         try {
             this.version = JSON.parse(fs_1.default.readFileSync(pathToPackageJSON).toString()).version;
             console.log(constants_js_1.APP_PREFIX, `Testomatio Reporter v${this.version}`);
@@ -329,7 +329,11 @@ class Client {
      */
     formatLogs({ error, steps, logs }) {
         error = error?.trim();
-        logs = logs?.trim();
+        logs = logs
+            ?.trim()
+            .split('\n')
+            .map(l => (0, utils_js_1.truncate)(l))
+            .join('\n');
         if (Array.isArray(steps)) {
             steps = steps
                 .map(step => (0, utils_js_1.formatStep)(step))

package/lib/junit-adapter/csharp.js

@@ -7,17 +7,21 @@ 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>
+        // Extract example from title if not already present
         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());
+                // Extract parameters as object with numeric keys for API
+                const params = exampleMatch[1].split(',').map(param => param.trim());
+                t.example = {};
+                params.forEach((param, index) => {
+                    t.example[index] = param;
+                });
             }
-            t.title = title.trim();
         }
+        // Remove parameters from title to avoid duplicates in Test Suite
+        // The example field will be used for grouping on import
+        t.title = t.title.replace(/\(.*?\)/, '').trim();
         const suite = t.suite_title.split('.');
         t.suite_title = suite.pop();
         t.file = namespaceToFileName(t.file);

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

@@ -198,8 +198,19 @@ class NUnitXmlParser {
         }
         // Build file path from suite path and class name
         const filePath = this.buildFilePath(suitePath, className, parentSuite);
+        // For parameterized tests, format example as expected by Testomatio API
+        // Convert array of parameters to object with numeric keys
+        let example = null;
+        if (isParameterized && parameters.length > 0) {
+            example = {};
+            parameters.forEach((param, index) => {
+                example[index] = param;
+            });
+        }
         return {
-            title: isParameterized ? testName : methodName || testName,
+            // For runs: use full test name with parameters (TestBooleanValue(true))
+            // For import: API will group by base name using the example field
+            title: testName, // Full name with parameters for run display
             methodName: baseMethodName || methodName || testName,
             fullName: fullName,
             suitePath: suitePath,
@@ -213,8 +224,9 @@ class NUnitXmlParser {
             create: true,
             retry: false,
             // Parameterized test metadata
+            example: example, // Parameters as object for API grouping
             isParameterized: isParameterized,
-            parameters: parameters,
+            parameters: parameters, // Keep original array for reference
             baseMethodName: baseMethodName,
         };
     }

package/lib/pipe/testomatio.js

@@ -414,7 +414,6 @@ class TestomatioPipe {
                         tests: params.tests,
                     },
                 });
-                debug(constants_js_1.APP_PREFIX, '✅ Testrun finished');
                 if (this.runUrl) {
                     console.log(constants_js_1.APP_PREFIX, '📊 Report Saved. Report URL:', picocolors_1.default.magenta(this.runUrl));
                 }
@@ -425,7 +424,7 @@ class TestomatioPipe {
             if (this.runUrl && this.proceed) {
                 const notFinishedMessage = picocolors_1.default.yellow(picocolors_1.default.bold('Run was not finished because of $TESTOMATIO_PROCEED'));
                 console.log(constants_js_1.APP_PREFIX, `📊 ${notFinishedMessage}. Report URL: ${picocolors_1.default.magenta(this.runUrl)}`);
-                console.log(constants_js_1.APP_PREFIX, `🛬 Run to finish it: TESTOMATIO_RUN=${this.runId} npx start-test-run --finish`);
+                console.log(constants_js_1.APP_PREFIX, `🛬 Run to finish it: TESTOMATIO_RUN=${this.runId} npx @testomatio/reporter finish`);
             }
             if (this.hasUnmatchedTests) {
                 console.log('');