test: test case for catching invalid package resolution error (#21601)

hi-ogawa · web-flow · commit c9b9359fe88f · 2026-02-09T20:04:12.000+09:00
diff --git a/packages/vite/src/node/ssr/runtime/__tests__/fixtures/invalid-package/deps/test-dep-invalid-exports/package.json b/packages/vite/src/node/ssr/runtime/__tests__/fixtures/invalid-package/deps/test-dep-invalid-exports/package.json
@@ -0,0 +1,4 @@
+{
+  "name": "test-dep-invalid-exports",
+  "private": true
+}
diff --git a/packages/vite/src/node/ssr/runtime/__tests__/fixtures/invalid-package/test.js b/packages/vite/src/node/ssr/runtime/__tests__/fixtures/invalid-package/test.js
@@ -0,0 +1,8 @@
+export async function test() {
+  try {
+    const result = await import('test-dep-invalid-exports')
+    return { ok: true, data: result }
+  } catch (e) {
+    return { ok: false, data: e }
+  }
+}
diff --git a/packages/vite/src/node/ssr/runtime/__tests__/package.json b/packages/vite/src/node/ssr/runtime/__tests__/package.json
@@ -8,6 +8,7 @@
   "dependencies": {
     "@vitejs/cjs-external": "link:./fixtures/cjs-external",
     "@vitejs/esm-external": "link:./fixtures/esm-external",
+    "test-dep-invalid-exports": "file:./fixtures/invalid-package/deps/test-dep-invalid-exports",
     "tinyspy": "2.2.0",
     "birpc": "^0.2.19"
   }
diff --git a/packages/vite/src/node/ssr/runtime/__tests__/server-runtime.spec.ts b/packages/vite/src/node/ssr/runtime/__tests__/server-runtime.spec.ts
@@ -515,3 +515,25 @@ describe('virtual module hmr', async () => {
     }
   })
 })
+
+describe('invalid package', async () => {
+  const it = await createModuleRunnerTester({
+    environments: {
+      ssr: {
+        resolve: {
+          noExternal: true,
+        },
+      },
+    },
+  })
+
+  it('can catch resolve error on runtime', async ({ runner }) => {
+    const mod = await runner.import('./fixtures/invalid-package/test.js')
+    expect(await mod.test()).toMatchInlineSnapshot(`
+      {
+        "data": [Error: Failed to resolve entry for package "test-dep-invalid-exports". The package may have incorrect main/module/exports specified in its package.json.],
+        "ok": false,
+      }
+    `)
+  })
+})
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml

-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +{
 +  "name": "test-dep-invalid-exports",
 +  "private": true
 +}
Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,7 @@`
`8`	`8`	`"dependencies": {`
`9`	`9`	`"@vitejs/cjs-external": "link:./fixtures/cjs-external",`
`10`	`10`	`"@vitejs/esm-external": "link:./fixtures/esm-external",`
	`11`	`+ "test-dep-invalid-exports": "file:./fixtures/invalid-package/deps/test-dep-invalid-exports",`
`11`	`12`	`"tinyspy": "2.2.0",`
`12`	`13`	`"birpc": "^0.2.19"`
`13`	`14`	`}`