fix: incorrectly setting the name of currently running concurrent test (

#14335)
jestjs · Jul 21, 2023 · 7628207 · 7628207
1 parent 10bfa61
commit 7628207
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,7 @@
 
 ### Fixes
 
+- `[jest-circus]` Fix snapshot matchers in concurrent tests when nr of tests exceeds `maxConcurrency` ([#14335](https://github.com/jestjs/jest/pull/14335))
 - `[jest-snapshot]` Move `@types/prettier` from `dependencies` to `devDependencies` ([#14328](https://github.com/jestjs/jest/pull/14328))
 
 ### Chore & Maintenance

diff --git a/packages/jest-circus/src/run.ts b/packages/jest-circus/src/run.ts
@@ -140,9 +140,8 @@ function startTestsConcurrently(concurrentTests: Array<ConcurrentTestEntry>) {
   jestExpect.setState({currentConcurrentTestName: testNameStorage});
   for (const test of concurrentTests) {
     try {
-      const promise = testNameStorage.run(getTestID(test), () =>
-        mutex(test.fn),
-      );
+      const testFn = test.fn;
+      const promise = mutex(() => testNameStorage.run(getTestID(test), testFn));
       // Avoid triggering the uncaught promise rejection handler in case the
       // test fails before being awaited on.
       // eslint-disable-next-line @typescript-eslint/no-empty-function