test: add test case for mocking invalid package and module (#9617)

Author: hi-ogawa

pnpm-lock.yaml

@@ -0,0 +1,4 @@
+{
+  "name": "test-dep-invalid",
+  "private": true
+}

test/cli/deps/dep-invalid/package.json

@@ -0,0 +1,9 @@
+// @ts-expect-error no type
+import * as dep from 'test-dep-invalid'
+import { expect, test, vi } from 'vitest'
+
+vi.mock('test-dep-invalid', () => ({ mocked: 'ok' }))
+
+test('basic', () => {
+  expect(dep).toMatchObject({ mocked: 'ok' })
+})

test/cli/fixtures/invalid-package/mock-bad-dep.test.ts

@@ -0,0 +1,12 @@
+import { expect, test, vi } from 'vitest'
+import { hello } from './wrapper.js'
+
+vi.mock('test-dep-invalid', () => ({}))
+
+vi.mock(import('./wrapper.js'), () => {
+  return { hello: () => 'mock-hello' }
+})
+
+test('basic', () => {
+  expect(hello()).toBe('mock-hello')
+})

test/cli/fixtures/invalid-package/mock-wrapper-and-bad-dep.test.ts

@@ -0,0 +1,10 @@
+import { expect, test, vi } from 'vitest'
+import { hello } from './wrapper.js'
+
+vi.mock(import('./wrapper.js'), () => {
+  return { hello: () => 'mock-hello' }
+})
+
+test('basic', () => {
+  expect(hello()).toBe('mock-hello')
+})

test/cli/fixtures/invalid-package/mock-wrapper.test.ts

@@ -0,0 +1,4 @@
+// @ts-expect-error no type
+import * as dep from 'test-dep-invalid'
+
+export const hello = () => dep

test/cli/fixtures/invalid-package/wrapper.ts

@@ -29,6 +29,7 @@
     "flatted": "catalog:",
     "obug": "^2.1.1",
     "playwright": "catalog:",
+    "test-dep-invalid": "link:./deps/dep-invalid",
     "tinyspy": "catalog:",
     "typescript": "catalog:",
     "unplugin-swc": "^1.5.9",

test/cli/package.json

@@ -1,5 +1,7 @@
+import path from 'node:path'
 import { expect, test } from 'vitest'
-import { runInlineTests } from '../../test-utils'
+import { rolldownVersion } from 'vitest/node'
+import { runInlineTests, runVitest } from '../../test-utils'
 
 test('setting resetMocks works if restoreMocks is also set', async () => {
   const { stderr, testTree } = await runInlineTests({
@@ -39,3 +41,115 @@ test('spy is not called here', () => {
     }
   `)
 })
+
+test('invalid packages', async () => {
+  const { results, errorTree } = await runVitest({
+    root: path.join(import.meta.dirname, '../fixtures/invalid-package'),
+  })
+  const testModuleErrors = Object.fromEntries(
+    results.map(testModule => [
+      testModule.relativeModuleId,
+      testModule.errors().map(e => e.message),
+    ]),
+  )
+
+  // requires Vite 8 for relaxed import analysis validataion
+  // https://github.com/vitejs/vite/pull/21601
+  if (rolldownVersion) {
+    expect(testModuleErrors).toMatchInlineSnapshot(`
+      {
+        "mock-bad-dep.test.ts": [],
+        "mock-wrapper-and-bad-dep.test.ts": [],
+        "mock-wrapper.test.ts": [],
+      }
+    `)
+    expect(errorTree()).toMatchInlineSnapshot(`
+      {
+        "mock-bad-dep.test.ts": {
+          "basic": "passed",
+        },
+        "mock-wrapper-and-bad-dep.test.ts": {
+          "basic": "passed",
+        },
+        "mock-wrapper.test.ts": {
+          "basic": "passed",
+        },
+      }
+    `)
+  }
+  else {
+    expect(testModuleErrors).toMatchInlineSnapshot(`
+      {
+        "mock-bad-dep.test.ts": [
+          "Failed to resolve entry for package "test-dep-invalid". The package may have incorrect main/module/exports specified in its package.json.",
+        ],
+        "mock-wrapper-and-bad-dep.test.ts": [
+          "Failed to resolve entry for package "test-dep-invalid". The package may have incorrect main/module/exports specified in its package.json.",
+        ],
+        "mock-wrapper.test.ts": [
+          "Failed to resolve entry for package "test-dep-invalid". The package may have incorrect main/module/exports specified in its package.json.",
+        ],
+      }
+    `)
+    expect(errorTree()).toMatchInlineSnapshot(`
+        {
+          "mock-bad-dep.test.ts": {},
+          "mock-wrapper-and-bad-dep.test.ts": {},
+          "mock-wrapper.test.ts": {},
+        }
+      `)
+  }
+})
+
+test('mocking modules with syntax error', async () => {
+  // TODO: manual mocked module still gets transformed so this is not supported yet.
+  const { errorTree, results } = await runInlineTests({
+    './syntax-error.js': `syntax error`,
+    './basic.test.js': /* ts */ `
+import * as dep from './syntax-error.js'
+
+vi.mock('./syntax-error.js', () => {
+  return { mocked: 'ok' }
+})
+
+test('can mock invalid module', () => {
+  expect(dep).toMatchObject({ mocked: 'ok' })
+})
+    `,
+  })
+
+  const testModuleErrors = Object.fromEntries(
+    results.map(testModule => [
+      testModule.relativeModuleId,
+      testModule.errors().map(e => e.message),
+    ]),
+  )
+  if (rolldownVersion) {
+    expect(testModuleErrors).toMatchInlineSnapshot(`
+      {
+        "basic.test.js": [
+          "Parse failure: Parse failed with 1 error:
+      Expected a semicolon or an implicit semicolon after a statement, but found none
+      1: syntax error
+               ^
+      At file: /syntax-error.js:1:6",
+        ],
+      }
+    `)
+  }
+  else {
+    expect(testModuleErrors).toMatchInlineSnapshot(`
+      {
+        "basic.test.js": [
+          "Parse failure: Expected ';', '}' or <eof>
+      At file: /syntax-error.js:1:7",
+        ],
+      }
+    `)
+  }
+  expect(errorTree()).toMatchInlineSnapshot(`
+    {
+      "basic.test.js": {},
+    }
+  `)
+})