@testomatio/reporter 2.14.0-beta → 2.14.0

package/lib/allureReader.d.ts

@@ -139,7 +139,7 @@ declare class AllureReader {
      * @returns {string|null}
      */
     extractTmsIdFromSource(contents: string, test: object): string | null;
-    convertSteps(steps: any, depth?: number): any;
+    convertSteps(steps: any, resultsDir?: string, depth?: number): any;
     /**
      * Check whether any step in the given (already converted) subtree already
      * carries an `error`. Used to keep the failure message on the deepest failed
@@ -211,6 +211,24 @@ declare class AllureReader {
      * @returns {string[]}
      */
     sourceCandidatesForTest(t: object, index: Map<string, string[]>): string[];
+    /**
+     * @param {Array<{source?: string}>|undefined} attachments
+     * @param {string} resultsDir
+     * @returns {string[]} paths of attachments that exist on disk
+     */
+    resolveAttachments(attachments: Array<{
+        source?: string;
+    }> | undefined, resultsDir: string): string[];
+    /**
+     * Replaces step artifact paths with S3 links, dropping failed uploads (a local path
+     * would be a dead link in the UI).
+     *
+     * @param {Array<object>|undefined} steps
+     * @param {string} runId
+     * @param {string} rid
+     * @returns {Promise<number>} number of uploaded step artifacts
+     */
+    uploadStepArtifacts(steps: Array<object> | undefined, runId: string, rid: string): Promise<number>;
     uploadArtifacts(): Promise<void>;
     uploadData(): Promise<any[]>;
 }

package/lib/allureReader.js

@@ -160,7 +160,7 @@ class AllureReader {
             suite_title: this.extractSuiteTitle(result),
             file: this.extractFile(result),
             run_time: this.calculateRunTime(result),
-            steps: this.convertSteps(result.steps || []),
+            steps: this.convertSteps(result.steps || [], resultsDir),
             message: result.statusDetails?.message || '',
             stack: result.statusDetails?.trace || '',
             meta: this.extractMeta(result),
@@ -182,20 +182,9 @@ class AllureReader {
         if (result.parameters && result.parameters.length > 0) {
             test.example = this.convertParameters(result.parameters);
         }
-        if (result.attachments && result.attachments.length > 0) {
-            const attachments = result.attachments
-                .map(att => {
-                const fullPath = path_1.default.join(resultsDir, att.source);
-                if (fs_1.default.existsSync(fullPath)) {
-                    return fullPath;
-                }
-                debug('Attachment file not found:', fullPath);
-                return null;
-            })
-                .filter(Boolean);
-            if (attachments.length > 0) {
-                test.files = attachments;
-            }
+        const attachments = this.resolveAttachments(result.attachments, resultsDir);
+        if (attachments.length > 0) {
+            test.files = attachments;
         }
         return test;
     }
@@ -479,7 +468,7 @@ class AllureReader {
     extractTmsIdFromSource(contents, test) {
         return this.extractTmsIdsFromSource(contents, test)[0] || null;
     }
-    convertSteps(steps, depth = 0) {
+    convertSteps(steps, resultsDir = '', depth = 0) {
         if (depth >= 10)
             return null;
         return steps
@@ -489,8 +478,13 @@ class AllureReader {
                 title: step.name || step.title || 'Unknown step',
                 status: this.mapStepStatus(step.status),
                 duration: this.calculateRunTime(step),
-                steps: this.convertSteps(step.steps || [], depth + 1),
+                steps: this.convertSteps(step.steps || [], resultsDir, depth + 1),
             };
+            // step attachments stay on the step; uploadArtifacts() swaps the paths for links
+            const attachments = this.resolveAttachments(step.attachments, resultsDir);
+            if (attachments.length > 0) {
+                convertedStep.artifacts = attachments;
+            }
             // Attach the failure description (error message + trace with the failing
             // code line) straight onto the failed step. Testomat.io renders a step's
             // `error` inline in the step tree, so the failure shows up on the exact
@@ -798,14 +792,62 @@ class AllureReader {
         }
         return paths;
     }
+    /**
+     * @param {Array<{source?: string}>|undefined} attachments
+     * @param {string} resultsDir
+     * @returns {string[]} paths of attachments that exist on disk
+     */
+    resolveAttachments(attachments, resultsDir) {
+        if (!attachments || !attachments.length)
+            return [];
+        return attachments
+            .map(att => {
+            if (!att?.source)
+                return null;
+            const fullPath = path_1.default.join(resultsDir || '', att.source);
+            if (fs_1.default.existsSync(fullPath))
+                return fullPath;
+            debug('Attachment file not found:', fullPath);
+            return null;
+        })
+            .filter(Boolean);
+    }
+    /**
+     * Replaces step artifact paths with S3 links, dropping failed uploads (a local path
+     * would be a dead link in the UI).
+     *
+     * @param {Array<object>|undefined} steps
+     * @param {string} runId
+     * @param {string} rid
+     * @returns {Promise<number>} number of uploaded step artifacts
+     */
+    async uploadStepArtifacts(steps, runId, rid) {
+        if (!steps || !steps.length)
+            return 0;
+        let uploaded = 0;
+        for (const step of steps) {
+            if (step.artifacts?.length) {
+                const links = await Promise.all(step.artifacts.map(f => this.uploader.uploadFileByPath(f, [runId, rid, 'steps', path_1.default.basename(f)])));
+                step.artifacts = links.filter(link => !!link);
+                uploaded += step.artifacts.length;
+                if (!step.artifacts.length)
+                    delete step.artifacts;
+            }
+            uploaded += await this.uploadStepArtifacts(step.steps, runId, rid);
+        }
+        return uploaded;
+    }
     async uploadArtifacts() {
-        for (const test of this._tests.filter(t => t.files && t.files.length > 0)) {
+        for (const test of this._tests) {
             const runId = this.runId || this.store.runId || Date.now().toString();
-            const artifacts = await Promise.all(test.files.map(f => this.uploader.uploadFileByPath(f, [runId, test.rid, path_1.default.basename(f)])));
-            test.artifacts = artifacts.filter(a => a && a.link).map(a => a.link);
+            // uploadFileByPath resolves to a link string, or undefined if skipped or failed
+            const links = await Promise.all((test.files || []).map(f => this.uploader.uploadFileByPath(f, [runId, test.rid, path_1.default.basename(f)])));
+            test.artifacts = links.filter(link => !!link);
             delete test.files;
-            if (test.artifacts.length > 0) {
-                console.log(constants_js_1.APP_PREFIX, `🗄️ Uploaded ${picocolors_1.default.bold(`${test.artifacts.length} artifacts`)} for test ${test.title}`);
+            const stepArtifacts = await this.uploadStepArtifacts(test.steps, runId, test.rid);
+            const total = test.artifacts.length + stepArtifacts;
+            if (total > 0) {
+                console.log(constants_js_1.APP_PREFIX, `🗄️ Uploaded ${picocolors_1.default.bold(`${total} artifacts`)} for test ${test.title}`);
             }
         }
     }

package/lib/bin/cli.js

@@ -53,7 +53,7 @@ program
 program
     .command('start')
     .description('Start a new run and return its ID')
-    .option('--kind <type>', 'Specify run type: automated, manual, or mixed')
+    .option('--kind <type>', 'Specify run type: automated, manual, mixed, or detect')
     .option('--filter <filter>', 'Scope the prepared run to tests matching the filter (no execution)')
     .option('--format <format>', 'Machine-readable output: print only the run id to stdout (e.g. --format id)')
     .option('--warn', 'Exit 0 instead of 1 when the filter matches no tests (warn only)')
@@ -121,7 +121,7 @@ program
     .option('--filter <filter>', 'Additional execution filter')
     .option('--filter-list <filter>', 'Get a list of all tests by filter before running')
     .option('--format <format>', 'Machine-readable output format for --filter-list (grep, json, newline, ids)')
-    .option('--kind <type>', 'Specify run type: automated, manual, or mixed')
+    .option('--kind <type>', 'Specify run type: automated, manual, mixed, or detect')
     .option('--remote <profile>', 'Trigger run on the named Testomat.io CI profile instead of executing locally')
     .option('--remote-param <kv>', 'key=value pair forwarded to the CI profile config (repeat for multiple)', (value, prev) => prev.concat([value]), [])
     .option('--warn', 'Exit 0 instead of 1 when the filter matches no tests (warn only)')

package/lib/pipe/html.d.ts

@@ -12,6 +12,7 @@ declare class HtmlPipe {
     filenameMsg: string;
     tests: any[];
     configuration: any;
+    startedAt: any;
     htmlReportDir: any;
     htmlReportName: string;
     templateFolderPath: string;

package/lib/pipe/html.js

@@ -31,6 +31,7 @@ class HtmlPipe {
         this.filenameMsg = '';
         this.tests = [];
         this.configuration = null;
+        this.startedAt = null;
         if (this.isHtml) {
             this.isEnabled = true;
             this.htmlReportDir = params.reportDir || process.env.TESTOMATIO_HTML_REPORT_FOLDER || constants_js_1.HTML_REPORT.FOLDER;
@@ -58,6 +59,7 @@ class HtmlPipe {
         }
     }
     async createRun(params = {}) {
+        this.startedAt ??= new Date();
         if (params?.configuration && typeof params.configuration === 'object') {
             this.configuration = { ...(this.configuration || {}), ...params.configuration };
         }
@@ -215,7 +217,7 @@ class HtmlPipe {
             parallel: runParams.isParallel || 'No parallel info',
             runUrl: this.store.runUrl || '',
             executionTime: testExecutionSumTime(aggregatedTests),
-            executionDate: getCurrentDateTimeFormatted(),
+            executionDate: getDateTimeFormatted(this.startedAt || new Date()),
             description: [this.description, runParams.description || this.store.coverageDescription || this.store.description]
                 .filter(Boolean)
                 .join('\n\n') || '',
@@ -599,17 +601,17 @@ function formatDuration(duration) {
     return `${hours}h ${minutes}m ${seconds}s ${milliseconds}ms`;
 }
 /**
- * Retrieves the current date and time in a formatted string.
+ * Formats a date and time for display in the report.
+ * @param {Date} date - Date and time to format.
  * @returns {string} - The formatted date and time string (e.g., "(01/01/2023 12:00:00)").
  */
-function getCurrentDateTimeFormatted() {
-    const currentDate = new Date();
-    const day = currentDate.getDate().toString().padStart(2, '0');
-    const month = (currentDate.getMonth() + 1).toString().padStart(2, '0');
-    const year = currentDate.getFullYear();
-    const hours = currentDate.getHours().toString().padStart(2, '0');
-    const minutes = currentDate.getMinutes().toString().padStart(2, '0');
-    const seconds = currentDate.getSeconds().toString().padStart(2, '0');
+function getDateTimeFormatted(date) {
+    const day = date.getDate().toString().padStart(2, '0');
+    const month = (date.getMonth() + 1).toString().padStart(2, '0');
+    const year = date.getFullYear();
+    const hours = date.getHours().toString().padStart(2, '0');
+    const minutes = date.getMinutes().toString().padStart(2, '0');
+    const seconds = date.getSeconds().toString().padStart(2, '0');
     return `(${day}/${month}/${year} ${hours}:${minutes}:${seconds})`;
 }
 /**

package/lib/xmlReader.js

@@ -459,8 +459,10 @@ class XmlReader {
             if (!files.length)
                 continue;
             const runId = this.runId || this.store.runId || Date.now().toString();
-            test.artifacts = await Promise.all(files.map(f => this.uploader.uploadFileByPath(f, [runId, path_1.default.basename(f)])));
-            log_js_1.log.info(`🗄️ Uploaded ${picocolors_1.default.bold(`${files.length} artifacts`)} for test ${test.title}`);
+            // undefined for skipped/failed uploads; keeping those serializes as `null` links
+            const links = await Promise.all(files.map(f => this.uploader.uploadFileByPath(f, [runId, path_1.default.basename(f)])));
+            test.artifacts = links.filter(link => !!link);
+            log_js_1.log.info(`🗄️ Uploaded ${picocolors_1.default.bold(`${test.artifacts.length} artifacts`)} for test ${test.title}`);
         }
     }
     async createRun() {

package/package.json

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

package/src/allureReader.js

@@ -183,7 +183,7 @@ class AllureReader {
       suite_title: this.extractSuiteTitle(result),
       file: this.extractFile(result),
       run_time: this.calculateRunTime(result),
-      steps: this.convertSteps(result.steps || []),
+      steps: this.convertSteps(result.steps || [], resultsDir),
       message: result.statusDetails?.message || '',
       stack: result.statusDetails?.trace || '',
       meta: this.extractMeta(result),
@@ -209,21 +209,9 @@ class AllureReader {
       test.example = this.convertParameters(result.parameters);
     }
 
-    if (result.attachments && result.attachments.length > 0) {
-      const attachments = result.attachments
-        .map(att => {
-          const fullPath = path.join(resultsDir, att.source);
-          if (fs.existsSync(fullPath)) {
-            return fullPath;
-          }
-          debug('Attachment file not found:', fullPath);
-          return null;
-        })
-        .filter(Boolean);
-
-      if (attachments.length > 0) {
-        test.files = attachments;
-      }
+    const attachments = this.resolveAttachments(result.attachments, resultsDir);
+    if (attachments.length > 0) {
+      test.files = attachments;
     }
 
     return test;
@@ -531,7 +519,7 @@ class AllureReader {
     return this.extractTmsIdsFromSource(contents, test)[0] || null;
   }
 
-  convertSteps(steps, depth = 0) {
+  convertSteps(steps, resultsDir = '', depth = 0) {
     if (depth >= 10) return null;
 
     return steps
@@ -541,9 +529,15 @@ class AllureReader {
           title: step.name || step.title || 'Unknown step',
           status: this.mapStepStatus(step.status),
           duration: this.calculateRunTime(step),
-          steps: this.convertSteps(step.steps || [], depth + 1),
+          steps: this.convertSteps(step.steps || [], resultsDir, depth + 1),
         };
 
+        // step attachments stay on the step; uploadArtifacts() swaps the paths for links
+        const attachments = this.resolveAttachments(step.attachments, resultsDir);
+        if (attachments.length > 0) {
+          convertedStep.artifacts = attachments;
+        }
+
         // Attach the failure description (error message + trace with the failing
         // code line) straight onto the failed step. Testomat.io renders a step's
         // `error` inline in the step tree, so the failure shows up on the exact
@@ -876,16 +870,68 @@ class AllureReader {
     return paths;
   }
 
+  /**
+   * @param {Array<{source?: string}>|undefined} attachments
+   * @param {string} resultsDir
+   * @returns {string[]} paths of attachments that exist on disk
+   */
+  resolveAttachments(attachments, resultsDir) {
+    if (!attachments || !attachments.length) return [];
+
+    return attachments
+      .map(att => {
+        if (!att?.source) return null;
+        const fullPath = path.join(resultsDir || '', att.source);
+        if (fs.existsSync(fullPath)) return fullPath;
+        debug('Attachment file not found:', fullPath);
+        return null;
+      })
+      .filter(Boolean);
+  }
+
+  /**
+   * Replaces step artifact paths with S3 links, dropping failed uploads (a local path
+   * would be a dead link in the UI).
+   *
+   * @param {Array<object>|undefined} steps
+   * @param {string} runId
+   * @param {string} rid
+   * @returns {Promise<number>} number of uploaded step artifacts
+   */
+  async uploadStepArtifacts(steps, runId, rid) {
+    if (!steps || !steps.length) return 0;
+
+    let uploaded = 0;
+    for (const step of steps) {
+      if (step.artifacts?.length) {
+        const links = await Promise.all(
+          step.artifacts.map(f => this.uploader.uploadFileByPath(f, [runId, rid, 'steps', path.basename(f)])),
+        );
+        step.artifacts = links.filter(link => !!link);
+        uploaded += step.artifacts.length;
+        if (!step.artifacts.length) delete step.artifacts;
+      }
+      uploaded += await this.uploadStepArtifacts(step.steps, runId, rid);
+    }
+    return uploaded;
+  }
+
   async uploadArtifacts() {
-    for (const test of this._tests.filter(t => t.files && t.files.length > 0)) {
+    for (const test of this._tests) {
       const runId = this.runId || this.store.runId || Date.now().toString();
-      const artifacts = await Promise.all(
-        test.files.map(f => this.uploader.uploadFileByPath(f, [runId, test.rid, path.basename(f)])),
+
+      // uploadFileByPath resolves to a link string, or undefined if skipped or failed
+      const links = await Promise.all(
+        (test.files || []).map(f => this.uploader.uploadFileByPath(f, [runId, test.rid, path.basename(f)])),
       );
-      test.artifacts = artifacts.filter(a => a && a.link).map(a => a.link);
+      test.artifacts = links.filter(link => !!link);
       delete test.files;
-      if (test.artifacts.length > 0) {
-        console.log(APP_PREFIX, `🗄️ Uploaded ${pc.bold(`${test.artifacts.length} artifacts`)} for test ${test.title}`);
+
+      const stepArtifacts = await this.uploadStepArtifacts(test.steps, runId, test.rid);
+
+      const total = test.artifacts.length + stepArtifacts;
+      if (total > 0) {
+        console.log(APP_PREFIX, `🗄️ Uploaded ${pc.bold(`${total} artifacts`)} for test ${test.title}`);
       }
     }
   }

package/src/bin/cli.js

@@ -51,7 +51,7 @@ program
 program
   .command('start')
   .description('Start a new run and return its ID')
-  .option('--kind <type>', 'Specify run type: automated, manual, or mixed')
+  .option('--kind <type>', 'Specify run type: automated, manual, mixed, or detect')
   .option('--filter <filter>', 'Scope the prepared run to tests matching the filter (no execution)')
   .option('--format <format>', 'Machine-readable output: print only the run id to stdout (e.g. --format id)')
   .option('--warn', 'Exit 0 instead of 1 when the filter matches no tests (warn only)')
@@ -130,7 +130,7 @@ program
   .option('--filter <filter>', 'Additional execution filter')
   .option('--filter-list <filter>', 'Get a list of all tests by filter before running')
   .option('--format <format>', 'Machine-readable output format for --filter-list (grep, json, newline, ids)')
-  .option('--kind <type>', 'Specify run type: automated, manual, or mixed')
+  .option('--kind <type>', 'Specify run type: automated, manual, mixed, or detect')
   .option('--remote <profile>', 'Trigger run on the named Testomat.io CI profile instead of executing locally')
   .option(
     '--remote-param <kv>',

package/src/pipe/html.js

@@ -32,6 +32,7 @@ class HtmlPipe {
     this.filenameMsg = '';
     this.tests = [];
     this.configuration = null;
+    this.startedAt = null;
 
     if (this.isHtml) {
       this.isEnabled = true;
@@ -70,6 +71,8 @@ class HtmlPipe {
   }
 
   async createRun(params = {}) {
+    this.startedAt ??= new Date();
+
     if (params?.configuration && typeof params.configuration === 'object') {
       this.configuration = { ...(this.configuration || {}), ...params.configuration };
     }
@@ -266,7 +269,7 @@ class HtmlPipe {
       parallel: runParams.isParallel || 'No parallel info',
       runUrl: this.store.runUrl || '',
       executionTime: testExecutionSumTime(aggregatedTests),
-      executionDate: getCurrentDateTimeFormatted(),
+      executionDate: getDateTimeFormatted(this.startedAt || new Date()),
       description:
         [this.description, runParams.description || this.store.coverageDescription || this.store.description]
           .filter(Boolean)
@@ -722,17 +725,17 @@ function formatDuration(duration) {
 }
 
 /**
- * Retrieves the current date and time in a formatted string.
+ * Formats a date and time for display in the report.
+ * @param {Date} date - Date and time to format.
  * @returns {string} - The formatted date and time string (e.g., "(01/01/2023 12:00:00)").
  */
-function getCurrentDateTimeFormatted() {
-  const currentDate = new Date();
-  const day = currentDate.getDate().toString().padStart(2, '0');
-  const month = (currentDate.getMonth() + 1).toString().padStart(2, '0');
-  const year = currentDate.getFullYear();
-  const hours = currentDate.getHours().toString().padStart(2, '0');
-  const minutes = currentDate.getMinutes().toString().padStart(2, '0');
-  const seconds = currentDate.getSeconds().toString().padStart(2, '0');
+function getDateTimeFormatted(date) {
+  const day = date.getDate().toString().padStart(2, '0');
+  const month = (date.getMonth() + 1).toString().padStart(2, '0');
+  const year = date.getFullYear();
+  const hours = date.getHours().toString().padStart(2, '0');
+  const minutes = date.getMinutes().toString().padStart(2, '0');
+  const seconds = date.getSeconds().toString().padStart(2, '0');
 
   return `(${day}/${month}/${year} ${hours}:${minutes}:${seconds})`;
 }

package/src/xmlReader.js

@@ -533,8 +533,10 @@ class XmlReader {
       if (!files.length) continue;
 
       const runId = this.runId || this.store.runId || Date.now().toString();
-      test.artifacts = await Promise.all(files.map(f => this.uploader.uploadFileByPath(f, [runId, path.basename(f)])));
-      log.info(`🗄️ Uploaded ${pc.bold(`${files.length} artifacts`)} for test ${test.title}`);
+      // undefined for skipped/failed uploads; keeping those serializes as `null` links
+      const links = await Promise.all(files.map(f => this.uploader.uploadFileByPath(f, [runId, path.basename(f)])));
+      test.artifacts = links.filter(link => !!link);
+      log.info(`🗄️ Uploaded ${pc.bold(`${test.artifacts.length} artifacts`)} for test ${test.title}`);
     }
   }
 

package/types/types.d.ts

@@ -337,8 +337,8 @@ export interface PipeResult {
  * `TESTOMATIO_CI_PROFILE` and `TESTOMATIO_CI_OVERRIDE`.
  */
 export interface CreateRunParams {
-  /** Run kind. Defaults to `automated` server-side. */
-  kind?: 'automated' | 'manual' | 'mixed';
+  /** Run kind. Defaults to `automated` server-side. `detect` resolves to one of the other three from the scoped tests. */
+  kind?: 'automated' | 'manual' | 'mixed' | 'detect';
 
   /** Run title. */
   title?: string;