fix: async tests in Mocha

Fixed the issue with async tests not being reported correctly.
qase-tms · Aug 28, 2024 · 2586f87 · 2586f87
1 parent f5cad12
commit 2586f87
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/qase-mocha/changelog.md b/qase-mocha/changelog.md
@@ -1,3 +1,9 @@
+# [email protected]
+
+## What's new
+
+Fixed the issue with async tests not being reported correctly.
+
 # [email protected]
 
 ## What's new

diff --git a/qase-mocha/package.json b/qase-mocha/package.json
@@ -1,6 +1,6 @@
 {
   "name": "mocha-qase-reporter",
-  "version": "1.0.0-beta.3",
+  "version": "1.0.0-beta.4",
   "description": "Mocha Cypress Reporter",
   "homepage": "https://github.com/qase-tms/qase-javascript",
   "sideEffects": false,

diff --git a/qase-mocha/src/reporter.ts b/qase-mocha/src/reporter.ts
@@ -86,7 +86,8 @@ export class MochaQaseReporter extends reporters.Base {
     this.runner.on(Events.EVENT_HOOK_BEGIN, (hook: Hook) => this.addMethods(hook.ctx));
 
     this.runner.on(Events.EVENT_TEST_BEGIN, () => this.onStartTest());
-    this.runner.on(Events.EVENT_TEST_END, (test) => this.onEndTest(test));
+    // eslint-disable-next-line @typescript-eslint/no-misused-promises
+    this.runner.on(Events.EVENT_TEST_END, async (test) => await this.onEndTest(test));
   };
 
   private onStartRun() {
@@ -116,7 +117,7 @@ export class MochaQaseReporter extends reporters.Base {
     this.currentType = 'test';
   }
 
-  private onEndTest(test: Mocha.Test) {
+  private async onEndTest(test: Mocha.Test) {
     if (this.metadata.ignore) {
       return;
     }
@@ -172,7 +173,8 @@ export class MochaQaseReporter extends reporters.Base {
       title: this.metadata.title && this.metadata.title != '' ? this.metadata.title : test.title,
     };
 
-    deasyncPromise(this.reporter.addTestResult(result));
+    await this.reporter.addTestResult(result);
+
     this.metadata.clear();
     this.currentTest = new currentTest();
   }