testit-adapter-mocha 3.7.9 → 4.0.0

package/lib/reporter.d.ts

@@ -6,6 +6,7 @@ declare const Reporter: typeof reporters.Base;
 export declare class TmsReporter extends Reporter {
     private readonly strategy;
     private readonly additions;
+    private isRunEnded;
     private attachmentsQueue;
     private autotestsQueue;
     private autotestsForTestRun;

package/lib/reporter.js

@@ -11,6 +11,8 @@ const step_1 = require("./step");
 const utils_1 = require("./utils");
 const Reporter = mocha_1.reporters.Base;
 const Events = mocha_1.Runner.constants;
+// eslint-disable-next-line @typescript-eslint/no-var-requires
+const superagent = require("superagent");
 const emptyTest = () => ({
     autoTestExternalId: "",
     outcome: "Passed",
@@ -29,9 +31,29 @@ const StateConstants = {
     STATE_PASSED: 'passed',
     STATE_PENDING: 'pending',
 };
+let superagentCallbackGuardInstalled = false;
+const installSuperagentDoubleCallbackGuard = () => {
+    if (superagentCallbackGuardInstalled) {
+        return;
+    }
+    superagentCallbackGuardInstalled = true;
+    const RequestProto = superagent.Request?.prototype;
+    const originalCallback = RequestProto?.callback;
+    if (typeof originalCallback !== "function") {
+        return;
+    }
+    RequestProto.callback = function guardedCallback(...args) {
+        if (this.__tmsCallbackDone) {
+            return;
+        }
+        this.__tmsCallbackDone = true;
+        return originalCallback.apply(this, args);
+    };
+};
 class TmsReporter extends Reporter {
     constructor(runner, options) {
         super(runner, options);
+        this.isRunEnded = false;
         this.attachmentsQueue = [];
         this.autotestsQueue = [];
         this.autotestsForTestRun = [];
@@ -46,10 +68,18 @@ class TmsReporter extends Reporter {
             this.runner.on(Events.EVENT_TEST_END, (test) => this.onEndTest(test));
         };
         this.onStartRun = () => {
-            this.strategy.setup();
+            (0, deasync_promise_1.default)(this.strategy.setup().catch((err) => {
+                console.log("Error setup test run. \n", err?.body ?? err);
+            }));
         };
         this.onEndRun = () => {
-            (0, deasync_promise_1.default)(this.teardown());
+            if (this.isRunEnded) {
+                return;
+            }
+            this.isRunEnded = true;
+            (0, deasync_promise_1.default)(this.teardown().catch((err) => {
+                console.log("Error during teardown. \n", err?.body ?? err);
+            }));
         };
         this.addAttachments = (pathsOrContent, fileName) => {
             const target = this.currentType === "test" ? this.currentTest : this.currentStep;
@@ -58,6 +88,8 @@ class TmsReporter extends Reporter {
                 : this.additions.addAttachments(pathsOrContent);
             promise.then((attachments) => {
                 target.attachments?.push(...attachments);
+            }).catch((err) => {
+                console.log("Error loading attachment. \n", err?.body ?? err?.error ?? err);
             });
             this.attachmentsQueue.push(promise);
             return promise;
@@ -92,6 +124,7 @@ class TmsReporter extends Reporter {
             this.currentType = prevType;
             this.currentStep = emptyStep();
         };
+        installSuperagentDoubleCallbackGuard();
         const config = new testit_js_commons_1.ConfigComposer().compose(options?.tmsOptions);
         this.strategy = testit_js_commons_1.StrategyFactory.create(config);
         this.additions = new testit_js_commons_1.Additions(config);
@@ -104,13 +137,17 @@ class TmsReporter extends Reporter {
     }
     async teardown() {
         await Promise.all(this.attachmentsQueue).catch((err) => {
-            console.log("Error loading attachments. \n", err.body);
+            console.log("Error loading attachments. \n", err?.body ?? err);
+        });
+        await Promise.all(this.autotestsQueue).catch((err) => {
+            console.log("Error load autotests. \n", err?.body ?? err);
         });
-        await Promise.all(this.autotestsQueue);
         await this.strategy.loadTestRun(this.autotestsForTestRun).catch((err) => {
-            console.log("Error load test run. \n", err.body);
+            console.log("Error load test run. \n", err?.body ?? err);
+        });
+        await this.strategy.teardown().catch((err) => {
+            console.log("Error complete test run. \n", err?.body ?? err);
         });
-        await this.strategy.teardown();
     }
     clearContext(ctx) {
         ctx.externalId = undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testit-adapter-mocha",
-  "version": "3.7.9",
+  "version": "4.0.0",
   "description": "Mocha adapter for Test IT",
   "main": "lib/index.js",
   "types": "lib/types.d.ts",
@@ -31,7 +31,7 @@
   "dependencies": {
     "mocha": "^10.2.0",
     "deasync-promise": "^1.0.1",
-    "testit-js-commons": "3.7.9"
+    "testit-js-commons": "4.0.0"
   },
   "files": [
     "lib"