fix: serialize changedFiles passed to workers

Author: thymikee

packages/jest-reporters/src/coverage_reporter.ts

@@ -170,7 +170,12 @@ export default class CoverageReporter extends BaseReporter {
           const result = await worker.worker({
             config,
             globalConfig,
-            options: this._options,
+            options: {
+              ...this._options,
+              changedFiles:
+                this._options.changedFiles &&
+                Array.from(this._options.changedFiles),
+            },
             path: filename,
           });
 

packages/jest-reporters/src/coverage_worker.ts

@@ -8,7 +8,7 @@
 import fs from 'fs';
 import {Config} from '@jest/types';
 import exit from 'exit';
-import {CoverageReporterOptions} from './types';
+import {CoverageReporterSerializedOptions} from './types';
 
 import generateEmptyCoverage, {
   CoverageWorkerResult,
@@ -18,7 +18,7 @@ export type CoverageWorkerData = {
   globalConfig: Config.GlobalConfig;
   config: Config.ProjectConfig;
   path: Config.Path;
-  options?: CoverageReporterOptions;
+  options?: CoverageReporterSerializedOptions;
 };
 
 export {CoverageWorkerResult};
@@ -40,6 +40,6 @@ export function worker({
     path,
     globalConfig,
     config,
-    options && options.changedFiles,
+    options && options.changedFiles && new Set(options.changedFiles),
   );
 }

packages/jest-reporters/src/types.ts

@@ -41,6 +41,10 @@ export type CoverageReporterOptions = {
   changedFiles?: Set<Config.Path>;
 };
 
+export type CoverageReporterSerializedOptions = {
+  changedFiles?: Array<Config.Path>;
+};
+
 export type OnTestStart = (test: Test) => Promise<void>;
 export type OnTestFailure = (
   test: Test,

packages/jest-runner/src/index.ts

@@ -127,7 +127,12 @@ class TestRunner {
 
         return worker.worker({
           config: test.context.config,
-          context: this._context,
+          context: {
+            ...this._context,
+            changedFiles:
+              this._context.changedFiles &&
+              Array.from(this._context.changedFiles),
+          },
           globalConfig: this._globalConfig,
           path: test.path,
           serializableModuleMap: watcher.isWatchMode()

packages/jest-runner/src/testWorker.ts

@@ -12,15 +12,15 @@ import HasteMap, {SerializableModuleMap, ModuleMap} from 'jest-haste-map';
 import exit from 'exit';
 import {separateMessageFromStack} from 'jest-message-util';
 import Runtime from 'jest-runtime';
-import {ErrorWithCode, TestRunnerContext} from './types';
+import {ErrorWithCode, TestRunnerSerializedContext} from './types';
 import runTest from './runTest';
 
 type WorkerData = {
   config: Config.ProjectConfig;
   globalConfig: Config.GlobalConfig;
   path: Config.Path;
   serializableModuleMap: SerializableModuleMap | null;
-  context?: TestRunnerContext;
+  context?: TestRunnerSerializedContext;
 };
 
 // Make sure uncaught errors are logged before we exit.
@@ -86,7 +86,10 @@ export async function worker({
       globalConfig,
       config,
       getResolver(config, moduleMap),
-      context,
+      context && {
+        ...config,
+        changedFiles: context.changedFiles && new Set(context.changedFiles),
+      },
     );
   } catch (error) {
     throw formatError(error);

packages/jest-runner/src/types.ts

@@ -53,6 +53,10 @@ export type TestRunnerContext = {
   changedFiles?: Set<Config.Path>;
 };
 
+export type TestRunnerSerializedContext = {
+  changedFiles?: Array<Config.Path>;
+};
+
 // TODO: Should live in `@jest/core` or `jest-watcher`
 export type WatcherState = {
   interrupted: boolean;