ci: node.js@24 and rewrite tests

Author: alexander-akait

.github/workflows/nodejs.yml

@@ -58,14 +58,14 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, windows-latest, macos-latest]
-        node-version: [18.x, 20.x, 22.x, 23.x]
+        node-version: [18.x, 20.x, 22.x, 24.x]
         shard: ["1/4", "2/4", "3/4", "4/4"]
         webpack-version: [latest]
         dev-server-version: [latest]
         exclude:
           # Node.js v23.7.0 is broken on windows
           - os: windows-latest
-            node-version: 23.x
+            node-version: 24.x
 
     steps:
       - uses: actions/checkout@v4

test/build/config-format/typescript-esnext-mjs/main.ts renamed to test/build/config-format/typescript-cjs-using-nodejs/main.ts

@@ -1,16 +1,15 @@
 /* eslint-disable node/no-unsupported-features/es-syntax */
-/** eslint-disable **/
-import * as path from "path";
+const path = require("path");
 
 // cspell:ignore elopment
 const mode: string = "dev" + "elopment";
 const config = {
   mode,
   entry: "./main.ts",
   output: {
-    path: path.resolve(__dirname, "dist"),
+    path: path.resolve("dist"),
     filename: "foo.bundle.js",
   },
 };
 
-export = config;
+module.exports = config;

test/build/config-format/typescript-esnext-mjs/package.json renamed to test/build/config-format/typescript-cjs-using-nodejs/package.json

@@ -5,14 +5,14 @@ const { resolve } = require("path");
 
 describe("webpack cli", () => {
   it("should support typescript esnext file", async () => {
-    const env = { ...process.env };
-
+    const [major, minor] = process.versions.node.split(".").map(Number);
     const { exitCode, stderr, stdout } = await run(__dirname, ["-c", "./webpack.config.mts"], {
-      nodeOptions: ["--experimental-loader=ts-node/esm"],
-      env,
+      env: { NODE_NO_WARNINGS: 1 },
+      // Fallback to `ts-node/esm` for old Node.js versions
+      nodeOptions: major >= 22 && minor >= 6 ? [] : ["--experimental-loader=ts-node/esm"],
     });
 
-    expect(stderr).not.toBeFalsy(); // Deprecation warning logs on stderr
+    expect(stderr).toBeFalsy(); // Deprecation warning logs on stderr
     expect(stdout).toBeTruthy();
     expect(exitCode).toBe(0);
     expect(existsSync(resolve(__dirname, "dist/foo.bundle.js"))).toBeTruthy();

test/build/config-format/typescript-esnext-mjs/tsconfig.json renamed to test/build/config-format/typescript-cjs-using-nodejs/tsconfig.json

@@ -0,0 +1 @@
+console.log("Rimuru Tempest");

test/build/config-format/typescript-commonjs-using-typescript/webpack.config.cts renamed to test/build/config-format/typescript-cjs-using-nodejs/webpack.config.cts

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

test/build/config-format/typescript-commonjs-using-nodejs/tsconfig.json

@@ -0,0 +1,6 @@
+{
+  "compilerOptions": {
+    "module": "esnext",
+    "allowSyntheticDefaultImports": true
+  }
+}

test/build/config-format/typescript-esnext/main.ts renamed to test/build/config-format/typescript-mjs-using-nodejs/main.ts

@@ -5,16 +5,16 @@ const { resolve } = require("path");
 
 describe("webpack cli", () => {
   it("should support typescript esnext file", async () => {
-    const env = { ...process.env };
-
-    env.WEBPACK_CLI_FORCE_LOAD_ESM_CONFIG = true;
-
+    const [major, minor] = process.versions.node.split(".").map(Number);
     const { exitCode, stderr, stdout } = await run(__dirname, ["-c", "./webpack.config.ts"], {
-      nodeOptions: ["--experimental-loader=ts-node/esm"],
-      env,
+      env: { NODE_NO_WARNINGS: 1 },
+      nodeOptions:
+        major >= 22 && minor >= 6
+          ? ["--no-experimental-strip-types", "--experimental-loader=ts-node/esm"]
+          : ["--experimental-loader=ts-node/esm"],
     });
 
-    expect(stderr).not.toBeFalsy(); // Deprecation warning logs on stderr
+    expect(stderr).toBeFalsy(); // Deprecation warning logs on stderr
     expect(stdout).toBeTruthy();
     expect(exitCode).toBe(0);
     expect(existsSync(resolve(__dirname, "dist/foo.bundle.js"))).toBeTruthy();

test/build/config-format/typescript-esnext/package.json renamed to test/build/config-format/typescript-mjs-using-nodejs/package.json

@@ -4,9 +4,13 @@ const { resolve } = require("path");
 
 describe("webpack cli", () => {
   it("should support typescript file", async () => {
+    const [major, minor] = process.versions.node.split(".").map(Number);
     const { exitCode, stderr, stdout } = await run(__dirname, ["-c", "./webpack.config.ts"], {
-      // For `nyc`, remove it when we drop `nyc` usage
-      nodeOptions: ["--require=ts-node/register"],
+      env: { NODE_NO_WARNINGS: 1 },
+      nodeOptions:
+        major >= 22 && minor >= 6
+          ? ["--no-experimental-strip-types", "--require=ts-node/register"]
+          : ["--require=ts-node/register"],
     });
 
     expect(stderr).toBeFalsy();

test/build/config-format/typescript-esnext/tsconfig.json renamed to test/build/config-format/typescript-mjs-using-nodejs/tsconfig.json

@@ -0,0 +1,3 @@
+{
+  "type": "commonjs"
+}

test/build/config-format/typescript-esnext-mjs/typescript.test.js renamed to test/build/config-format/typescript-mjs-using-nodejs/typescript.test.js

@@ -1,11 +1,13 @@
+// eslint-disable-next-line n/no-unpublished-require
 const { run } = require("../../../utils/test-utils");
 const { existsSync } = require("fs");
 const { resolve } = require("path");
 
 describe("webpack cli", () => {
   it("should support typescript file", async () => {
     const [major, minor] = process.versions.node.split(".").map(Number);
-    const { exitCode, stderr, stdout } = await run(__dirname, ["-c", "./webpack.config.cts"], {
+    const { exitCode, stderr, stdout } = await run(__dirname, ["-c", "./webpack.config.ts"], {
+      env: { NODE_NO_WARNINGS: 1 },
       nodeOptions: major >= 22 && minor >= 6 ? ["--no-experimental-strip-types"] : [],
     });
 

test/build/config-format/typescript-esnext-mjs/webpack.config.mts renamed to test/build/config-format/typescript-mjs-using-nodejs/webpack.config.mts

@@ -1,4 +1,3 @@
-/* eslint-disable node/no-unsupported-features/es-syntax */
 /** eslint-disable **/
 const path = require("path");
 

test/build/config-format/typescript-ts-node-loader/main.ts

@@ -0,0 +1 @@
+console.log("Rimuru Tempest");

test/build/config-format/typescript-ts-node-loader/package.json

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

test/build/config-format/typescript-ts-node-loader/tsconfig.json

@@ -0,0 +1,6 @@
+{
+  "compilerOptions": {
+    "module": "esnext",
+    "allowSyntheticDefaultImports": true
+  }
+}

test/build/config-format/typescript-esnext/typescript.test.js renamed to test/build/config-format/typescript-ts-node-loader/typescript.test.js

@@ -0,0 +1,14 @@
+import * as path from "path";
+
+// cspell:ignore elopment
+const mode: string = "dev" + "elopment";
+const config = {
+  mode,
+  entry: "./main.ts",
+  output: {
+    path: path.resolve("dist"),
+    filename: "foo.bundle.js",
+  },
+};
+
+export default config;