fix: pass resolve conditions when loading stub module

CHANGELOG.md

@@ -94,6 +94,7 @@
 - `[jest-util]` Always load `mjs` files with `import` ([#15447](https://github.com/jestjs/jest/pull/15447))
 - `[jest-worker]` Properly handle a circular reference error when worker tries to send an assertion fails where either the expected or actual value is circular ([#15191](https://github.com/jestjs/jest/pull/15191))
 - `[jest-worker]` Properly handle a BigInt when worker tries to send an assertion fails where either the expected or actual value is BigInt ([#15191](https://github.com/jestjs/jest/pull/15191))
+- `[jest-resolve,jest-runtime,jest-resolve-dependencies]` Pass the conditions when resolving stub modules ([#15489](https://github.com/jestjs/jest/pull/15489))
 
 ### Performance
 

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:1182:17)
+      at createNoMappedModuleFoundError (../../packages/jest-resolve/build/index.js:1184: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:1182:17)
+      at createNoMappedModuleFoundError (../../packages/jest-resolve/build/index.js:1184:17)
       at Object.require (index.js:10:1)
       at Object.require (__tests__/index.js:10:20)"
 `;

e2e/resolve-conditions/__tests__/module-name-mapper.test.mjs

@@ -0,0 +1,14 @@
+/**
+ * Copyright (c) Meta Platforms, Inc. and affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ * @jest-environment jest-environment-jsdom
+ */
+
+import {fn} from 'fake-dual-dep2';
+
+test('returns correct message', () => {
+  expect(fn()).toBe('from browser');
+});

e2e/resolve-conditions/package.json

@@ -9,6 +9,9 @@
     "testMatch": [
       "<rootDir>/**/*.test.*"
     ],
+    "moduleNameMapper": {
+      "^fake-dual-dep2$": "fake-dual-dep2"
+    },
     "transform": {}
   }
 }

packages/jest-resolve-dependencies/src/index.ts

@@ -36,6 +36,7 @@ export class DependencyResolver {
 
   resolve(file: string, options?: ResolveModuleConfig): Array<string> {
     const dependencies = this._hasteFS.getDependencies(file);
+    const fallbackOptions: ResolveModuleConfig = {conditions: undefined};
     if (!dependencies) {
       return [];
     }
@@ -51,11 +52,15 @@ export class DependencyResolver {
         resolvedDependency = this._resolver.resolveModule(
           file,
           dependency,
-          options,
+          options ?? fallbackOptions,
         );
       } catch {
         try {
-          resolvedDependency = this._resolver.getMockModule(file, dependency);
+          resolvedDependency = this._resolver.getMockModule(
+            file,
+            dependency,
+            options ?? fallbackOptions,
+          );
         } catch {
           // leave resolvedDependency as undefined if nothing can be found
         }
@@ -73,6 +78,7 @@ export class DependencyResolver {
         resolvedMockDependency = this._resolver.getMockModule(
           resolvedDependency,
           path.basename(dependency),
+          options ?? fallbackOptions,
         );
       } catch {
         // leave resolvedMockDependency as undefined if nothing can be found

packages/jest-resolve/src/__tests__/resolve.test.ts

@@ -742,7 +742,9 @@ describe('getMockModuleAsync', () => {
     } as ResolverConfig);
     const src = require.resolve('../');
 
-    await resolver.resolveModuleAsync(src, 'dependentModule');
+    await resolver.resolveModuleAsync(src, 'dependentModule', {
+      conditions: ['browser'],
+    });
 
     expect(mockUserResolverAsync.async).toHaveBeenCalled();
     expect(mockUserResolverAsync.async.mock.calls[0][0]).toBe(
@@ -752,6 +754,10 @@ describe('getMockModuleAsync', () => {
       'basedir',
       path.dirname(src),
     );
+    expect(mockUserResolverAsync.async.mock.calls[0][1]).toHaveProperty(
+      'conditions',
+      ['browser'],
+    );
   });
 });
 

packages/jest-resolve/src/resolver.ts

@@ -341,11 +341,11 @@
   resolveModule(
     from: string,
     moduleName: string,
-    options?: ResolveModuleConfig,
+    options: ResolveModuleConfig,
   ): string {
     const dirname = path.dirname(from);
     const module =
-      this.resolveStubModuleName(from, moduleName) ||
+      this.resolveStubModuleName(from, moduleName, options) ||
       this.resolveModuleFromDirIfExists(dirname, moduleName, options);
     if (module) return module;
 
@@ -362,7 +362,7 @@
   ): Promise<string> {
     const dirname = path.dirname(from);
     const module =
-      (await this.resolveStubModuleNameAsync(from, moduleName)) ||
+      (await this.resolveStubModuleNameAsync(from, moduleName, options)) ||
       (await this.resolveModuleFromDirIfExistsAsync(
         dirname,
         moduleName,
@@ -482,25 +482,37 @@
     );
   }
 
-  getMockModule(from: string, name: string): string | null {
+  getMockModule(
+    from: string,
+    name: string,
+    options: Pick<ResolveModuleConfig, 'conditions'>,
+  ): string | null {
     const mock = this._moduleMap.getMockModule(name);
     if (mock) {
       return mock;
     } else {
-      const moduleName = this.resolveStubModuleName(from, name);
+      const moduleName = this.resolveStubModuleName(from, name, options);
       if (moduleName) {
         return this.getModule(moduleName) || moduleName;
       }
     }
     return null;
   }
 
-  async getMockModuleAsync(from: string, name: string): Promise<string | null> {
+  async getMockModuleAsync(
+    from: string,
+    name: string,
+    options: Pick<ResolveModuleConfig, 'conditions'>,
+  ): Promise<string | null> {
     const mock = this._moduleMap.getMockModule(name);
     if (mock) {
       return mock;
     } else {
-      const moduleName = await this.resolveStubModuleNameAsync(from, name);
+      const moduleName = await this.resolveStubModuleNameAsync(
+        from,
+        name,
+        options,
+      );
       if (moduleName) {
         return this.getModule(moduleName) || moduleName;
       }
@@ -536,7 +548,7 @@
     virtualMocks: Map<string, boolean>,
     from: string,
     moduleName = '',
-    options?: ResolveModuleConfig,
+    options: ResolveModuleConfig,
   ): string {
     const stringifiedOptions = options ? JSON.stringify(options) : '';
     const key = from + path.delimiter + moduleName + stringifiedOptions;
@@ -552,7 +564,7 @@
       moduleName,
       options,
     );
-    const mockPath = this._getMockPath(from, moduleName);
+    const mockPath = this._getMockPath(from, moduleName, options);
 
     const sep = path.delimiter;
     const id =
@@ -570,7 +582,7 @@
     virtualMocks: Map<string, boolean>,
     from: string,
     moduleName = '',
-    options?: ResolveModuleConfig,
+    options: ResolveModuleConfig,
   ): Promise<string> {
     const stringifiedOptions = options ? JSON.stringify(options) : '';
     const key = from + path.delimiter + moduleName + stringifiedOptions;
@@ -589,7 +601,7 @@
       moduleName,
       options,
     );
-    const mockPath = await this._getMockPathAsync(from, moduleName);
+    const mockPath = await this._getMockPathAsync(from, moduleName, options);
 
     const sep = path.delimiter;
     const id =
@@ -611,15 +623,15 @@
     virtualMocks: Map<string, boolean>,
     from: string,
     moduleName: string,
-    options?: ResolveModuleConfig,
+    options: ResolveModuleConfig,
   ): string | null {
     if (this.isCoreModule(moduleName)) {
       return moduleName;
     }
     if (moduleName.startsWith('data:')) {
       return moduleName;
     }
-    return this._isModuleResolved(from, moduleName)
+    return this._isModuleResolved(from, moduleName, options)
       ? this.getModule(moduleName)
       : this._getVirtualMockPath(virtualMocks, from, moduleName, options);
   }
@@ -628,7 +640,7 @@
     virtualMocks: Map<string, boolean>,
     from: string,
     moduleName: string,
-    options?: ResolveModuleConfig,
+    options: ResolveModuleConfig,
   ): Promise<string | null> {
     if (this.isCoreModule(moduleName)) {
       return moduleName;
@@ -639,32 +651,38 @@
     const isModuleResolved = await this._isModuleResolvedAsync(
       from,
       moduleName,
+      options,
     );
     return isModuleResolved
       ? this.getModule(moduleName)
       : this._getVirtualMockPathAsync(virtualMocks, from, moduleName, options);
   }
 
-  private _getMockPath(from: string, moduleName: string): string | null {
+  private _getMockPath(
+    from: string,
+    moduleName: string,
+    options: Pick<ResolveModuleConfig, 'conditions'>,
+  ): string | null {
     return this.isCoreModule(moduleName)
       ? null
-      : this.getMockModule(from, moduleName);
+      : this.getMockModule(from, moduleName, options);
   }
 
   private async _getMockPathAsync(
     from: string,
     moduleName: string,
+    options: Pick<ResolveModuleConfig, 'conditions'>,
   ): Promise<string | null> {
     return this.isCoreModule(moduleName)
       ? null
-      : this.getMockModuleAsync(from, moduleName);
+      : this.getMockModuleAsync(from, moduleName, options);
   }
 
   private _getVirtualMockPath(
     virtualMocks: Map<string, boolean>,
     from: string,
     moduleName: string,
-    options?: ResolveModuleConfig,
+    options: ResolveModuleConfig,
   ): string {
     const virtualMockPath = this.getModulePath(from, moduleName);
     return virtualMocks.get(virtualMockPath)
@@ -688,23 +706,33 @@
         : from;
   }
 
-  private _isModuleResolved(from: string, moduleName: string): boolean {
+  private _isModuleResolved(
+    from: string,
+    moduleName: string,
+    options: Pick<ResolveModuleConfig, 'conditions'>,
+  ): boolean {
     return !!(
-      this.getModule(moduleName) || this.getMockModule(from, moduleName)
+      this.getModule(moduleName) ||
+      this.getMockModule(from, moduleName, options)
     );
   }
 
   private async _isModuleResolvedAsync(
     from: string,
     moduleName: string,
+    options: Pick<ResolveModuleConfig, 'conditions'>,
   ): Promise<boolean> {
     return !!(
       this.getModule(moduleName) ||
-      (await this.getMockModuleAsync(from, moduleName))
+      (await this.getMockModuleAsync(from, moduleName, options))
     );
   }
 
-  resolveStubModuleName(from: string, moduleName: string): string | null {
+  resolveStubModuleName(
+    from: string,
+    moduleName: string,
+    options: Pick<ResolveModuleConfig, 'conditions'>,
+  ): string | null {
     const dirname = path.dirname(from);
 
     const {extensions, moduleDirectory, paths} = this._prepareForResolution(
@@ -727,11 +755,11 @@
           let module: string | null = null;
           for (const possibleModuleName of possibleModuleNames) {
             const updatedName = mapModuleName(possibleModuleName);
-
             module =
               this.getModule(updatedName) ||
               Resolver.findNodeModule(updatedName, {
                 basedir: dirname,
+                conditions: options?.conditions,
                 extensions,
                 moduleDirectory,
                 paths,
@@ -763,6 +791,7 @@
   async resolveStubModuleNameAsync(
     from: string,
     moduleName: string,
+    options?: Pick<ResolveModuleConfig, 'conditions'>,
   ): Promise<string | null> {
     const dirname = path.dirname(from);
 
@@ -791,6 +820,7 @@
               this.getModule(updatedName) ||
               (await Resolver.findNodeModuleAsync(updatedName, {
                 basedir: dirname,
+                conditions: options?.conditions,
                 extensions,
                 moduleDirectory,
                 paths,