@testomatio/reporter 2.8.6-beta-fix-xml-batch → 2.9.0

package/lib/adapter/codecept.js

@@ -43,6 +43,7 @@ if (MAJOR_VERSION === 3 && MINOR_VERSION < 7) {
 }
 function CodeceptReporter(config) {
     const failedTests = [];
+    const reportedTestUids = new Set();
     let videos = [];
     let traces = [];
     const reportTestPromises = [];
@@ -141,6 +142,7 @@ function CodeceptReporter(config) {
             return;
         const error = hook?.ctx?.currentTest?.err;
         for (const test of suite.tests) {
+            reportedTestUids.add(test.uid);
             const reportTestPromise = client.addTestRun('failed', {
                 ...stripExampleFromTitle(test.title),
                 rid: test.uid,
@@ -171,9 +173,26 @@ function CodeceptReporter(config) {
             await finalizeRun('all.after');
         });
     });
+    event.dispatcher.on(event.test.skipped, test => {
+        const { uid, tags, title } = test.simplify();
+        if (uid && reportedTestUids.has(uid))
+            return;
+        index_js_1.services.setContext(null);
+        const reportTestPromise = client.addTestRun(constants_js_1.STATUS.SKIPPED, {
+            ...stripExampleFromTitle(title),
+            rid: uid,
+            test_id: (0, utils_js_1.getTestomatIdFromTestTitle)(`${title} ${tags?.join(' ')}`),
+            suite_title: test.parent && stripTagsFromTitle(stripExampleFromTitle(test.parent.title).title),
+            time: test.duration,
+            meta: test.meta,
+        });
+        reportTestPromises.push(reportTestPromise);
+        reportedTestUids.add(uid);
+    });
     event.dispatcher.on(event.test.after, test => {
         const { uid, tags, title, artifacts } = test.simplify();
         const error = test.err || null;
+        reportedTestUids.add(uid);
         failedTests.push(uid || title);
         const testObj = getTestAndMessage(title);
         const files = buildArtifactFiles(artifacts);

package/lib/pipe/coverage.js

@@ -221,9 +221,10 @@ class CoveragePipe {
      */
     #getChangedFilesFromGit(cmd) {
         try {
+            // Capture stderr (instead of ignoring it) so Git's actual error is available for diagnostics
             const result = (0, child_process_1.execSync)(cmd, {
                 encoding: 'utf-8',
-                stdio: ['pipe', 'pipe', 'ignore']
+                stdio: ['pipe', 'pipe', 'pipe']
             });
             return result
                 .split('\n')
@@ -231,15 +232,26 @@ class CoveragePipe {
                 .filter(Boolean);
         }
         catch (err) {
-            const errorMessage = err.message || '';
-            // Git edge: Not a git repository or other error
+            // Prefer Git's own stderr output, fall back to the generic error message
+            const gitOutput = (err.stderr || '').toString().trim();
+            const errorMessage = gitOutput || err.message || '';
+            // Git edge: Not a git repository
             if (errorMessage.includes('Not a git repository')) {
                 log_js_1.log.error('❌ Error: This folder is not a Git repository.');
+                return [];
             }
-            else {
-                throw new Error(`❌ Git command failed ("${cmd}"):\n`, errorMessage);
+            // Git edge: the branch/ref to diff against is not available locally.
+            // This is common in CI, where a shallow checkout fetches only the current branch.
+            if (errorMessage.includes('unknown revision') ||
+                errorMessage.includes('ambiguous argument') ||
+                errorMessage.includes('bad revision')) {
+                log_js_1.log.error(`❌ Git command failed ("${cmd}"):\n${errorMessage}`);
+                log_js_1.log.error(`🔍 Branch "${this.branch}" was not found locally. ` +
+                    `In CI this usually means a shallow checkout — fetch full history first, e.g. ` +
+                    `actions/checkout with "fetch-depth: 0", or run "git fetch origin ${this.branch}:${this.branch}".`);
+                return [];
             }
-            return [];
+            throw new Error(`❌ Git command failed ("${cmd}"):\n${errorMessage}`);
         }
     }
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testomatio/reporter",
-  "version": "2.8.6-beta-fix-xml-batch",
+  "version": "2.9.0",
   "description": "Testomatio Reporter Client",
   "engines": {
     "node": ">=18"

package/src/adapter/codecept.js

@@ -48,6 +48,7 @@ if (MAJOR_VERSION === 3 && MINOR_VERSION < 7) {
 
 function CodeceptReporter(config) {
   const failedTests = [];
+  const reportedTestUids = new Set();
   let videos = [];
   let traces = [];
   const reportTestPromises = [];
@@ -161,6 +162,7 @@ function CodeceptReporter(config) {
     const error = hook?.ctx?.currentTest?.err;
 
     for (const test of suite.tests) {
+      reportedTestUids.add(test.uid);
       const reportTestPromise = client.addTestRun('failed', {
         ...stripExampleFromTitle(test.title),
         rid: test.uid,
@@ -197,9 +199,29 @@ function CodeceptReporter(config) {
     });
   });
 
+  event.dispatcher.on(event.test.skipped, test => {
+    const { uid, tags, title } = test.simplify();
+
+    if (uid && reportedTestUids.has(uid)) return;
+
+    services.setContext(null);
+
+    const reportTestPromise = client.addTestRun(STATUS.SKIPPED, {
+      ...stripExampleFromTitle(title),
+      rid: uid,
+      test_id: getTestomatIdFromTestTitle(`${title} ${tags?.join(' ')}`),
+      suite_title: test.parent && stripTagsFromTitle(stripExampleFromTitle(test.parent.title).title),
+      time: test.duration,
+      meta: test.meta,
+    });
+    reportTestPromises.push(reportTestPromise);
+    reportedTestUids.add(uid);
+  });
+
   event.dispatcher.on(event.test.after, test => {
     const { uid, tags, title, artifacts } = test.simplify();
     const error = test.err || null;
+    reportedTestUids.add(uid);
     failedTests.push(uid || title);
     const testObj = getTestAndMessage(title);
     const files = buildArtifactFiles(artifacts);
@@ -211,8 +233,8 @@ function CodeceptReporter(config) {
 
     // Build step hierarchy with screenshot from screenshotOnFail
     const stepHierarchy = buildUnifiedStepHierarchy(
-      test.steps, 
-      hookSteps, 
+      test.steps,
+      hookSteps,
       screenshotOnFailPath
     );
 

package/src/pipe/coverage.js

@@ -261,9 +261,10 @@ class CoveragePipe { // or Changes for the future???
      */
     #getChangedFilesFromGit(cmd) {
         try {
+            // Capture stderr (instead of ignoring it) so Git's actual error is available for diagnostics
             const result = execSync(cmd, {
                 encoding: 'utf-8',
-                stdio: ['pipe', 'pipe', 'ignore']
+                stdio: ['pipe', 'pipe', 'pipe']
             });
 
             return result
@@ -272,16 +273,33 @@ class CoveragePipe { // or Changes for the future???
                 .filter(Boolean);
         }
         catch (err) {
-            const errorMessage = err.message || '';
-            // Git edge: Not a git repository or other error
+            // Prefer Git's own stderr output, fall back to the generic error message
+            const gitOutput = (err.stderr || '').toString().trim();
+            const errorMessage = gitOutput || err.message || '';
+
+            // Git edge: Not a git repository
             if (errorMessage.includes('Not a git repository')) {
-                log.error( '❌ Error: This folder is not a Git repository.');
+                log.error('❌ Error: This folder is not a Git repository.');
+                return [];
             }
-            else {
-                throw new Error(`❌ Git command failed ("${cmd}"):\n`, errorMessage);
+
+            // Git edge: the branch/ref to diff against is not available locally.
+            // This is common in CI, where a shallow checkout fetches only the current branch.
+            if (
+                errorMessage.includes('unknown revision') ||
+                errorMessage.includes('ambiguous argument') ||
+                errorMessage.includes('bad revision')
+            ) {
+                log.error(`❌ Git command failed ("${cmd}"):\n${errorMessage}`);
+                log.error(
+                    `🔍 Branch "${this.branch}" was not found locally. ` +
+                    `In CI this usually means a shallow checkout — fetch full history first, e.g. ` +
+                    `actions/checkout with "fetch-depth: 0", or run "git fetch origin ${this.branch}:${this.branch}".`
+                );
+                return [];
             }
 
-            return [];
+            throw new Error(`❌ Git command failed ("${cmd}"):\n${errorMessage}`);
         }
     }