cucumberjs-qase-reporter 2.4.0 → 2.4.2

package/changelog.md

@@ -1,3 +1,21 @@
+# cucumberjs-qase-reporter@2.4.2
+
+## Security
+
+- Bumped `uuid` from `^9.0.1` to `^11.1.1`, closing the advisory on uuid `v3/v5/v6` missing buffer bounds check that downstream consumers (e.g. Dependabot on user repos) saw via this reporter. Only `v4` is used here, so the advisory was not exploitable through this code path, but the version pin matters for users running `npm audit` on their projects.
+
+## Changed
+
+- `engines.node` raised from `>=14` to `>=18` to match uuid 11+ requirements; Node 14/16 are EOL.
+- Bumped `qase-javascript-commons` pin from `~2.6.0` to `~2.7.2` so users no longer pull the older registry copy of `commons` that depended on uuid 9.
+
+# cucumberjs-qase-reporter@2.4.1
+
+## Fixed
+
+- Gherkin step `execution.start_time` and `execution.end_time` are now populated from the Cucumber `testStepStarted` / `testStepFinished` envelope timestamps (Unix seconds with fractional ms). Previously both fields were hardcoded to `null`, breaking timeline rendering on the Qase side.
+- Step `execution.duration` now includes the `nanos` part of the Cucumber duration. Previously only `seconds * 1000` was emitted, so any sub-second step was reported as 0 ms.
+
 # cucumberjs-qase-reporter@2.4.0
 
 ## Changed

package/dist/modules/eventStorage.d.ts

@@ -1,10 +1,11 @@
-import { Attachment as Attach, GherkinDocument, Pickle, TestCaseStarted, TestStepFinished } from '@cucumber/messages';
+import { Attachment as Attach, GherkinDocument, Pickle, TestCaseStarted, TestStepFinished, TestStepStarted } from '@cucumber/messages';
 import { TestCase } from '@cucumber/messages/dist/esm/src/messages';
 import { Attachment } from 'qase-javascript-commons';
 import { ScenarioData } from '../models';
 export declare class EventStorage {
     private pickles;
     private testCaseStarts;
+    private testStepStarted;
     private testStepFinished;
     private testCases;
     private scenarios;
@@ -14,6 +15,7 @@ export declare class EventStorage {
     addAttachment(attachment: Attach): void;
     addTestCase(testCase: TestCase): void;
     addTestCaseStarted(testCaseStarted: TestCaseStarted): void;
+    addTestStepStarted(step: TestStepStarted): void;
     addTestStepFinished(step: TestStepFinished): void;
     getPickle(id: string): Pickle | undefined;
     getTestCase(id: string): TestCase | undefined;
@@ -22,6 +24,7 @@ export declare class EventStorage {
     getAttachments(key: string): Attachment[];
     getTestStepFinished(id: string): TestStepFinished | undefined;
     getAllTestStepFinished(): Map<string, TestStepFinished>;
+    getAllTestStepStarted(): Map<string, TestStepStarted>;
     private appendAttachment;
     static getFileNameFromMediaType(mediaType: string): string;
 }

package/dist/modules/eventStorage.js

@@ -5,6 +5,7 @@ const uuid_1 = require("uuid");
 class EventStorage {
     pickles = {};
     testCaseStarts = {};
+    testStepStarted = {};
     testStepFinished = {};
     testCases = {};
     scenarios = {};
@@ -55,6 +56,9 @@ class EventStorage {
     addTestCaseStarted(testCaseStarted) {
         this.testCaseStarts[testCaseStarted.id] = testCaseStarted;
     }
+    addTestStepStarted(step) {
+        this.testStepStarted[step.testStepId] = step;
+    }
     addTestStepFinished(step) {
         this.testStepFinished[step.testStepId] = step;
     }
@@ -79,6 +83,9 @@ class EventStorage {
     getAllTestStepFinished() {
         return new Map(Object.entries(this.testStepFinished));
     }
+    getAllTestStepStarted() {
+        return new Map(Object.entries(this.testStepStarted));
+    }
     appendAttachment(key, attachment) {
         const list = this.attachments[key] ?? [];
         this.attachments[key] = list;

package/dist/modules/stepConverter.d.ts

@@ -1,6 +1,11 @@
-import { PickleStep, TestStepFinished } from '@cucumber/messages';
+import { PickleStep, TestStepFinished, TestStepStarted } from '@cucumber/messages';
 import { TestCase } from '@cucumber/messages/dist/esm/src/messages';
 import { Attachment, TestStepType } from 'qase-javascript-commons';
 export declare class StepConverter {
-    static convert(pickleSteps: readonly PickleStep[], testCase: TestCase, finishedSteps: Map<string, TestStepFinished>, getAttachments: (stepId: string) => Attachment[]): TestStepType[];
+    static convert(pickleSteps: readonly PickleStep[], testCase: TestCase, startedSteps: Map<string, TestStepStarted>, finishedSteps: Map<string, TestStepFinished>, getAttachments: (stepId: string) => Attachment[]): TestStepType[];
+    /**
+     * Cucumber `Timestamp` is `{ seconds, nanos }`. Convert to a Unix-seconds
+     * number with fractional milliseconds.
+     */
+    private static toSeconds;
 }

package/dist/modules/stepConverter.js

@@ -4,7 +4,7 @@ exports.StepConverter = void 0;
 const qase_javascript_commons_1 = require("qase-javascript-commons");
 const statusMaps_1 = require("./statusMaps");
 class StepConverter {
-    static convert(pickleSteps, testCase, finishedSteps, getAttachments) {
+    static convert(pickleSteps, testCase, startedSteps, finishedSteps, getAttachments) {
         const results = [];
         for (const s of testCase.testSteps) {
             const finished = finishedSteps.get(s.id);
@@ -13,6 +13,9 @@ class StepConverter {
             const step = pickleSteps.find((ps) => ps.id === s.pickleStepId);
             if (!step)
                 continue;
+            const started = startedSteps.get(s.id);
+            const startTimeSec = started ? StepConverter.toSeconds(started.timestamp) : null;
+            const endTimeSec = StepConverter.toSeconds(finished.timestamp);
             results.push({
                 id: s.id,
                 step_type: qase_javascript_commons_1.StepType.GHERKIN,
@@ -23,9 +26,10 @@ class StepConverter {
                 },
                 execution: {
                     status: statusMaps_1.STEP_STATUS_MAP[finished.testStepResult.status],
-                    start_time: null,
-                    end_time: null,
-                    duration: finished.testStepResult.duration.seconds * 1000,
+                    start_time: startTimeSec,
+                    end_time: endTimeSec,
+                    duration: finished.testStepResult.duration.seconds * 1000
+                        + Math.round(finished.testStepResult.duration.nanos / 1e6),
                 },
                 attachments: getAttachments(s.id),
                 steps: [],
@@ -34,5 +38,12 @@ class StepConverter {
         }
         return results;
     }
+    /**
+     * Cucumber `Timestamp` is `{ seconds, nanos }`. Convert to a Unix-seconds
+     * number with fractional milliseconds.
+     */
+    static toSeconds(ts) {
+        return ts.seconds + ts.nanos / 1e9;
+    }
 }
 exports.StepConverter = StepConverter;

package/dist/reporter.js

@@ -78,6 +78,9 @@ class CucumberQaseReporter extends cucumber_1.Formatter {
             else if (envelope.testCaseStarted) {
                 this.storage.addTestCaseStarted(envelope.testCaseStarted);
             }
+            else if (envelope.testStepStarted) {
+                this.storage.addTestCaseStepStarted(envelope.testStepStarted);
+            }
             else if (envelope.testStepFinished) {
                 this.storage.addTestCaseStep(envelope.testStepFinished);
             }

package/dist/storage.d.ts

@@ -1,4 +1,4 @@
-import { Attachment as Attach, GherkinDocument, Pickle, TestCaseFinished, TestCaseStarted, TestStepFinished } from '@cucumber/messages';
+import { Attachment as Attach, GherkinDocument, Pickle, TestCaseFinished, TestCaseStarted, TestStepFinished, TestStepStarted } from '@cucumber/messages';
 import { StepStatusEnum, TestResultType, TestStatusEnum } from 'qase-javascript-commons';
 import { NetworkProfiler } from 'qase-javascript-commons/profilers';
 import { TestCase } from '@cucumber/messages/dist/esm/src/messages';
@@ -16,6 +16,7 @@ export declare class Storage {
     addAttachment(attachment: Attach): void;
     addTestCase(testCase: TestCase): void;
     addTestCaseStarted(testCaseStarted: TestCaseStarted): void;
+    addTestCaseStepStarted(testCaseStepStarted: TestStepStarted): void;
     addTestCaseStep(testCaseStep: TestStepFinished): void;
     convertTestCase(testCase: TestCaseFinished): undefined | TestResultType;
 }

package/dist/storage.js

@@ -37,6 +37,9 @@ class Storage {
         this.statusTracker.onTestStarted(testCaseStarted.id);
         this.profilerTracker.onTestStart(testCaseStarted.id);
     }
+    addTestCaseStepStarted(testCaseStepStarted) {
+        this.events.addTestStepStarted(testCaseStepStarted);
+    }
     addTestCaseStep(testCaseStep) {
         this.events.addTestStepFinished(testCaseStep);
         this.statusTracker.applyStep(testCaseStep);
@@ -67,7 +70,7 @@ class Storage {
                 }
             }
         }
-        const steps = stepConverter_1.StepConverter.convert(pickle.steps, tc, this.events.getAllTestStepFinished(), (stepId) => this.events.getAttachments(stepId));
+        const steps = stepConverter_1.StepConverter.convert(pickle.steps, tc, this.events.getAllTestStepStarted(), this.events.getAllTestStepFinished(), (stepId) => this.events.getAttachments(stepId));
         const profilerSteps = this.profilerTracker.getEvents(testCase.testCaseStartedId);
         this.profilerTracker.reset(testCase.testCaseStartedId);
         return resultBuilder_1.ResultBuilder.build({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cucumberjs-qase-reporter",
-  "version": "2.4.0",
+  "version": "2.4.2",
   "description": "Qase TMS CucumberJS Reporter",
   "homepage": "https://github.com/qase-tms/qase-javascript",
   "main": "./dist/index.js",
@@ -28,7 +28,7 @@
     "url": "git+https://github.com/qase-tms/qase-javascript.git"
   },
   "engines": {
-    "node": ">=14"
+    "node": ">=18"
   },
   "scripts": {
     "build": "npm run clean && tsc --project tsconfig.build.json",
@@ -40,7 +40,7 @@
   "license": "Apache-2.0",
   "dependencies": {
     "@cucumber/messages": "^22.0.0",
-    "qase-javascript-commons": "~2.6.0"
+    "qase-javascript-commons": "~2.7.2"
   },
   "peerDependencies": {
     "@cucumber/cucumber": ">=7.0.0"
@@ -50,6 +50,6 @@
     "@types/jest": "^29.5.14",
     "jest": "^29.7.0",
     "ts-jest": "^29.4.5",
-    "uuid": "^9.0.1"
+    "uuid": "^11.1.1"
   }
 }