fix(test): TestRunner avoids using jest mock's require

xiaoxiaojx · xiaoxiaojx · commit 1cd5216e6c82 · 2025-07-12T00:17:49.000+08:00
diff --git a/test/configCases/externals/externals-array/webpack.config.js b/test/configCases/externals/externals-array/webpack.config.js
@@ -1,17 +1,17 @@
 const webpack = require("../../../../");
 
-/** @type {import("../../../../").Configuration[]} */
+/** @type {import("../../../../types").Configuration[]} */
 module.exports = [
 	{
 		output: {
-			libraryTarget: "commonjs2"
+			libraryTarget: "global"
 		},
 		externals: {
-			external: ["webpack", "version"]
+			external: ["process", "version"]
 		},
 		plugins: [
 			new webpack.DefinePlugin({
-				EXPECTED: JSON.stringify(webpack.version)
+				EXPECTED: JSON.stringify(process.version)
 			})
 		]
 	},
diff --git a/test/configCases/externals/node-require/index.js b/test/configCases/externals/node-require/index.js
@@ -0,0 +1,22 @@
+function getMajorVersion(versionStr) {
+	const match = versionStr.match(/^v?(\d+)\./);
+	if (match) {
+	  return parseInt(match[1], 10);
+	}
+	return null;
+}
+
+it("should not fail on optional externals", function() {	
+	if (getMajorVersion(NODE_VERSION) <= 12) {
+		const external = require("external");
+		// The behavior of jest mock's require is different from that of node require, so it works fine here.
+		expect(external).toBe(EXPECTED);
+	} else {
+		try {
+			require("external");
+		} catch (e) {
+			// Since there is no webpack in node_modules, node require will report an error here.
+			expect(e.message).toContain("Cannot find module 'webpack'");
+		}
+	}
+});
diff --git a/test/configCases/externals/node-require/webpack.config.js b/test/configCases/externals/node-require/webpack.config.js
@@ -0,0 +1,17 @@
+const webpack = require("../../../../");
+
+/** @type {import("../../../../types").Configuration} */
+module.exports = {
+	output: {
+		libraryTarget: "commonjs2"
+	},
+	externals: {
+		external: ["webpack", "version"]
+	},
+	plugins: [
+		new webpack.DefinePlugin({
+			NODE_VERSION: JSON.stringify(process.version),
+			EXPECTED: JSON.stringify(webpack.version)
+		})
+	]
+};
diff --git a/test/runner/index.js b/test/runner/index.js
@@ -1,4 +1,5 @@
 const fs = require("fs");
+const { Module } = require("module");
 const path = require("path");
 const { fileURLToPath, pathToFileURL } = require("url");
 const vm = require("vm");
@@ -292,7 +293,11 @@ class TestRunner {
 		}
 		const moduleInfo = this._resolveModule(currentDirectory, module);
 		if (!moduleInfo) {
-			return require(module.startsWith("node:") ? module.slice(5) : module);
+			// node v12.2.0+ has Module.createRequire
+			const rawRequire = Module.createRequire
+				? Module.createRequire(currentDirectory)
+				: require;
+			return rawRequire(module.startsWith("node:") ? module.slice(5) : module);
 		}
 		const { modulePath } = moduleInfo;
 		if (
