cucumberjs-qase-reporter 2.1.4 → 2.1.6

package/changelog.md

@@ -1,3 +1,16 @@
+# 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
+
+- Added support for status filter in the test run.
+- Improved error handling.
+
 # qase-cucumberjs@2.1.3
 
 ## What's new

package/dist/storage.js

@@ -145,7 +145,13 @@ class Storage {
      */
     addTestCaseStep(testCaseStep) {
         const oldStatus = this.testCaseStartedResult[testCaseStep.testCaseStartedId];
-        const newStatus = Storage.statusMap[testCaseStep.testStepResult.status];
+        // Create error object for status determination
+        let error = null;
+        if (testCaseStep.testStepResult.message) {
+            error = new Error(testCaseStep.testStepResult.message);
+        }
+        // Determine status based on error type
+        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) {
@@ -155,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;
                 }
             }
@@ -207,6 +213,7 @@ class Storage {
                 }
             }
         }
+        const steps = this.convertSteps(pickle.steps, tc);
         return {
             attachments: this.attachments[testCase.testCaseStartedId] ?? [],
             author: null,
@@ -226,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,
@@ -279,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.4",
+  "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.1"
+    "qase-javascript-commons": "~2.4.8"
   },
   "peerDependencies": {
     "@cucumber/cucumber": ">=7.0.0"