fix multiple tests reusing suite names

kolodny · Jan 25, 2024 · 867e238 · 867e238
1 parent 3950ee2
commit 867e238
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/src/blocks.ts b/src/blocks.ts
@@ -2,7 +2,7 @@
 import { state } from './state';
 import { isInNode } from './is-in-node';
 
-const seen = new Set<string>();
+const seen = state.seen;
 const createId = (name: string, parent: string) => {
   const separator = parent ? ' ' : '';
   const concat = `${parent}${separator}${name}`;

diff --git a/src/bootstrap.ts b/src/bootstrap.ts
@@ -66,6 +66,7 @@ export const bootstrap = async (args: BootstrapArgs): Promise<any> => {
       Promise.resolve().then(async () => {
         for (const k of known) {
           state.tests = {};
+          state.seen.clear();
           await importer(k);
           await (window as any).waitForSafetestReady;
           const tests = Object.keys(state.tests);

diff --git a/src/state.ts b/src/state.ts
@@ -9,6 +9,7 @@ export interface State {
   __filename?: string;
   activeTest?: string;
   pendingExpects: Record<string, number>;
+  seen: Set<string>;
   options: RenderOptions;
   isGlobalSetupTeardownRegistered: boolean;
   debugging: Set<string>;
@@ -55,6 +56,7 @@ export const state: State = {
   tests: {},
   retryMap: {},
   options: {},
+  seen: new Set(),
   currentSuite: '',
   currentTest: '',
   isGlobalSetupTeardownRegistered: false,