@stanterprise/playwright-reporter 0.0.6 → 0.0.7

package/dist/handlers/onEndHandler.js

@@ -20,11 +20,21 @@ function mapTestStatus(status) {
     }
 }
 function handleOnEndEvent(result, runId, client, options) {
+    // Validate and sanitize inputs to prevent serialization failures
+    const startTimeMs = result.startTime?.getTime();
+    const duration = result.duration;
+    // Ensure valid numeric values (protobuf cannot serialize NaN or undefined)
+    const validStartTimeMs = typeof startTimeMs === "number" && !isNaN(startTimeMs)
+        ? startTimeMs
+        : Date.now();
+    const validDuration = typeof duration === "number" && !isNaN(duration) && duration >= 0
+        ? duration
+        : 0;
     const request = new events_1.TestRunEndEventRequest({
         run_id: runId,
         final_status: mapTestStatus(result.status),
-        start_time: (0, utils_1.createTimestampFromMs)(result.startTime.getTime()),
-        duration: (0, utils_1.createDuration)(result.duration),
+        start_time: (0, utils_1.createTimestampFromMs)(validStartTimeMs),
+        duration: (0, utils_1.createDuration)(validDuration),
     });
     (0, grpcClient_1.reportUnary)(options, client, "/testsystem.v1.observer.TestEventCollector/ReportEnd", request, options.grpcTimeout).catch((error) => {
         if (error instanceof Error) {

package/dist/utils/timeHelpers.js

@@ -12,15 +12,24 @@ const Timestamp = protobuf_1.google.google.protobuf.Timestamp;
  * Create a protobuf Timestamp from a JavaScript Date
  */
 function createTimestamp(date) {
+    const ms = date.getTime();
+    // Validate to prevent NaN from causing protobuf serialization failures
+    if (isNaN(ms)) {
+        throw new Error(`Invalid date provided to createTimestamp: ${date.toString()}`);
+    }
     return new Timestamp({
-        seconds: Math.floor(date.getTime() / 1000),
-        nanos: (date.getTime() % 1000) * 1000000,
+        seconds: Math.floor(ms / 1000),
+        nanos: (ms % 1000) * 1000000,
     });
 }
 /**
  * Create a protobuf Timestamp from milliseconds since epoch
  */
 function createTimestampFromMs(ms) {
+    // Validate to prevent NaN from causing protobuf serialization failures
+    if (typeof ms !== "number" || isNaN(ms)) {
+        throw new Error(`Invalid milliseconds provided to createTimestampFromMs: ${ms}`);
+    }
     return new Timestamp({
         seconds: Math.floor(ms / 1000),
         nanos: (ms % 1000) * 1000000,
@@ -38,6 +47,13 @@ function createTimestampFromMs(ms) {
  * @returns A Duration object (implemented as Timestamp due to export limitations)
  */
 function createDuration(durationMs) {
+    // Validate to prevent NaN from causing protobuf serialization failures
+    if (typeof durationMs !== "number" || isNaN(durationMs)) {
+        throw new Error(`Invalid duration provided to createDuration: ${durationMs}`);
+    }
+    if (durationMs < 0) {
+        throw new Error(`Negative duration provided to createDuration: ${durationMs}`);
+    }
     return new Timestamp({
         seconds: Math.floor(durationMs / 1000),
         nanos: (durationMs % 1000) * 1000000,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stanterprise/playwright-reporter",
-  "version": "0.0.6",
+  "version": "0.0.7",
   "description": "Custom Playwright test reporter that sends test results to Stanterprise via gRPC",
   "homepage": "https://github.com/stanterprise/stanterprise-playwright-reporter#readme",
   "bugs": {