cucumberjs-qase-reporter 2.1.5 → 2.1.7

package/changelog.md

@@ -1,3 +1,15 @@
+# qase-cucumberjs@2.1.7
+
+## What's new
+
+- Added support for QaseParameters and QaseGroupParameters tags.
+
+# 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/models.d.ts

@@ -3,6 +3,8 @@ export interface TestMetadata {
     fields: Record<string, string>;
     title: string | null;
     isIgnore: boolean;
+    parameters: Record<string, string>;
+    group_params: Record<string, string>;
 }
 export interface ScenarioData {
     name: string;

package/dist/storage.d.ts

@@ -100,6 +100,7 @@ export declare class Storage {
     /**
      * @param {Pickle} pickle
      * @param {number[]} ids
+     * @param {Record<string, string>} parameters
      * @private
      */
     private getSignature;

package/dist/storage.js

@@ -8,6 +8,8 @@ const qaseIdRegExp = /^@[Qq]-?(\d+)$/;
 const newQaseIdRegExp = /^@[Qq]ase[Ii][Dd]=(\d+(?:,\s*\d+)*)$/;
 const qaseTitleRegExp = /^@[Qq]ase[Tt]itle=(.+)$/;
 const qaseFieldsRegExp = /^@[Qq]ase[Ff]ields=(.+)$/;
+const qaseParametersRegExp = /^@[Qq]ase[Pp]arameters=(.+)$/;
+const qaseGroupParametersRegExp = /^@[Qq]ase[Gg]roup[Pp]arameters=(.+)$/;
 const qaseIgnoreRegExp = /^@[Qq]ase[Ii][Gg][Nn][Oo][Rr][Ee]$/;
 class Storage {
     /**
@@ -151,7 +153,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 +163,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 +215,10 @@ class Storage {
                 }
             }
         }
+        // Merge parameters from tags with parameters from Gherkin examples
+        // Parameters from tags take precedence over Gherkin examples
+        params = { ...params, ...metadata.parameters };
+        const steps = this.convertSteps(pickle.steps, tc);
         return {
             attachments: this.attachments[testCase.testCaseStartedId] ?? [],
             author: null,
@@ -228,11 +234,11 @@ class Storage {
             message: error?.message ?? null,
             muted: false,
             params: params,
-            group_params: {},
+            group_params: metadata.group_params,
             relations: relations,
             run_id: null,
-            signature: this.getSignature(pickle, metadata.ids),
-            steps: this.convertSteps(pickle.steps, tc),
+            signature: this.getSignature(pickle, metadata.ids, params),
+            steps: steps,
             testops_id: metadata.ids.length > 0 ? metadata.ids : null,
             id: tcs.id,
             title: metadata.title ?? pickle.name,
@@ -285,7 +291,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,
@@ -308,6 +314,8 @@ class Storage {
             fields: {},
             title: null,
             isIgnore: false,
+            parameters: {},
+            group_params: {},
         };
         for (const tag of tags) {
             if (qaseIdRegExp.test(tag.name)) {
@@ -333,6 +341,28 @@ class Storage {
                     // do nothing
                 }
             }
+            if (qaseParametersRegExp.test(tag.name)) {
+                const value = tag.name.replace(/^@[Qq]ase[Pp]arameters=/, '');
+                try {
+                    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+                    const record = JSON.parse(value);
+                    metadata.parameters = { ...metadata.parameters, ...record };
+                }
+                catch (e) {
+                    // do nothing
+                }
+            }
+            if (qaseGroupParametersRegExp.test(tag.name)) {
+                const value = tag.name.replace(/^@[Qq]ase[Gg]roup[Pp]arameters=/, '');
+                try {
+                    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+                    const record = JSON.parse(value);
+                    metadata.group_params = { ...metadata.group_params, ...record };
+                }
+                catch (e) {
+                    // do nothing
+                }
+            }
             if (qaseIgnoreRegExp.test(tag.name)) {
                 metadata.isIgnore = true;
             }
@@ -342,10 +372,11 @@ class Storage {
     /**
      * @param {Pickle} pickle
      * @param {number[]} ids
+     * @param {Record<string, string>} parameters
      * @private
      */
-    getSignature(pickle, ids) {
-        return (0, qase_javascript_commons_1.generateSignature)(ids, [...pickle.uri.split('/'), pickle.name], {});
+    getSignature(pickle, ids, parameters = {}) {
+        return (0, qase_javascript_commons_1.generateSignature)(ids, [...pickle.uri.split('/'), pickle.name], parameters);
     }
     getError(testCaseId) {
         const testErrors = this.testCaseStartedErrors[testCaseId];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cucumberjs-qase-reporter",
-  "version": "2.1.5",
+  "version": "2.1.7",
   "description": "Qase TMS CucumberJS Reporter",
   "homepage": "https://github.com/qase-tms/qase-javascript",
   "main": "./dist/index.js",
@@ -40,16 +40,16 @@
   "license": "Apache-2.0",
   "dependencies": {
     "@cucumber/messages": "^22.0.0",
-    "qase-javascript-commons": "~2.4.2"
+    "qase-javascript-commons": "~2.4.16"
   },
   "peerDependencies": {
     "@cucumber/cucumber": ">=7.0.0"
   },
   "devDependencies": {
-    "@jest/globals": "^29.5.0",
-    "@types/jest": "^29.5.2",
-    "jest": "^29.5.0",
-    "ts-jest": "^29.1.0",
-    "uuid": "^9.0.0"
+    "@jest/globals": "^29.7.0",
+    "@types/jest": "^29.5.14",
+    "jest": "^29.7.0",
+    "ts-jest": "^29.4.5",
+    "uuid": "^9.0.1"
   }
 }

package/tsconfig.build.json

@@ -2,7 +2,9 @@
   "extends": "./tsconfig.json",
 
   "compilerOptions": {
-    "noEmit": false
+    "noEmit": false,
+    "skipLibCheck": true,
+    "types": []
   },
 
   "include": ["./src/**/*.ts"]