fix: correctly automock nested interoped defaults (#2559)

* fix: correctly automock nested interoped defaults

* chore: cleanup

* chore: cleanup

examples/mocks/src/external/cjs-pseudoesm.cjs

@@ -0,0 +1,4 @@
+const fn = () => {}
+
+exports.fn = fn
+Object.defineProperty(exports, '__esModule', { value: true, enumerable: false })

examples/mocks/src/external/default-cjs.cjs

@@ -0,0 +1 @@
+module.exports = require('./cjs-pseudoesm.cjs')

examples/mocks/test/external.test.ts

@@ -1,4 +1,4 @@
-import '../src/external.mjs'
+import '../src/external/external.mjs'
 import { expect, test, vi } from 'vitest'
 import axios from 'axios'
 

examples/mocks/test/nested-default.spec.ts

@@ -0,0 +1,9 @@
+// @vitest-environment jsdom
+
+import * as modDefaultCjs from '../src/external/default-cjs.cjs'
+
+vi.mock('../src/external/default-cjs.cjs')
+
+test('default is mocked', () => {
+  expect(vi.isMockFunction(modDefaultCjs.default.fn)).toBe(true)
+})

examples/mocks/vite.config.ts

@@ -24,7 +24,7 @@ export default defineConfig({
     globals: true,
     environment: 'node',
     deps: {
-      external: [/src\/external\.mjs/],
+      external: [/src\/external/],
     },
   },
 })

packages/vite-node/src/client.ts

@@ -414,6 +414,26 @@ export class ViteNodeRunner {
           return defaultExport !== undefined
         return prop in mod || (defaultExport && prop in defaultExport)
       },
+      // this is needed for mocker to know what is available to mock
+      ownKeys(mod) {
+        const keys = Reflect.ownKeys(mod)
+        if (!defaultExport || isPrimitive(defaultExport))
+          return keys
+        const allKeys = [...keys, 'default', ...Reflect.ownKeys(defaultExport)]
+        return Array.from(new Set(allKeys))
+      },
+      getOwnPropertyDescriptor(mod, prop) {
+        const descriptor = Reflect.getOwnPropertyDescriptor(mod, prop)
+        if (descriptor)
+          return descriptor
+        if (prop === 'default' && defaultExport !== undefined) {
+          return {
+            value: defaultExport,
+            enumerable: true,
+            configurable: true,
+          }
+        }
+      },
     })
   }
 }