@testdino/playwright 1.0.2 → 1.0.4

package/dist/index.d.mts

@@ -338,9 +338,9 @@ interface TestdinoConfig {
      */
     token?: string;
     /**
-     * CI build ID for grouping sharded runs
+     * CI run ID for grouping sharded runs
      */
-    ciBuildId?: string;
+    ciRunId?: string;
     /**
      * Server URL (for development/testing)
      */
@@ -378,6 +378,7 @@ interface BaseEvent {
 interface TestRunBeginEvent extends BaseEvent {
     type: 'run:begin';
     metadata: CompleteMetadata;
+    ciRunId?: string;
 }
 interface TestBeginEvent extends BaseEvent {
     type: 'test:begin';

package/dist/index.d.ts

@@ -338,9 +338,9 @@ interface TestdinoConfig {
      */
     token?: string;
     /**
-     * CI build ID for grouping sharded runs
+     * CI run ID for grouping sharded runs
      */
-    ciBuildId?: string;
+    ciRunId?: string;
     /**
      * Server URL (for development/testing)
      */
@@ -378,6 +378,7 @@ interface BaseEvent {
 interface TestRunBeginEvent extends BaseEvent {
     type: 'run:begin';
     metadata: CompleteMetadata;
+    ciRunId?: string;
 }
 interface TestBeginEvent extends BaseEvent {
     type: 'test:begin';

package/dist/index.js

@@ -391,7 +391,7 @@ var EventBuffer = class {
   events = [];
   maxSize;
   onFlush;
-  isFlushing = false;
+  flushPromise = null;
   constructor(options) {
     this.maxSize = options.maxSize || 10;
     this.onFlush = options.onFlush || (async () => {
@@ -409,12 +409,30 @@ var EventBuffer = class {
   }
   /**
    * Flush all buffered events
+   * If a flush is already in progress, waits for it to complete then flushes any new events
    */
   async flush() {
-    if (this.isFlushing || this.events.length === 0) {
+    if (this.flushPromise) {
+      await this.flushPromise;
+      if (this.events.length > 0) {
+        return this.flush();
+      }
+      return;
+    }
+    if (this.events.length === 0) {
       return;
     }
-    this.isFlushing = true;
+    this.flushPromise = this.doFlush();
+    try {
+      await this.flushPromise;
+    } finally {
+      this.flushPromise = null;
+    }
+  }
+  /**
+   * Internal flush implementation
+   */
+  async doFlush() {
     try {
       const eventsToFlush = [...this.events];
       this.events = [];
@@ -422,8 +440,6 @@ var EventBuffer = class {
     } catch (error) {
       console.error("Failed to flush events:", error);
       throw error;
-    } finally {
-      this.isFlushing = false;
     }
   }
   /**
@@ -1783,7 +1799,7 @@ var TestdinoReporter = class {
         mappedConfig.debug = cliConfig.debug;
       }
       if (cliConfig.ciRunId !== void 0 && typeof cliConfig.ciRunId === "string") {
-        mappedConfig.ciBuildId = cliConfig.ciRunId;
+        mappedConfig.ciRunId = cliConfig.ciRunId;
       }
       if (cliConfig.artifacts !== void 0 && typeof cliConfig.artifacts === "boolean") {
         mappedConfig.artifacts = cliConfig.artifacts;
@@ -1885,8 +1901,23 @@ var TestdinoReporter = class {
         type: "run:begin",
         runId: this.runId,
         metadata,
+        ciRunId: this.config.ciRunId,
+        // Include ciRunId for shard grouping
         ...this.getEventMetadata()
       };
+      if (this.config.debug) {
+        console.log(`\u{1F50D} TestDino: run:begin event details:`);
+        console.log(`   runId: ${this.runId}`);
+        console.log(`   ciRunId: ${this.config.ciRunId ?? "(not set)"}`);
+        console.log(`   shard: ${config?.shard ? `${config.shard.current}/${config.shard.total}` : "(not sharded)"}`);
+        if (metadata.skeleton) {
+          console.log(`   skeleton.totalTests: ${metadata.skeleton.totalTests}`);
+          console.log(`   skeleton.suites: ${metadata.skeleton.suites?.length ?? 0}`);
+          console.log(`   skeleton: ${JSON.stringify(metadata.skeleton, null, 2)}`);
+        } else {
+          console.log(`   skeleton: (not available)`);
+        }
+      }
       await this.sendEvents([beginEvent]);
       this.registerSignalHandlers();
       return true;
@@ -2090,6 +2121,9 @@ var TestdinoReporter = class {
    */
   async onEnd(result) {
     if (this.quotaExceeded) {
+      if (this.pendingTestEndPromises.size > 0) {
+        await Promise.allSettled(Array.from(this.pendingTestEndPromises));
+      }
       console.log("\u2705 TestDino: Tests completed (quota limit reached; not streamed to TestDino)");
       this.wsClient?.close();
       this.removeSignalHandlers();
@@ -2579,7 +2613,7 @@ var TestdinoReporter = class {
         try {
           await Promise.race([
             Promise.allSettled(Array.from(this.pendingTestEndPromises)),
-            this.timeoutPromise(500, "Pending events timeout")
+            this.timeoutPromise(2e3, "Pending events timeout")
           ]);
         } catch {
         }