fix(vite-node): coerce to string in import(dep) (#3430)

Co-authored-by: Vladimir <sleuths.slews0s@icloud.com>

Author: jcbhmr

packages/vite-node/src/client.ts

@@ -267,7 +267,7 @@ export class ViteNodeRunner {
     const mod = this.moduleCache.getByModuleId(moduleId)
 
     const request = async (dep: string) => {
-      const [id, depFsPath] = await this.resolveUrl(dep, fsPath)
+      const [id, depFsPath] = await this.resolveUrl(`${dep}`, fsPath)
       return this.dependencyRequest(id, depFsPath, callstack)
     }
 

test/core/test/imports.test.ts

@@ -52,6 +52,12 @@ test('data with dynamic import works', async () => {
   expect(hi).toBe('hi')
 })
 
+test('dynamic import coerces to string', async () => {
+  const dataUri = 'data:text/javascript;charset=utf-8,export default "hi"'
+  const { default: hi } = await import({ toString: () => dataUri } as string)
+  expect(hi).toBe('hi')
+})
+
 test('dynamic import has Module symbol', async () => {
   const stringTimeoutMod = await import('./../src/timeout')