Fix requireActual of node module

CHANGELOG.md

@@ -74,6 +74,7 @@
 - `[jest-haste-map]` [**BREAKING**] Recover files correctly after haste name collisions are fixed ([#7329](https://github.com/facebook/jest/pull/7329))
 - `[jest-haste-map]` Remove legacy condition for duplicate module detection ([#7333](https://github.com/facebook/jest/pull/7333))
 - `[jest-haste-map]` Fix `require` detection with trailing commas and ignore `import typeof` modules ([#7385](https://github.com/facebook/jest/pull/7385))
+- `[jest-runtime]` Fix `requireActual` on node_modules with mock present ([#7404](https://github.com/facebook/jest/pull/7404))
 
 ### Chore & Maintenance
 

packages/jest-runtime/src/__tests__/runtime_require_actual.test.js

@@ -0,0 +1,26 @@
+/**
+ * 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.
+ *
+ */
+
+'use strict';
+
+let createRuntime;
+
+describe('Runtime requireActual', () => {
+  beforeEach(() => {
+    createRuntime = require('createRuntime');
+  });
+
+  it('requires node module when manual mock exists', () =>
+    createRuntime(__filename).then(runtime => {
+      const exports = runtime.requireActual(
+        runtime.__mockRootPath,
+        'mocked-node-module',
+      );
+      expect(exports.isManualMockModule).toBe(false);
+    }));
+});

packages/jest-runtime/src/__tests__/test_root/__mocks__/mocked-node-module.js

@@ -0,0 +1,11 @@
+/**
+ * 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.
+ *
+ */
+
+'use strict';
+
+exports.isManualMockModule = true;

packages/jest-runtime/src/index.js

@@ -283,6 +283,7 @@ class Runtime {
     from: Path,
     moduleName?: string,
     options: ?InternalModuleOptions,
+    isRequireActual: ?boolean,
   ) {
     const moduleID = this._resolver.getModuleID(
       this._virtualMocks,
@@ -303,6 +304,7 @@ class Runtime {
       moduleName && this._resolver.getMockModule(from, moduleName);
     if (
       (!options || !options.isInternalModule) &&
+      !isRequireActual &&
       !moduleResource &&
       manualMock &&
       manualMock !== this._isCurrentlyExecutingManualMock &&
@@ -353,6 +355,10 @@ class Runtime {
     return this.requireModule(from, to, {isInternalModule: true});
   }
 
+  requireActual(from: Path, moduleName: string) {
+    return this.requireModule(from, moduleName, undefined, true);
+  }
+
   requireMock(from: Path, moduleName: string) {
     const moduleID = this._resolver.getModuleID(
       this._virtualMocks,
@@ -802,7 +808,7 @@ class Runtime {
         : this.requireModuleOrMock.bind(this, from.filename);
     moduleRequire.cache = Object.create(null);
     moduleRequire.extensions = Object.create(null);
-    moduleRequire.requireActual = this.requireModule.bind(this, from.filename);
+    moduleRequire.requireActual = this.requireActual.bind(this, from.filename);
     moduleRequire.requireMock = this.requireMock.bind(this, from.filename);
     moduleRequire.resolve = (moduleName, options) =>
       this._requireResolve(from.filename, moduleName, options);