@testomatio/reporter 2.7.9-beta.1-markdown → 2.7.9-beta.2-markdown

package/README.md

@@ -135,6 +135,7 @@ Bring this reporter on CI and never lose test results again!
   - [Gitlab](./docs/pipes/gitlab.md)
   - [CSV](./docs/pipes/csv.md)
   - [HTML report](./docs/pipes/html.md)
+  - [Markdown report](./docs/pipes/markdown.md)
   - [Bitbucket](./docs/pipes/bitbucket.md)
 - 🔗 [Linking Tests](./docs/linking-tests.md)
 - 📓 [JUnit](./docs/junit.md)

package/lib/client.js

@@ -199,6 +199,9 @@ class Client {
          */
         const { rid, error = null, steps: originalSteps, title, suite_title } = testData;
         let steps = originalSteps;
+        // Capture step artifact paths BEFORE uploadStepArtifacts mutates them to URLs,
+        // so we can exclude them from the test-level artifacts list later.
+        const stepArtifactPaths = collectStepArtifactPaths(steps);
         // Upload artifacts from steps
         try {
             await this.uploadStepArtifacts(steps, rid);
@@ -208,8 +211,14 @@ class Client {
         }
         const uploadedFiles = [];
         const stackArtifactsEnabled = (0, utils_js_1.transformEnvVarToBoolean)(process.env.TESTOMATIO_STACK_ARTIFACTS);
-        const { time = 0, example = null, files = [], filesBuffers = [], code = null, file, suite_id, test_id, timestamp, links, manuallyAttachedArtifacts, overwrite, tags, } = testData;
-        let { message = '', meta = {} } = testData;
+        const { time = 0, example = null, filesBuffers = [], code = null, file, suite_id, test_id, timestamp, links, overwrite, tags, } = testData;
+        let { files = [], manuallyAttachedArtifacts, message = '', meta = {} } = testData;
+        if (stepArtifactPaths.size) {
+            files = files.filter(f => !isStepArtifact(f, stepArtifactPaths));
+            if (Array.isArray(manuallyAttachedArtifacts)) {
+                manuallyAttachedArtifacts = manuallyAttachedArtifacts.filter(a => !isStepArtifact(a, stepArtifactPaths));
+            }
+        }
         meta = Object.entries(meta)
             .filter(([, value]) => value !== null && value !== undefined)
             .reduce((acc, [key, value]) => {
@@ -371,6 +380,47 @@ class Client {
     }
 }
 exports.Client = Client;
+/**
+ * Walks the step tree and returns the set of artifact path/url values
+ * referenced by `step.artifacts` at any depth.
+ *
+ * @param {any} steps
+ * @returns {Set<string>}
+ */
+function collectStepArtifactPaths(steps) {
+    const paths = new Set();
+    if (!Array.isArray(steps))
+        return paths;
+    const walk = arr => {
+        for (const step of arr) {
+            if (!step)
+                continue;
+            if (Array.isArray(step.artifacts)) {
+                for (const a of step.artifacts) {
+                    if (typeof a === 'string')
+                        paths.add(a);
+                    else if (a && typeof a === 'object' && typeof a.path === 'string')
+                        paths.add(a.path);
+                }
+            }
+            if (Array.isArray(step.steps))
+                walk(step.steps);
+        }
+    };
+    walk(steps);
+    return paths;
+}
+/**
+ * @param {string|{path?: string}|null|undefined} item
+ * @param {Set<string>} paths
+ * @returns {boolean}
+ */
+function isStepArtifact(item, paths) {
+    if (!item)
+        return false;
+    const p = typeof item === 'object' ? item.path : item;
+    return typeof p === 'string' && paths.has(p);
+}
 /**
  *
  * @param {TestData} testData

package/lib/pipe/markdown.js

@@ -111,7 +111,6 @@ class MarkdownPipe {
             executionDate: getCurrentDateTimeFormatted(),
             tests: aggregated,
             stats,
-            envVars: collectEnvironmentVariables(),
         };
         const md = renderDocument(data);
         fs_1.default.writeFileSync(outputPath, md, 'utf-8');
@@ -136,7 +135,6 @@ function renderDocument(data) {
     const sections = [];
     sections.push(renderHeader(data));
     sections.push(renderRunMetadata(data));
-    sections.push(renderEnvSection(data.envVars));
     sections.push(renderTests(data.tests));
     return sections.filter(Boolean).join('\n\n') + '\n';
 }
@@ -176,36 +174,6 @@ function renderRunMetadata(data) {
     }
     return lines.join('\n');
 }
-function renderEnvSection(envVars) {
-    if (!envVars)
-        return '';
-    const blocks = ['## Environment'];
-    const groups = [
-        { title: 'Testomat.io variables', vars: envVars.testomatio || {} },
-        { title: 'S3 variables', vars: envVars.s3 || {} },
-    ];
-    for (const group of groups) {
-        const entries = Object.entries(group.vars).filter(([, v]) => v && v.isSet);
-        if (!entries.length)
-            continue;
-        entries.sort((a, b) => a[0].localeCompare(b[0]));
-        const lines = [];
-        lines.push('<details>');
-        lines.push(`<summary>${group.title} (${entries.length})</summary>`);
-        lines.push('');
-        lines.push('| Variable | Value |');
-        lines.push('| -------- | ----- |');
-        for (const [name, info] of entries) {
-            lines.push(`| \`${name}\` | ${mdTableCell(String(info.value ?? ''))} |`);
-        }
-        lines.push('');
-        lines.push('</details>');
-        blocks.push(lines.join('\n'));
-    }
-    if (blocks.length === 1)
-        return '';
-    return blocks.join('\n\n');
-}
 function renderTests(tests) {
     if (!Array.isArray(tests) || tests.length === 0) {
         return '## Tests\n\n_No test results recorded._';
@@ -240,22 +208,27 @@ function renderTest(test) {
         displayStatus = 'todo';
     }
     const duration = formatStepDuration(test.run_time);
-    let header = `#### ${mdInline(title)}`;
-    if (duration)
-        header += ` — ${duration}`;
+    const header = `#### ${mdInline(title)}`;
     const lines = [header];
-    const meta = [`- **Status:** ${displayStatus}`];
+    const rows = [['Status', displayStatus]];
     const retries = computeRetries(test);
     if (retries.retryCount > 0) {
-        let retryLine = `- **Retries:** ${retries.retryCount}`;
+        let v = String(retries.retryCount);
         if (retries.flaky)
-            retryLine += ' (flaky)';
-        meta.push(retryLine);
+            v += ' (flaky)';
+        rows.push(['Retries', v]);
+    }
+    if (duration) {
+        rows.push(['Duration', duration]);
     }
     if (typeof test.test_id === 'string' && test.test_id) {
-        meta.push(`- **Test ID:** \`${test.test_id}\``);
+        rows.push(['Test ID', `\`${test.test_id}\``]);
     }
-    lines.push(meta.join('\n'));
+    const metaTable = ['| Key | Value |', '| --- | ----- |'];
+    for (const [k, v] of rows) {
+        metaTable.push(`| ${k} | ${v} |`);
+    }
+    lines.push(metaTable.join('\n'));
     const stepsBlock = renderSteps(test);
     if (stepsBlock)
         lines.push(stepsBlock);
@@ -285,10 +258,27 @@ function renderSteps(test) {
     }
     if (typeof steps === 'string' && steps.trim()) {
         const cleaned = steps.replace((0, utils_js_1.ansiRegExp)(), '').trim();
+        const parts = cleaned
+            .split(/<br\s*\/?>|\r?\n/i)
+            .map(s => s.trim())
+            .filter(Boolean);
+        if (parts.length > 1) {
+            const bullets = parts.map(line => formatStringStepBullet(line)).join('\n');
+            return `**Steps**\n\n${bullets}`;
+        }
         return `**Steps**\n\n${fence(cleaned)}`;
     }
     return '';
 }
+function formatStringStepBullet(line) {
+    const match = line.match(/^(.*?)[\s ]+(\d+)\s*ms\s*$/i);
+    if (match) {
+        const title = mdInline(match[1].trim());
+        const dur = `${match[2]}ms`;
+        return `- ${title} _(${dur})_`;
+    }
+    return `- ${mdInline(line)}`;
+}
 function renderStepTree(steps, depth) {
     const indent = '  '.repeat(depth);
     const lines = [];
@@ -400,7 +390,10 @@ function renderArtifacts(test) {
     }
     if (!items.length)
         return '';
-    const lines = ['**Artifacts**', ''];
+    const lines = [];
+    lines.push('<details>');
+    lines.push(`<summary><strong>Artifacts</strong> (${items.length})</summary>`);
+    lines.push('');
     for (const item of items) {
         if (item.isImage) {
             lines.push(`- ![${mdInline(item.name)}](${item.href})`);
@@ -409,6 +402,8 @@ function renderArtifacts(test) {
             lines.push(`- [${mdInline(item.name)}](${item.href})`);
         }
     }
+    lines.push('');
+    lines.push('</details>');
     return lines.join('\n');
 }
 function normalizeArtifact(raw) {
@@ -507,11 +502,6 @@ function mdInline(text) {
         return '';
     return String(text).replace(/\r?\n/g, ' ').trim();
 }
-function mdTableCell(text) {
-    if (text == null)
-        return '';
-    return String(text).replace(/\|/g, '\\|').replace(/\r?\n/g, '<br>').trim();
-}
 function formatStepDuration(value) {
     if (typeof value !== 'number' || Number.isNaN(value) || value <= 0)
         return '';
@@ -672,27 +662,4 @@ function aggregateTestRetries(tests) {
     });
     return aggregated;
 }
-const SENSITIVE_PATTERNS = [/TOKEN/, /SECRET/, /PASSWORD/, /KEY/, /^TESTOMATIO$/];
-function isSensitiveVarName(name) {
-    return SENSITIVE_PATTERNS.some(re => re.test(name));
-}
-function collectEnvironmentVariables() {
-    const groups = { testomatio: {}, s3: {} };
-    for (const [name, value] of Object.entries(process.env)) {
-        if (value === undefined)
-            continue;
-        let group = null;
-        if (name === 'TESTOMATIO' || name.startsWith('TESTOMATIO_'))
-            group = 'testomatio';
-        else if (name.startsWith('S3_'))
-            group = 's3';
-        if (!group)
-            continue;
-        let displayValue = value;
-        if (isSensitiveVarName(name))
-            displayValue = '***';
-        groups[group][name] = { value: displayValue, isSet: true };
-    }
-    return groups;
-}
 module.exports = MarkdownPipe;

package/package.json

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

package/src/client.js

@@ -215,6 +215,10 @@ class Client {
     const { rid, error = null, steps: originalSteps, title, suite_title } = testData;
     let steps = originalSteps;
 
+    // Capture step artifact paths BEFORE uploadStepArtifacts mutates them to URLs,
+    // so we can exclude them from the test-level artifacts list later.
+    const stepArtifactPaths = collectStepArtifactPaths(steps);
+
     // Upload artifacts from steps
     try {
       await this.uploadStepArtifacts(steps, rid);
@@ -228,7 +232,6 @@ class Client {
     const {
       time = 0,
       example = null,
-      files = [],
       filesBuffers = [],
       code = null,
       file,
@@ -236,11 +239,17 @@ class Client {
       test_id,
       timestamp,
       links,
-      manuallyAttachedArtifacts,
       overwrite,
       tags,
     } = testData;
-    let { message = '', meta = {} } = testData;
+    let { files = [], manuallyAttachedArtifacts, message = '', meta = {} } = testData;
+
+    if (stepArtifactPaths.size) {
+      files = files.filter(f => !isStepArtifact(f, stepArtifactPaths));
+      if (Array.isArray(manuallyAttachedArtifacts)) {
+        manuallyAttachedArtifacts = manuallyAttachedArtifacts.filter(a => !isStepArtifact(a, stepArtifactPaths));
+      }
+    }
 
     meta = Object.entries(meta)
       .filter(([, value]) => value !== null && value !== undefined)
@@ -450,6 +459,43 @@ class Client {
   }
 }
 
+/**
+ * Walks the step tree and returns the set of artifact path/url values
+ * referenced by `step.artifacts` at any depth.
+ *
+ * @param {any} steps
+ * @returns {Set<string>}
+ */
+function collectStepArtifactPaths(steps) {
+  const paths = new Set();
+  if (!Array.isArray(steps)) return paths;
+  const walk = arr => {
+    for (const step of arr) {
+      if (!step) continue;
+      if (Array.isArray(step.artifacts)) {
+        for (const a of step.artifacts) {
+          if (typeof a === 'string') paths.add(a);
+          else if (a && typeof a === 'object' && typeof a.path === 'string') paths.add(a.path);
+        }
+      }
+      if (Array.isArray(step.steps)) walk(step.steps);
+    }
+  };
+  walk(steps);
+  return paths;
+}
+
+/**
+ * @param {string|{path?: string}|null|undefined} item
+ * @param {Set<string>} paths
+ * @returns {boolean}
+ */
+function isStepArtifact(item, paths) {
+  if (!item) return false;
+  const p = typeof item === 'object' ? item.path : item;
+  return typeof p === 'string' && paths.has(p);
+}
+
 /**
  *
  * @param {TestData} testData

package/src/pipe/markdown.js

@@ -133,7 +133,6 @@ class MarkdownPipe {
       executionDate: getCurrentDateTimeFormatted(),
       tests: aggregated,
       stats,
-      envVars: collectEnvironmentVariables(),
     };
 
     const md = renderDocument(data);
@@ -163,7 +162,6 @@ function renderDocument(data) {
   const sections = [];
   sections.push(renderHeader(data));
   sections.push(renderRunMetadata(data));
-  sections.push(renderEnvSection(data.envVars));
   sections.push(renderTests(data.tests));
   return sections.filter(Boolean).join('\n\n') + '\n';
 }
@@ -211,41 +209,6 @@ function renderRunMetadata(data) {
   return lines.join('\n');
 }
 
-function renderEnvSection(envVars) {
-  if (!envVars) return '';
-
-  const blocks = ['## Environment'];
-
-  const groups = [
-    { title: 'Testomat.io variables', vars: envVars.testomatio || {} },
-    { title: 'S3 variables', vars: envVars.s3 || {} },
-  ];
-
-  for (const group of groups) {
-    const entries = Object.entries(group.vars).filter(([, v]) => v && v.isSet);
-    if (!entries.length) continue;
-
-    entries.sort((a, b) => a[0].localeCompare(b[0]));
-
-    const lines = [];
-    lines.push('<details>');
-    lines.push(`<summary>${group.title} (${entries.length})</summary>`);
-    lines.push('');
-    lines.push('| Variable | Value |');
-    lines.push('| -------- | ----- |');
-    for (const [name, info] of entries) {
-      lines.push(`| \`${name}\` | ${mdTableCell(String(info.value ?? ''))} |`);
-    }
-    lines.push('');
-    lines.push('</details>');
-
-    blocks.push(lines.join('\n'));
-  }
-
-  if (blocks.length === 1) return '';
-  return blocks.join('\n\n');
-}
-
 function renderTests(tests) {
   if (!Array.isArray(tests) || tests.length === 0) {
     return '## Tests\n\n_No test results recorded._';
@@ -286,23 +249,30 @@ function renderTest(test) {
   }
 
   const duration = formatStepDuration(test.run_time);
-  let header = `#### ${mdInline(title)}`;
-  if (duration) header += ` — ${duration}`;
+  const header = `#### ${mdInline(title)}`;
 
   const lines = [header];
 
-  const meta = [`- **Status:** ${displayStatus}`];
+  const rows = [['Status', displayStatus]];
 
   const retries = computeRetries(test);
   if (retries.retryCount > 0) {
-    let retryLine = `- **Retries:** ${retries.retryCount}`;
-    if (retries.flaky) retryLine += ' (flaky)';
-    meta.push(retryLine);
+    let v = String(retries.retryCount);
+    if (retries.flaky) v += ' (flaky)';
+    rows.push(['Retries', v]);
+  }
+  if (duration) {
+    rows.push(['Duration', duration]);
   }
   if (typeof test.test_id === 'string' && test.test_id) {
-    meta.push(`- **Test ID:** \`${test.test_id}\``);
+    rows.push(['Test ID', `\`${test.test_id}\``]);
   }
-  lines.push(meta.join('\n'));
+
+  const metaTable = ['| Key | Value |', '| --- | ----- |'];
+  for (const [k, v] of rows) {
+    metaTable.push(`| ${k} | ${v} |`);
+  }
+  lines.push(metaTable.join('\n'));
 
   const stepsBlock = renderSteps(test);
   if (stepsBlock) lines.push(stepsBlock);
@@ -335,12 +305,32 @@ function renderSteps(test) {
 
   if (typeof steps === 'string' && steps.trim()) {
     const cleaned = steps.replace(ansiRegExp(), '').trim();
+    const parts = cleaned
+      .split(/<br\s*\/?>|\r?\n/i)
+      .map(s => s.trim())
+      .filter(Boolean);
+
+    if (parts.length > 1) {
+      const bullets = parts.map(line => formatStringStepBullet(line)).join('\n');
+      return `**Steps**\n\n${bullets}`;
+    }
+
     return `**Steps**\n\n${fence(cleaned)}`;
   }
 
   return '';
 }
 
+function formatStringStepBullet(line) {
+  const match = line.match(/^(.*?)[\s ]+(\d+)\s*ms\s*$/i);
+  if (match) {
+    const title = mdInline(match[1].trim());
+    const dur = `${match[2]}ms`;
+    return `- ${title} _(${dur})_`;
+  }
+  return `- ${mdInline(line)}`;
+}
+
 function renderStepTree(steps, depth) {
   const indent = '  '.repeat(depth);
   const lines = [];
@@ -445,7 +435,10 @@ function renderArtifacts(test) {
 
   if (!items.length) return '';
 
-  const lines = ['**Artifacts**', ''];
+  const lines = [];
+  lines.push('<details>');
+  lines.push(`<summary><strong>Artifacts</strong> (${items.length})</summary>`);
+  lines.push('');
   for (const item of items) {
     if (item.isImage) {
       lines.push(`- ![${mdInline(item.name)}](${item.href})`);
@@ -453,6 +446,8 @@ function renderArtifacts(test) {
       lines.push(`- [${mdInline(item.name)}](${item.href})`);
     }
   }
+  lines.push('');
+  lines.push('</details>');
   return lines.join('\n');
 }
 
@@ -549,11 +544,6 @@ function mdInline(text) {
   return String(text).replace(/\r?\n/g, ' ').trim();
 }
 
-function mdTableCell(text) {
-  if (text == null) return '';
-  return String(text).replace(/\|/g, '\\|').replace(/\r?\n/g, '<br>').trim();
-}
-
 function formatStepDuration(value) {
   if (typeof value !== 'number' || Number.isNaN(value) || value <= 0) return '';
   if (value < 1000) return `${value}ms`;
@@ -714,30 +704,4 @@ function aggregateTestRetries(tests) {
   return aggregated;
 }
 
-const SENSITIVE_PATTERNS = [/TOKEN/, /SECRET/, /PASSWORD/, /KEY/, /^TESTOMATIO$/];
-
-function isSensitiveVarName(name) {
-  return SENSITIVE_PATTERNS.some(re => re.test(name));
-}
-
-function collectEnvironmentVariables() {
-  const groups = { testomatio: {}, s3: {} };
-
-  for (const [name, value] of Object.entries(process.env)) {
-    if (value === undefined) continue;
-
-    let group = null;
-    if (name === 'TESTOMATIO' || name.startsWith('TESTOMATIO_')) group = 'testomatio';
-    else if (name.startsWith('S3_')) group = 's3';
-    if (!group) continue;
-
-    let displayValue = value;
-    if (isSensitiveVarName(name)) displayValue = '***';
-
-    groups[group][name] = { value: displayValue, isSet: true };
-  }
-
-  return groups;
-}
-
 export default MarkdownPipe;