@testomatio/reporter 2.3.7-beta.-stack-artifacts → 2.3.7-beta.10-stack-artifacts

package/lib/client.js

@@ -192,9 +192,11 @@ class Client {
             message = error?.message;
         }
         let fullLogs = this.formatLogs({ error: errorFormatted, steps, logs: testData.logs });
-        if (stackArtifactsEnabled) {
+        if (stackArtifactsEnabled && status && (steps || testData.logs || error)) {
             const timestamp = +new Date;
-            uploadedFiles.push(this.uploader.uploadFileAsBuffer(Buffer.from(stripColors(fullLogs), 'utf8'), [this.runId, rid, `logs_${timestamp}.log`]));
+            if (fullLogs && fullLogs.trim().length > 0) {
+                uploadedFiles.push(this.uploader.uploadFileAsBuffer(Buffer.from(stripColors(fullLogs), 'utf8'), [this.runId, rid, `logs_${timestamp}.txt`]));
+            }
             fullLogs = '';
         }
         if (!this.pipes || !this.pipes.length)
@@ -294,7 +296,7 @@ class Client {
                 const uploadedArtifacts = this.uploader.successfulUploads.map(file => ({
                     relativePath: file.path.replace(process.cwd(), ''),
                     link: file.link,
-                    sizePretty: (0, filesize_1.filesize)(file.size, { round: 0 }).toString(),
+                    sizePretty: file.size == null ? 'unknown' : (0, filesize_1.filesize)(file.size, { round: 0 }).toString(),
                 }));
                 uploadedArtifacts.forEach(upload => {
                     debug(`🟢Uploaded artifact`, `${upload.relativePath},`, 'size:', `${upload.sizePretty},`, 'link:', `${upload.link}`);
@@ -304,7 +306,7 @@ class Client {
                 console.log(constants_js_1.APP_PREFIX, `🗄️ ${this.uploader.failedUploads.length} artifacts 🔴${picocolors_1.default.bold('failed')} to upload`);
                 const failedUploads = this.uploader.failedUploads.map(file => ({
                     relativePath: file.path.replace(process.cwd(), ''),
-                    sizePretty: (0, filesize_1.filesize)(file.size, { round: 0 }).toString(),
+                    sizePretty: file.size == null ? 'unknown' : (0, filesize_1.filesize)(file.size, { round: 0 }).toString(),
                 }));
                 const pathPadding = Math.max(...failedUploads.map(upload => upload.relativePath.length)) + 1;
                 failedUploads.forEach(upload => {

package/lib/utils/utils.js

@@ -471,10 +471,14 @@ function transformEnvVarToBoolean(value) {
     return Boolean(value);
 }
 function truncate(s, size = 255) {
-    if (s.toString().trim().length < size) {
-        return s.toString();
+    if (s === undefined || s === null) {
+        return '';
+    }
+    const str = s.toString();
+    if (str.trim().length < size) {
+        return str;
     }
-    return `${s.toString().substring(0, size)}...`;
+    return `${str.substring(0, size)}...`;
 }
 
 module.exports.getPackageVersion = getPackageVersion;

package/package.json

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

package/src/client.js

@@ -210,14 +210,16 @@ class Client {
 
     let fullLogs = this.formatLogs({ error: errorFormatted, steps, logs: testData.logs });
 
-    if (stackArtifactsEnabled) {
+    if (stackArtifactsEnabled && status && (steps || testData.logs || error)) {
       const timestamp = +new Date;
-      uploadedFiles.push(
-        this.uploader.uploadFileAsBuffer(
-          Buffer.from(stripColors(fullLogs), 'utf8'),
-          [this.runId, rid, `logs_${timestamp}.log`]
-        )
-      );
+      if (fullLogs && fullLogs.trim().length > 0) {
+        uploadedFiles.push(
+          this.uploader.uploadFileAsBuffer(
+            Buffer.from(stripColors(fullLogs), 'utf8'),
+            [this.runId, rid, `logs_${timestamp}.txt`]
+          )
+        );
+      }
       fullLogs = '';
     }
 
@@ -336,7 +338,7 @@ class Client {
           const uploadedArtifacts = this.uploader.successfulUploads.map(file => ({
             relativePath: file.path.replace(process.cwd(), ''),
             link: file.link,
-            sizePretty: prettyBytes(file.size, { round: 0 }).toString(),
+            sizePretty: file.size == null ? 'unknown' : prettyBytes(file.size, { round: 0 }).toString(),
           }));
 
           uploadedArtifacts.forEach(upload => {
@@ -358,7 +360,7 @@ class Client {
           );
           const failedUploads = this.uploader.failedUploads.map(file => ({
             relativePath: file.path.replace(process.cwd(), ''),
-            sizePretty: prettyBytes(file.size, { round: 0 }).toString(),
+            sizePretty: file.size == null ? 'unknown' : prettyBytes(file.size, { round: 0 }).toString(),
           }));
 
           const pathPadding = Math.max(...failedUploads.map(upload => upload.relativePath.length)) + 1;

package/src/utils/utils.js

@@ -429,10 +429,14 @@ function transformEnvVarToBoolean(value) {
 }
 
 function truncate(s, size = 255) {
-  if (s.toString().trim().length < size) {
-    return s.toString();
+  if (s === undefined || s === null) {
+    return '';
   }
-  return `${s.toString().substring(0, size)}...`;
+  const str = s.toString();
+  if (str.trim().length < size) {
+    return str;
+  }
+  return `${str.substring(0, size)}...`;
 }
 
 export {