@wdio/browserstack-service 9.36.0 → 9.36.2

package/build/cli/modules/testHubModule.d.ts

@@ -19,8 +19,23 @@ export default class TestHubModule extends BaseModule {
      * slot with a fresh instance for the next test — so it stays valid across tests.
      * Flushed at the next test's first event (INIT_TEST / TEST PRE) or, for the worker's
      * last test, from service.after() via flushPendingTestFinishEvent().
+     *
+     * SDK-7493: keyed by test uuid, NOT a single slot. wdio does not await `onTestSkip`, so a
+     * skip report runs detached and can interleave with a live test — its events land between
+     * the live test's own, on the same per-worker tracked-instance context. With one slot the
+     * second stash silently REPLACED the first (the old guard only flushed when the instance
+     * OBJECT differed, and an interleave can present the same object), so one test's
+     * TestRunFinished was never sent and TRA left it rendering "In Progress" until the ~60-min
+     * idle reap. A map cannot evict: every deferred finish is delivered, each under its own uuid.
+     *
+     * NOTE ON REDUNDANCY: SDK-7493's queue-and-drain (skipReporter) already stops a skip report
+     * running while a test is in flight, so on the normal path that collision can no longer be
+     * triggered and this map is not the primary fix. It is retained deliberately as
+     * defence-in-depth for a reporting path with a real incident history (SDK-7265, SDK-7493,
+     * ~60-min reaps): if any future caller reintroduces an interleave, the worst case degrades
+     * to a late send rather than a silently lost TestRunFinished.
      */
-    private pendingTestFinish;
+    private pendingTestFinishes;
     /**
      * Create a new TestHubModule
      */
@@ -43,6 +58,7 @@ export default class TestHubModule extends BaseModule {
     sendTestFrameworkEvent(args: Record<string, unknown>, stateOverride?: {
         testFrameworkState: string;
         testHookState: string;
+        uuid?: string;
     }): Promise<boolean>;
     /**
      * Send test session event to the service

package/build/cli/skipReporter.d.ts

@@ -2,11 +2,18 @@ import type { Frameworks } from '@wdio/types';
 import type TestFramework from './frameworks/testFramework.js';
 export declare function markTestStarted(identifier: string): void;
 export declare function drainSkipReports(): Promise<void>;
-export declare function reportSkippedTest(framework: TestFramework, identifier: string, test: Frameworks.Test, suiteTitle?: string): Promise<void>;
+export declare function reportSkippedTest(framework: TestFramework, identifier: string, test: Frameworks.Test, suiteTitle?: string, options?: {
+    immediate?: boolean;
+}): Promise<void>;
 /**
  * Port of the legacy insights-handler skip propagation: when a BEFORE_ALL/BEFORE_EACH/
  * AFTER_EACH hook fails (or skips), mocha silently drops the remaining tests in the
  * suite — report each state-undefined test as skipped, recursing into nested describes.
+ *
+ * Reports IMMEDIATELY (SDK-7493): every caller of this — the failed-hook cascade in
+ * `afterHook` and the bail cascade in `afterTest` — is awaited by wdio, so these reports
+ * cannot interleave with a live test the way the un-awaited `onTestSkip` path could. They
+ * belong to the hook/test being reported, so they must not slide to end-of-run.
  */
 export declare function reportSuiteSkipped(framework: TestFramework, suite: {
     tests?: unknown[];