@testomatio/reporter 2.7.9-beta.3-markdown → 2.8.0

package/lib/client.js

@@ -214,9 +214,10 @@ class Client {
         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));
+            const isStepArtifact = item => stepArtifactPaths.has(typeof item === 'object' ? item?.path : item);
+            files = files.filter(f => !isStepArtifact(f));
             if (Array.isArray(manuallyAttachedArtifacts)) {
-                manuallyAttachedArtifacts = manuallyAttachedArtifacts.filter(a => !isStepArtifact(a, stepArtifactPaths));
+                manuallyAttachedArtifacts = manuallyAttachedArtifacts.filter(a => !isStepArtifact(a));
             }
         }
         meta = Object.entries(meta)
@@ -385,42 +386,27 @@ exports.Client = Client;
  * referenced by `step.artifacts` at any depth.
  *
  * @param {any} steps
+ * @param {Set<string>} [paths]
  * @returns {Set<string>}
  */
-function collectStepArtifactPaths(steps) {
-    const paths = new Set();
+function collectStepArtifactPaths(steps, 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);
-                }
+    for (const step of steps) {
+        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);
+        collectStepArtifactPaths(step.steps, paths);
+    }
     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/package.json

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

package/src/client.js

@@ -245,9 +245,10 @@ class Client {
     let { files = [], manuallyAttachedArtifacts, message = '', meta = {} } = testData;
 
     if (stepArtifactPaths.size) {
-      files = files.filter(f => !isStepArtifact(f, stepArtifactPaths));
+      const isStepArtifact = item => stepArtifactPaths.has(typeof item === 'object' ? item?.path : item);
+      files = files.filter(f => !isStepArtifact(f));
       if (Array.isArray(manuallyAttachedArtifacts)) {
-        manuallyAttachedArtifacts = manuallyAttachedArtifacts.filter(a => !isStepArtifact(a, stepArtifactPaths));
+        manuallyAttachedArtifacts = manuallyAttachedArtifacts.filter(a => !isStepArtifact(a));
       }
     }
 
@@ -464,38 +465,24 @@ class Client {
  * referenced by `step.artifacts` at any depth.
  *
  * @param {any} steps
+ * @param {Set<string>} [paths]
  * @returns {Set<string>}
  */
-function collectStepArtifactPaths(steps) {
-  const paths = new Set();
+function collectStepArtifactPaths(steps, 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);
-        }
+  for (const step of steps) {
+    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);
+    collectStepArtifactPaths(step.steps, paths);
+  }
   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