fix: unable to resolve path to mapped file with custom platform (#7312)

CHANGELOG.md

@@ -61,7 +61,8 @@
 - `[jest-config]` Use strings instead of `RegExp` instances in normalized configuration ([#7251](https://github.com/facebook/jest/pull/7251))
 - `[jest-circus]` Make sure to display real duration even if time is mocked ([#7264](https://github.com/facebook/jest/pull/7264))
 - `[expect]` Improves the failing message for `toStrictEqual` matcher. ([#7224](https://github.com/facebook/jest/pull/7224))
-- `[jest-mock]` [**BREAKING**] Fix bugs with mock/spy result tracking of recursive functions ([#6381](https://github.com/facebook/jest/pull/6381)
+- `[jest-mock]` [**BREAKING**] Fix bugs with mock/spy result tracking of recursive functions ([#6381](https://github.com/facebook/jest/pull/6381))
+- `[jest-resolve]` Fix not being able to resolve path to mapped file with custom platform ([#7312](https://github.com/facebook/jest/pull/7312))
 
 ### Chore & Maintenance
 

e2e/__tests__/__snapshots__/module_name_mapper.test.js.snap

@@ -32,7 +32,7 @@ exports[`moduleNameMapper wrong configuration 1`] = `
       12 | module.exports = () => 'test';
       13 | 
 
-      at packages/jest-resolve/build/index.js:410:17
+      at packages/jest-resolve/build/index.js:429:17
       at index.js:10:1
 
 "

e2e/resolve/__tests__/resolve.test.js

@@ -47,6 +47,11 @@ test('should resolve filename.native.js', () => {
   expect(platform.extension).toBe('native.js');
 });
 
+test('should resolve filename.native.js with moduleNameMapper', () => {
+  expect(testRequire('test2mapper')).not.toThrow();
+  expect(platform.extension).toBe('native.js');
+});
+
 test('should resolve filename.js', () => {
   expect(testRequire('../test3')).not.toThrow();
   expect(platform.extension).toBe('js');

e2e/resolve/package.json

@@ -2,9 +2,14 @@
   "name": "custom-resolve",
   "jest": {
     "haste": {
-      "platforms": ["native"],
+      "platforms": [
+        "native"
+      ],
       "defaultPlatform": "android"
     },
-    "testEnvironment": "node"
+    "testEnvironment": "node",
+    "moduleNameMapper": {
+      "test2mapper": "<rootDir>/test2mapper"
+    }
   }
 }

e2e/resolve/test2mapper.js

@@ -0,0 +1,8 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+
+module.exports = {extension: '.js'};

e2e/resolve/test2mapper.native.js

@@ -0,0 +1,8 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+
+module.exports = {extension: 'native.js'};

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

@@ -161,6 +161,7 @@ describe('getMockModule', () => {
 
     const moduleMap = ModuleMap.create('/');
     const resolver = new Resolver(moduleMap, {
+      extensions: ['.js'],
       moduleNameMapper: [
         {
           moduleName: '$1',

packages/jest-resolve/src/index.js

@@ -341,10 +341,23 @@ class Resolver {
   _resolveStubModuleName(from: Path, moduleName: string): ?Path {
     const dirname = path.dirname(from);
     const paths = this._options.modulePaths;
-    const extensions = this._options.extensions;
+    const extensions = this._options.extensions.slice();
     const moduleDirectory = this._options.moduleDirectories;
     const moduleNameMapper = this._options.moduleNameMapper;
     const resolver = this._options.resolver;
+    const defaultPlatform = this._options.defaultPlatform;
+
+    if (this._supportsNativePlatform) {
+      extensions.unshift(
+        ...this._options.extensions.map(ext => '.' + NATIVE_PLATFORM + ext),
+      );
+    }
+
+    if (defaultPlatform) {
+      extensions.unshift(
+        ...this._options.extensions.map(ext => '.' + defaultPlatform + ext),
+      );
+    }
 
     if (moduleNameMapper) {
       for (const {moduleName: mappedModuleName, regex} of moduleNameMapper) {