qase-javascript-commons 2.2.11 → 2.2.12

package/changelog.md

@@ -1,3 +1,10 @@
+# qase-javascript-commons@2.2.12
+
+## What's new
+
+Added a mutex to ensure correct result submission when running tests in multiple threads, preventing potential
+duplication.
+
 # qase-javascript-commons@2.2.11
 
 ## What's new

package/dist/reporters/testops-reporter.d.ts

@@ -102,6 +102,7 @@ export declare class TestOpsReporter extends AbstractReporter {
      * @private
      */
     private isTestRunReady;
+    private mutex;
     /**
      * @param {LoggerInterface} logger
      * @param {ReporterOptionsType & TestOpsOptionsType} options

package/dist/reporters/testops-reporter.js

@@ -12,6 +12,7 @@ const models_1 = require("../models");
 const qase_error_1 = require("../utils/qase-error");
 const axios_1 = __importDefault(require("axios"));
 const state_1 = require("../state/state");
+const async_mutex_1 = require("async-mutex");
 const defaultChunkSize = 200;
 /**
  * @class TestOpsReporter
@@ -109,6 +110,7 @@ class TestOpsReporter extends abstract_reporter_1.AbstractReporter {
      * @private
      */
     isTestRunReady = false;
+    mutex = new async_mutex_1.Mutex();
     /**
      * @param {LoggerInterface} logger
      * @param {ReporterOptionsType & TestOpsOptionsType} options
@@ -151,15 +153,21 @@ class TestOpsReporter extends abstract_reporter_1.AbstractReporter {
                 this.showLink(id, result.title);
             }
         }
-        await super.addTestResult(result);
-        if (!this.isTestRunReady) {
-            return;
+        const release = await this.mutex.acquire();
+        try {
+            await super.addTestResult(result);
+            if (!this.isTestRunReady) {
+                return;
+            }
+            const countOfResults = this.batchSize + this.firstIndex;
+            if (this.results.length >= countOfResults) {
+                const firstIndex = this.firstIndex;
+                this.firstIndex = countOfResults;
+                await this.publishResults(this.results.slice(firstIndex, countOfResults));
+            }
         }
-        const countOfResults = this.batchSize + this.firstIndex;
-        if (this.results.length >= countOfResults) {
-            const firstIndex = this.firstIndex;
-            this.firstIndex = countOfResults;
-            await this.publishResults(this.results.slice(firstIndex, countOfResults));
+        finally {
+            release();
         }
     }
     /**
@@ -238,7 +246,13 @@ class TestOpsReporter extends abstract_reporter_1.AbstractReporter {
      * @returns {Promise<void>}
      */
     async publish() {
-        await this.sendResults();
+        const release = await this.mutex.acquire();
+        try {
+            await this.sendResults();
+        }
+        finally {
+            release();
+        }
         await this.complete();
     }
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "qase-javascript-commons",
-  "version": "2.2.11",
+  "version": "2.2.12",
   "description": "Qase JS Reporters",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
@@ -34,7 +34,8 @@
     "mime-types": "^2.1.33",
     "qaseio": "~2.4.0",
     "strip-ansi": "^6.0.1",
-    "uuid": "^9.0.0"
+    "uuid": "^9.0.0",
+    "async-mutex": "~0.5.0"
   },
   "devDependencies": {
     "@jest/globals": "^29.5.0",