fix: resolve builtin modules correctly

close jestjs#15676

Author: JounQin

CHANGELOG.md

@@ -4,6 +4,10 @@
 
 - `[jest-resolver]` Implement the `defaultAsyncResolver` ([#15679](https://github.com/jestjs/jest/pull/15679))
 
+### Fixes
+
+- `[jest-resolver]` Resolve builtin modules correctly ([#15683](https://github.com/jestjs/jest/pull/15683))
+
 ### Chore & Maintenance
 
 - `[*]` Remove and deprecate `jest-repl` package ([#15673](https://github.com/jestjs/jest/pull/15673))

e2e/__tests__/__snapshots__/moduleNameMapper.test.ts.snap

@@ -41,7 +41,7 @@ exports[`moduleNameMapper wrong array configuration 1`] = `
       12 | module.exports = () => 'test';
       13 |
 
-      at createNoMappedModuleFoundError (../../packages/jest-resolve/build/index.js:1129:17)
+      at createNoMappedModuleFoundError (../../packages/jest-resolve/build/index.js:1142:17)
       at Object.require (index.js:10:1)
       at Object.require (__tests__/index.js:10:20)"
 `;
@@ -71,7 +71,7 @@ exports[`moduleNameMapper wrong configuration 1`] = `
       12 | module.exports = () => 'test';
       13 |
 
-      at createNoMappedModuleFoundError (../../packages/jest-resolve/build/index.js:1129:17)
+      at createNoMappedModuleFoundError (../../packages/jest-resolve/build/index.js:1142:17)
       at Object.require (index.js:10:1)
       at Object.require (__tests__/index.js:10:20)"
 `;

e2e/__tests__/__snapshots__/requireMissingExt.test.ts.snap

@@ -26,7 +26,7 @@ exports[`shows a proper error from deep requires 1`] = `
       12 | test('dummy', () => {
       13 |   expect(1).toBe(1);
 
-      at Resolver._throwModNotFoundError (../../packages/jest-resolve/build/index.js:875:11)
+      at Resolver._throwModNotFoundError (../../packages/jest-resolve/build/index.js:888:11)
       at Object.<anonymous> (node_modules/discord.js/src/index.js:21:12)
       at Object.require (__tests__/test.js:10:1)"
 `;

e2e/__tests__/__snapshots__/resolveNoFileExtensions.test.ts.snap

@@ -37,7 +37,7 @@ exports[`show error message with matching files 1`] = `
         |                  ^
       9 |
 
-      at Resolver._throwModNotFoundError (../../packages/jest-resolve/build/index.js:875:11)
+      at Resolver._throwModNotFoundError (../../packages/jest-resolve/build/index.js:888:11)
       at Object.require (index.js:8:18)
       at Object.require (__tests__/test.js:8:11)"
 `;

packages/jest-resolve/src/defaultResolver.ts

@@ -5,6 +5,7 @@
  * LICENSE file in the root directory of this source tree.
  */
 
+import {isBuiltin} from 'module';
 import {fileURLToPath} from 'url';
 import pnpResolver from 'jest-pnp-resolver';
 import {
@@ -73,6 +74,12 @@ function baseResolver(
   options: ResolverOptions,
   async?: true,
 ): string | Promise<string> {
+  // https://github.com/oxc-project/oxc-resolver/issues/565
+  // https://github.com/jestjs/jest/issues/15676
+  if (isBuiltin(path)) {
+    return path;
+  }
+
   if (process.versions.pnp && options.allowPnp !== false) {
     return pnpResolver(path, options);
   }
@@ -111,13 +118,12 @@ function baseResolver(
     unrsResolver = unrsResolver.cloneWithOptions(resolveOptions);
   } else {
     unrsResolver = new ResolverFactory(resolveOptions);
-    setResolver(unrsResolver);
   }
 
+  setResolver(unrsResolver);
+
   const finalResolver = (
-    resolve: (
-      resolver: ResolverFactory,
-    ) => ResolveResult | Promise<ResolveResult>,
+    resolve: () => ResolveResult | Promise<ResolveResult>,
   ) => {
     const resolveWithPathsFallback = (result: ResolveResult) => {
       if (!result.path && paths?.length) {
@@ -131,12 +137,13 @@ function baseResolver(
             ...resolveOptions,
             modules: paths,
           });
-          return resolve(unrsResolver);
+          setResolver(unrsResolver);
+          return resolve();
         }
       }
       return result;
     };
-    const result = resolve(unrsResolver!);
+    const result = resolve();
     if ('then' in result) {
       return result.then(resolveWithPathsFallback).then(handleResolveResult);
     }
@@ -145,8 +152,10 @@ function baseResolver(
     );
   };
 
-  return finalResolver((resolver: ResolverFactory) =>
-    async ? resolver.async(basedir, path) : resolver.sync(basedir, path),
+  return finalResolver(() =>
+    async
+      ? unrsResolver!.async(basedir, path)
+      : unrsResolver!.sync(basedir, path),
   );
 }