cucumberjs-qase-reporter 2.1.5 → 2.1.6

package/changelog.md

@@ -1,3 +1,9 @@
+# qase-cucumberjs@2.1.6
+
+## Bug fixes
+
+Fixed issue where cucumber-specific statuses like `AMBIGUOUS` were incorrectly mapped using generic status mapping instead of cucumber-specific mapping. Now cucumber-specific statuses are properly mapped according to cucumberjs conventions.
+
 # qase-cucumberjs@2.1.5
 
 ## What's new

package/dist/storage.js

@@ -151,7 +151,7 @@ class Storage {
             error = new Error(testCaseStep.testStepResult.message);
         }
         // Determine status based on error type
-        const newStatus = (0, qase_javascript_commons_1.determineTestStatus)(error, testCaseStep.testStepResult.status);
+        const newStatus = (0, qase_javascript_commons_1.determineTestStatus)(error, Storage.statusMap[testCaseStep.testStepResult.status]);
         this.testCaseSteps[testCaseStep.testStepId] = testCaseStep;
         if (newStatus !== qase_javascript_commons_1.TestStatusEnum.passed) {
             if (testCaseStep.testStepResult.message) {
@@ -161,7 +161,7 @@ class Storage {
                 this.testCaseStartedErrors[testCaseStep.testCaseStartedId]?.push(testCaseStep.testStepResult.message);
             }
             if (oldStatus) {
-                if (oldStatus !== qase_javascript_commons_1.TestStatusEnum.failed) {
+                if (oldStatus !== qase_javascript_commons_1.TestStatusEnum.failed && oldStatus !== qase_javascript_commons_1.TestStatusEnum.invalid) {
                     this.testCaseStartedResult[testCaseStep.testCaseStartedId] = newStatus;
                 }
             }
@@ -213,6 +213,7 @@ class Storage {
                 }
             }
         }
+        const steps = this.convertSteps(pickle.steps, tc);
         return {
             attachments: this.attachments[testCase.testCaseStartedId] ?? [],
             author: null,
@@ -232,7 +233,7 @@ class Storage {
             relations: relations,
             run_id: null,
             signature: this.getSignature(pickle, metadata.ids),
-            steps: this.convertSteps(pickle.steps, tc),
+            steps: steps,
             testops_id: metadata.ids.length > 0 ? metadata.ids : null,
             id: tcs.id,
             title: metadata.title ?? pickle.name,
@@ -285,7 +286,7 @@ class Storage {
         [cucumber_1.Status.PASSED]: qase_javascript_commons_1.TestStatusEnum.passed,
         [cucumber_1.Status.FAILED]: qase_javascript_commons_1.TestStatusEnum.failed,
         [cucumber_1.Status.SKIPPED]: qase_javascript_commons_1.TestStatusEnum.skipped,
-        [cucumber_1.Status.AMBIGUOUS]: qase_javascript_commons_1.TestStatusEnum.failed,
+        [cucumber_1.Status.AMBIGUOUS]: qase_javascript_commons_1.TestStatusEnum.invalid,
         [cucumber_1.Status.PENDING]: qase_javascript_commons_1.TestStatusEnum.skipped,
         [cucumber_1.Status.UNDEFINED]: qase_javascript_commons_1.TestStatusEnum.skipped,
         [cucumber_1.Status.UNKNOWN]: qase_javascript_commons_1.TestStatusEnum.skipped,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cucumberjs-qase-reporter",
-  "version": "2.1.5",
+  "version": "2.1.6",
   "description": "Qase TMS CucumberJS Reporter",
   "homepage": "https://github.com/qase-tms/qase-javascript",
   "main": "./dist/index.js",
@@ -40,7 +40,7 @@
   "license": "Apache-2.0",
   "dependencies": {
     "@cucumber/messages": "^22.0.0",
-    "qase-javascript-commons": "~2.4.2"
+    "qase-javascript-commons": "~2.4.8"
   },
   "peerDependencies": {
     "@cucumber/cucumber": ">=7.0.0"