fix: correctly serialize Set passed to worker (#9915)

jestjs · Apr 29, 2020 · a5d0b08 · a5d0b08
1 parent 0740ad4
commit a5d0b08
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,7 @@
 ### Fixes
 
 - `[jest-haste-map]` Add missing `@types/graceful-fs` dependency ([#9913](https://github.com/facebook/jest/pull/9913))
+- `[jest-runner]` Correctly serialize `Set` passed to worker ([#9915](https://github.com/facebook/jest/pull/9915))
 
 ### Chore & Maintenance
 

diff --git a/packages/jest-runner/src/index.ts b/packages/jest-runner/src/index.ts
@@ -158,6 +158,9 @@ class TestRunner {
             changedFiles:
               this._context.changedFiles &&
               Array.from(this._context.changedFiles),
+            sourcesRelatedToTestsInChangedFiles:
+              this._context.sourcesRelatedToTestsInChangedFiles &&
+              Array.from(this._context.sourcesRelatedToTestsInChangedFiles),
           },
           globalConfig: this._globalConfig,
           path: test.path,

diff --git a/packages/jest-runner/src/testWorker.ts b/packages/jest-runner/src/testWorker.ts
@@ -89,6 +89,9 @@ export async function worker({
       context && {
         ...context,
         changedFiles: context.changedFiles && new Set(context.changedFiles),
+        sourcesRelatedToTestsInChangedFiles:
+          context.sourcesRelatedToTestsInChangedFiles &&
+          new Set(context.sourcesRelatedToTestsInChangedFiles),
       },
     );
   } catch (error) {

diff --git a/packages/jest-runner/src/types.ts b/packages/jest-runner/src/types.ts
@@ -49,13 +49,15 @@ export type TestRunnerOptions = {
   serial: boolean;
 };
 
+// make sure all props here are present in the type below it as well
 export type TestRunnerContext = {
   changedFiles?: Set<Config.Path>;
   sourcesRelatedToTestsInChangedFiles?: Set<Config.Path>;
 };
 
 export type TestRunnerSerializedContext = {
   changedFiles?: Array<Config.Path>;
+  sourcesRelatedToTestsInChangedFiles?: Array<Config.Path>;
 };
 
 // TODO: Should live in `@jest/core` or `jest-watcher`