test: refactor config format test + more (#4684)

Author: alexander-akait

eslint.config.mjs

@@ -15,7 +15,7 @@ export default defineConfig([
     "test/build/config/error-array/webpack.config.js",
     "test/build/config-format/esm-require-await/webpack.config.js",
     "test/configtest/with-config-path/syntax-error.config.js",
-    "test/build/config-format/esm-require/webpack.config.js",
+    "test/build/config-format/auto/webpack.config.js",
   ]),
   {
     extends: [config],

test/build/config-format/auto/auto.test.mjs

@@ -0,0 +1,44 @@
+import { existsSync } from "node:fs";
+import { dirname, resolve } from "node:path";
+import { fileURLToPath } from "node:url";
+import { run } from "../../../utils/test-utils.js";
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+
+describe("auto loading different formats of configuration", () => {
+  it("should support configuration (Node.js build-in support)", async () => {
+    const { exitCode, stderr, stdout } = await run(__dirname, ["-c", "./webpack.config.js"], {
+      nodeOptions: [],
+    });
+
+    expect(stderr).toContain(
+      "Reparsing as ES module because module syntax was detected. This incurs a performance overhead.",
+    );
+    expect(stdout).toBeTruthy();
+    expect(exitCode).toBe(0);
+    expect(existsSync(resolve(__dirname, "dist/foo.bundle.js"))).toBeTruthy();
+  });
+
+  it("should support configuration (Node.js build-in support) with `mjs` extension", async () => {
+    const { exitCode, stderr, stdout } = await run(__dirname, ["-c", "./webpack.config.mjs"], {
+      nodeOptions: [],
+    });
+
+    expect(stderr).toBeFalsy();
+    expect(stdout).toBeTruthy();
+    expect(exitCode).toBe(0);
+    expect(existsSync(resolve(__dirname, "dist/foo.bundle.js"))).toBeTruthy();
+  });
+
+  it("should support configuration (Node.js build-in support) with `cjs` extension", async () => {
+    const { exitCode, stderr, stdout } = await run(__dirname, ["-c", "./webpack.config.cjs"], {
+      nodeOptions: [],
+    });
+
+    expect(stderr).toBeFalsy();
+    expect(stdout).toBeTruthy();
+    expect(exitCode).toBe(0);
+    expect(existsSync(resolve(__dirname, "dist/foo.bundle.js"))).toBeTruthy();
+  });
+});

…rmat/typescript-cjs-using-nodejs/main.ts test/build/config-format/auto/main.jstest/build/config-format/typescript-cjs-using-nodejs/main.ts renamed to test/build/config-format/auto/main.js test/build/config-format/typescript-cjs-using-nodejs/main.ts renamed to test/build/config-format/auto/main.js

@@ -1,5 +1,4 @@
 {
-  "type": "module",
   "engines": {
     "node": ">=18.12.0"
   }

…typescript-mjs-using-nodejs/package.json …st/build/config-format/auto/package.jsontest/build/config-format/typescript-mjs-using-nodejs/package.json renamed to test/build/config-format/auto/package.json test/build/config-format/typescript-mjs-using-nodejs/package.json renamed to test/build/config-format/auto/package.json

@@ -0,0 +1,13 @@
+const path = require("node:path");
+
+const mode = "development";
+const config = {
+  mode,
+  entry: "./main.js",
+  output: {
+    path: path.resolve("dist"),
+    filename: "foo.bundle.js",
+  },
+};
+
+module.exports = config;

test/build/config-format/auto/webpack.config.cjs

@@ -0,0 +1,13 @@
+import * as path from "node:path";
+
+const mode = "development";
+const config = {
+  mode,
+  entry: "./main.js",
+  output: {
+    path: path.resolve("dist"),
+    filename: "foo.bundle.js",
+  },
+};
+
+export default config;

test/build/config-format/auto/webpack.config.js

@@ -0,0 +1,13 @@
+import * as path from "node:path";
+
+const mode = "development";
+const config = {
+  mode,
+  entry: "./main.js",
+  output: {
+    path: path.resolve("dist"),
+    filename: "foo.bundle.js",
+  },
+};
+
+export default config;

test/build/config-format/auto/webpack.config.mjs

@@ -21,7 +21,13 @@ describe("webpack cli", () => {
   });
 
   it("should log error without transpilation", async () => {
-    const { exitCode, stderr, stdout } = await run(__dirname, ["--disable-interpret"]);
+    const [major] = process.versions.node.split(".").map(Number);
+    const { exitCode, stderr, stdout } = await run(__dirname, ["--disable-interpret"], {
+      nodeOptions: [
+        // Disable typescript strip types for tests
+        ...(major >= 24 ? ["--no-experimental-strip-types"] : []),
+      ],
+    });
 
     expect(exitCode).toBe(2);
     expect(stderr).toContain(`Failed to load '${resolve(__dirname, "webpack.config.ts")}' config`);