fix(vitest): cache fs code only for forks pool (#5909)

sheremet-va · web-flow · commit e30d9b4d58b5 · 2024-06-17T13:55:24.000+02:00
diff --git a/packages/vitest/src/node/pools/forks.ts b/packages/vitest/src/node/pools/forks.ts
@@ -29,7 +29,7 @@ function createChildProcessChannel(project: WorkspaceProject) {
     postMessage: message => emitter.emit(events.response, message),
   }
 
-  const rpc = createBirpc<RunnerRPC, RuntimeRPC>(createMethodsRPC(project), {
+  const rpc = createBirpc<RunnerRPC, RuntimeRPC>(createMethodsRPC(project, { cacheFs: true }), {
     eventNames: ['onCancel'],
     serialize: v8.serialize,
     deserialize: v => v8.deserialize(Buffer.from(v)),
diff --git a/packages/vitest/src/node/pools/rpc.ts b/packages/vitest/src/node/pools/rpc.ts
@@ -8,8 +8,13 @@ import type { WorkspaceProject } from '../workspace'
 const created = new Set()
 const promises = new Map<string, Promise<void>>()
 
-export function createMethodsRPC(project: WorkspaceProject): RuntimeRPC {
+interface MethodsOptions {
+  cacheFs?: boolean
+}
+
+export function createMethodsRPC(project: WorkspaceProject, options: MethodsOptions = {}): RuntimeRPC {
   const ctx = project.ctx
+  const cacheFs = options.cacheFs ?? false
   return {
     snapshotSaved(snapshot) {
       ctx.snapshot.add(snapshot)
@@ -30,7 +35,7 @@ export function createMethodsRPC(project: WorkspaceProject): RuntimeRPC {
     async fetch(id, transformMode) {
       const result = await project.vitenode.fetchResult(id, transformMode)
       const code = result.code
-      if (result.externalize) {
+      if (!cacheFs || result.externalize) {
         return result
       }
       if ('id' in result && typeof result.id === 'string') {
diff --git a/packages/vitest/src/node/pools/vmForks.ts b/packages/vitest/src/node/pools/vmForks.ts
@@ -34,20 +34,23 @@ function createChildProcessChannel(project: WorkspaceProject) {
     postMessage: message => emitter.emit(events.response, message),
   }
 
-  const rpc = createBirpc<RunnerRPC, RuntimeRPC>(createMethodsRPC(project), {
-    eventNames: ['onCancel'],
-    serialize: v8.serialize,
-    deserialize: v => v8.deserialize(Buffer.from(v)),
-    post(v) {
-      emitter.emit(events.message, v)
+  const rpc = createBirpc<RunnerRPC, RuntimeRPC>(
+    createMethodsRPC(project, { cacheFs: true }),
+    {
+      eventNames: ['onCancel'],
+      serialize: v8.serialize,
+      deserialize: v => v8.deserialize(Buffer.from(v)),
+      post(v) {
+        emitter.emit(events.message, v)
+      },
+      on(fn) {
+        emitter.on(events.response, fn)
+      },
+      onTimeoutError(functionName) {
+        throw new Error(`[vitest-pool]: Timeout calling "${functionName}"`)
+      },
     },
-    on(fn) {
-      emitter.on(events.response, fn)
-    },
-    onTimeoutError(functionName) {
-      throw new Error(`[vitest-pool]: Timeout calling "${functionName}"`)
-    },
-  })
+  )
 
   project.ctx.onCancel(reason => rpc.onCancel(reason))
 

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ function createChildProcessChannel(project: WorkspaceProject) {`
`29`	`29`	`postMessage: message => emitter.emit(events.response, message),`
`30`	`30`	`}`
`31`	`31`
`32`		`- const rpc = createBirpc<RunnerRPC, RuntimeRPC>(createMethodsRPC(project), {`
	`32`	`+ const rpc = createBirpc<RunnerRPC, RuntimeRPC>(createMethodsRPC(project, { cacheFs: true }), {`
`33`	`33`	`eventNames: ['onCancel'],`
`34`	`34`	`serialize: v8.serialize,`
`35`	`35`	`deserialize: v => v8.deserialize(Buffer.from(v)),`