@testomatio/reporter 1.6.7 → 1.6.8

package/lib/client.js

@@ -168,6 +168,7 @@ class Client {
     // stringify meta values and limit keys and values length to 255
     if (meta) {
       for (const key in meta) {
+        if (key === undefined || key === null) continue;
         if (typeof meta[key] === 'object') {
           meta[key] = JSON.stringify(meta[key]);
         }
@@ -332,9 +333,7 @@ class Client {
           console.log(
             '\n',
             APP_PREFIX,
-            `🗄️ ${chalk.bold(this.uploader.skippedUploads.length)} artifacts uploading 🟡${chalk.bold(
-              'skipped',
-            )} (due to large size)`,
+            `🗄️ ${chalk.bold(this.uploader.skippedUploads.length)} artifacts uploading 🟡${chalk.bold('skipped')}`,
           );
           const skippedUploads = this.uploader.skippedUploads.map(file => ({
             relativePath: file.path.replace(process.cwd(), ''),

package/lib/pipe/testomatio.js

@@ -371,7 +371,14 @@ class TestomatioPipe {
     if (!this.isEnabled) return;
 
     await this.#batchUpload();
-    if (this.batch.intervalFunction) clearInterval(this.batch.intervalFunction);
+    if (this.batch.intervalFunction) {
+      clearInterval(this.batch.intervalFunction);
+      // this code is required in case test is added after run is finished
+      // (e.g. if test has artifacts, add test function will be invoked only after artifacts are uploaded)
+      // batch stops working after run is finished; thus, disable it to use single test uploading
+      this.batch.intervalFunction = null;
+      this.batch.isEnabled = false;
+    }
 
     debug('Finishing run...');
 

package/lib/uploader.js

@@ -106,14 +106,14 @@ class S3Uploader {
 
     debug('Uploading to S3:', Key);
 
-    const s3Config = this.#getS3Config()
+    const s3Config = this.#getS3Config();
     const s3 = new S3(s3Config);
 
     const params = {
       Bucket: S3_BUCKET,
       Key,
       Body,
-    }
+    };
 
     // disable ACL for I AM roles
     if (!s3Config.credentials.sessionToken) {
@@ -206,6 +206,9 @@ class S3Uploader {
    * @returns
    */
   async uploadFileByPath(filePath, pathInS3) {
+    // sometimes artifacts uploading started before createRun function completion
+    this.isEnabled = this.isEnabled ?? this.checkEnabled();
+
     const [runId, rid] = pathInS3;
 
     if (!filePath) return;
@@ -248,7 +251,11 @@ class S3Uploader {
       this.storeUploadedFile(filePath, runId, rid, false);
       this.skippedUploads.push(skippedArtifact);
       debug(
-        chalk.yellow(`Artifacts file ${JSON.stringify(skippedArtifact)} exceeds the maximum allowed size. Skipping.`),
+        chalk.yellow(
+          `Artifacts file ${JSON.stringify(skippedArtifact)} exceeds the maximum allowed size ${
+            TESTOMATIO_ARTIFACT_MAX_SIZE_MB
+          }MB. Skipping.`,
+        ),
       );
       return;
     }

package/lib/utils/pipe_utils.js

@@ -15,8 +15,8 @@ function setS3Credentials(artifacts) {
   if (artifacts.BUCKET) process.env.S3_BUCKET = artifacts.BUCKET;
   if (artifacts.SESSION_TOKEN) process.env.S3_SESSION_TOKEN = artifacts.SESSION_TOKEN;
   if (artifacts.presign) process.env.TESTOMATIO_PRIVATE_ARTIFACTS = '1';
-   // endpoint is not received from the server; and shuld be empty if IAM used (credentails obtained from the testomat)
-   process.env.S3_ENDPOINT = artifacts.ENDPOINT || '';
+  // endpoint is not received from the server; and shuld be empty if IAM used (credentails obtained from the testomat)
+  process.env.S3_ENDPOINT = artifacts.ENDPOINT || '';
 }
 /**
  * Generates mode request parameters based on the input params.

package/lib/xmlReader.js

@@ -21,8 +21,8 @@ const config = require('./config');
 const ridRunId = randomUUID();
 
 const TESTOMATIO_URL = process.env.TESTOMATIO_URL || 'https://app.testomat.io';
-const { TESTOMATIO_RUNGROUP_TITLE, TESTOMATIO_TITLE, TESTOMATIO_ENV, TESTOMATIO_RUN,
-        TESTOMATIO_MARK_DETACHED } = process.env;
+const { TESTOMATIO_RUNGROUP_TITLE, TESTOMATIO_TITLE, TESTOMATIO_ENV, TESTOMATIO_RUN, TESTOMATIO_MARK_DETACHED } =
+  process.env;
 
 const options = {
   ignoreDeclaration: true,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testomatio/reporter",
-  "version": "1.6.7",
+  "version": "1.6.8",
   "description": "Testomatio Reporter Client",
   "main": "./lib/reporter.js",
   "typings": "typings/index.d.ts",