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

package/package.json

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

package/src/bin/startTest.js

@@ -18,7 +18,7 @@ const newArgs = ['run'];
 let i = 0;
 while (i < args.length) {
   const arg = args[i];
-  
+
   if (arg === '-c' || arg === '--command') {
     // Map -c/--command to positional argument for run command
     i++;
@@ -33,7 +33,7 @@ while (i < args.length) {
     // Map --launch to start command
     newArgs[0] = 'start';
   } else if (arg === '--finish') {
-    // Map --finish to finish command  
+    // Map --finish to finish command
     newArgs[0] = 'finish';
   } else {
     // Pass through other arguments
@@ -45,9 +45,9 @@ while (i < args.length) {
 // Execute the main CLI with mapped arguments
 
 const child = spawn(process.execPath, [cliPath, ...newArgs], {
-  stdio: 'inherit'
+  stdio: 'inherit',
 });
 
-child.on('exit', (code) => {
+child.on('exit', code => {
   process.exit(code);
-});
+});

package/src/client.js

@@ -170,12 +170,12 @@ class Client {
       title,
       suite_title,
     } = testData;
-    const steps = originalSteps;
+    let steps = originalSteps;
 
     const uploadedFiles = [];
     const stackArtifactsEnabled = transformEnvVarToBoolean(process.env.TESTOMATIO_STACK_ARTIFACTS);
 
-    
+
     const {
       time = 0,
       example = null,
@@ -210,18 +210,18 @@ class Client {
 
     let fullLogs = this.formatLogs({ error: errorFormatted, steps, logs: testData.logs });
 
-    if (stackArtifactsEnabled) {
-      const timestamp = +new Date;
+    if (stackArtifactsEnabled && fullLogs?.trim()?.length > 0) {
       uploadedFiles.push(
         this.uploader.uploadFileAsBuffer(
           Buffer.from(stripColors(fullLogs), 'utf8'),
-          [this.runId, rid, `logs_${timestamp}.log`]
+          [this.runId, rid, `logs_${+new Date}.log`]
         )
       );
       fullLogs = '';
+      steps = null;
     }
 
-    
+
     if (!this.pipes || !this.pipes.length)
       this.pipes = await pipesFactory(this.paramsForPipesFactory || {}, this.pipeStore);
 
@@ -336,7 +336,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 +358,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/pipe/testomatio.js

@@ -472,7 +472,7 @@ class TestomatioPipe {
       if (this.runUrl && this.proceed) {
         const notFinishedMessage = pc.yellow(pc.bold('Run was not finished because of $TESTOMATIO_PROCEED'));
         console.log(APP_PREFIX, `📊 ${notFinishedMessage}. Report URL: ${pc.magenta(this.runUrl)}`);
-        console.log(APP_PREFIX, `🛬 Run to finish it: TESTOMATIO_RUN=${this.runId} npx start-test-run --finish`);
+        console.log(APP_PREFIX, `🛬 Run to finish it: TESTOMATIO_RUN=${this.runId} npx @testomatio/reporter finish`);
       }
 
       if (this.hasUnmatchedTests) {