test: fix

alexander-akait · alexander-akait · commit e98048425191 · 2025-05-13T18:23:28.000+03:00
diff --git a/test/build/config-format/typescript-cjs-using-nodejs/tsconfig.json b/test/build/config-format/typescript-cjs-using-nodejs/tsconfig.json
@@ -3,7 +3,6 @@
     "target": "esnext",
     "allowImportingTsExtensions": true,
     "rewriteRelativeImportExtensions": true,
-    "verbatimModuleSyntax": true,
-    "module": "commonjs"
+    "module": "esnext"
   }
 }
diff --git a/test/build/config-format/typescript-cjs-using-nodejs/typescript.test.js b/test/build/config-format/typescript-cjs-using-nodejs/typescript.test.js
@@ -5,16 +5,20 @@ const { resolve } = require("path");
 
 describe("webpack cli", () => {
   it("should support typescript esnext file", async () => {
-    const [major, minor] = process.versions.node.split(".").map(Number);
-    const { exitCode, stderr, stdout } = await run(__dirname, ["-c", "./webpack.config.cts"], {
-      env: {
-        NODE_NO_WARNINGS: 1,
-        // Due nyc logic
-        WEBPACK_CLI_FORCE_LOAD_ESM_CONFIG: true,
+    const [major] = process.versions.node.split(".").map(Number);
+    const { exitCode, stderr, stdout } = await run(
+      __dirname,
+      ["-c", "./webpack.config.ts", "--disable-interpret"],
+      {
+        env: {
+          NODE_NO_WARNINGS: 1,
+          // Due nyc logic
+          WEBPACK_CLI_FORCE_LOAD_ESM_CONFIG: true,
+        },
+        // Fallback to `ts-node/esm` for old Node.js versions
+        nodeOptions: major >= 24 ? [] : ["--experimental-loader=ts-node/esm"],
       },
-      // Fallback to `ts-node/esm` for old Node.js versions
-      nodeOptions: major >= 22 && minor >= 6 ? [] : ["--experimental-loader=ts-node/esm"],
-    });
+    );
 
     expect(stderr).toBeFalsy(); // Deprecation warning logs on stderr
     expect(stdout).toBeTruthy();
diff --git a/test/build/config-format/typescript-cjs-using-nodejs/webpack.config.ts b/test/build/config-format/typescript-cjs-using-nodejs/webpack.config.ts
@@ -1,4 +1,3 @@
-/* eslint-disable node/no-unsupported-features/es-syntax */
 import * as path from "path";
 
 // cspell:ignore elopment
diff --git a/test/build/config-format/typescript-mjs-using-nodejs/tsconfig.json b/test/build/config-format/typescript-mjs-using-nodejs/tsconfig.json
@@ -3,7 +3,6 @@
     "target": "esnext",
     "allowImportingTsExtensions": true,
     "rewriteRelativeImportExtensions": true,
-    "verbatimModuleSyntax": true,
     "module": "esnext"
   }
 }
diff --git a/test/build/config-format/typescript-mjs-using-nodejs/typescript.test.js b/test/build/config-format/typescript-mjs-using-nodejs/typescript.test.js
@@ -5,16 +5,20 @@ const { resolve } = require("path");
 
 describe("webpack cli", () => {
   it("should support typescript esnext file", async () => {
-    const [major, minor] = process.versions.node.split(".").map(Number);
-    const { exitCode, stderr, stdout } = await run(__dirname, ["-c", "./webpack.config.mts"], {
-      env: {
-        NODE_NO_WARNINGS: 1,
-        // Due nyc logic
-        WEBPACK_CLI_FORCE_LOAD_ESM_CONFIG: true,
+    const [major] = process.versions.node.split(".").map(Number);
+    const { exitCode, stderr, stdout } = await run(
+      __dirname,
+      ["-c", "./webpack.config.ts", "--disable-interpret"],
+      {
+        env: {
+          NODE_NO_WARNINGS: 1,
+          // Due nyc logic
+          WEBPACK_CLI_FORCE_LOAD_ESM_CONFIG: true,
+        },
+        // Fallback to `ts-node/esm` for old Node.js versions
+        nodeOptions: major >= 24 ? [] : ["--experimental-loader=ts-node/esm"],
       },
-      // Fallback to `ts-node/esm` for old Node.js versions
-      nodeOptions: major >= 22 && minor >= 6 ? [] : ["--experimental-loader=ts-node/esm"],
-    });
+    );
 
     expect(stderr).toBeFalsy(); // Deprecation warning logs on stderr
     expect(stdout).toBeTruthy();
diff --git a/test/build/config-format/typescript-mjs-using-nodejs/webpack.config.ts b/test/build/config-format/typescript-mjs-using-nodejs/webpack.config.ts
@@ -1,5 +1,4 @@
-/* eslint-disable node/no-unsupported-features/es-syntax */
-const path = require("path");
+import * as path from "path";
 
 // cspell:ignore elopment
 const mode: string = "dev" + "elopment";
@@ -12,4 +11,4 @@ const config = {
   },
 };
 
-module.exports = config;
+export default config;
diff --git a/test/build/config-format/typescript-ts-node-require/typescript.test.js b/test/build/config-format/typescript-ts-node-require/typescript.test.js
@@ -6,11 +6,7 @@ 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"], {
-      env: {
-        NODE_NO_WARNINGS: 1,
-        // Due nyc logic
-        WEBPACK_CLI_FORCE_LOAD_ESM_CONFIG: true,
-      },
+      env: { NODE_NO_WARNINGS: 1 },
       nodeOptions:
         major >= 22 && minor >= 6
           ? ["--no-experimental-strip-types", "--require=ts-node/register"]
diff --git a/test/build/config-format/typescript-using-interpret/main.ts b/test/build/config-format/typescript-using-interpret/main.ts
diff --git a/test/build/config-format/typescript-using-interpret/package.json b/test/build/config-format/typescript-using-interpret/package.json
diff --git a/test/build/config-format/typescript-using-interpret/tsconfig.json b/test/build/config-format/typescript-using-interpret/tsconfig.json
diff --git a/test/build/config-format/typescript-using-interpret/typescript.test.js b/test/build/config-format/typescript-using-interpret/typescript.test.js
diff --git a/test/build/config-format/typescript-using-interpret/webpack.config.ts b/test/build/config-format/typescript-using-interpret/webpack.config.ts
diff --git a/test/build/config-format/typescript-using-nodejs/package.json b/test/build/config-format/typescript-using-nodejs/package.json
@@ -0,0 +1,6 @@
+{
+  "type": "module",
+  "engines": {
+    "node": ">=18.12.0"
+  }
+}
diff --git a/test/build/config-format/typescript-using-nodejs/tsconfig.json b/test/build/config-format/typescript-using-nodejs/tsconfig.json
@@ -3,7 +3,6 @@
     "target": "esnext",
     "allowImportingTsExtensions": true,
     "rewriteRelativeImportExtensions": true,
-    "verbatimModuleSyntax": true,
-    "module": "commonjs"
+    "module": "esnext"
   }
 }
diff --git a/test/build/config-format/typescript-using-nodejs/typescript.test.js b/test/build/config-format/typescript-using-nodejs/typescript.test.js
@@ -1,22 +1,26 @@
+// 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 esnext file", async () => {
-    const [major, minor] = process.versions.node.split(".").map(Number);
-    const { exitCode, stderr, stdout } = await run(__dirname, ["-c", "./webpack.config.ts"], {
-      env: {
-        NODE_NO_WARNINGS: 1,
-        // Due nyc logic
-        WEBPACK_CLI_FORCE_LOAD_ESM_CONFIG: true,
+    const [major] = process.versions.node.split(".").map(Number);
+    const { exitCode, stderr, stdout } = await run(
+      __dirname,
+      ["-c", "./webpack.config.ts", "--disable-interpret"],
+      {
+        env: {
+          NODE_NO_WARNINGS: 1,
+          // Due nyc logic
+          WEBPACK_CLI_FORCE_LOAD_ESM_CONFIG: true,
+        },
+        // Fallback to `ts-node/esm` for old Node.js versions
+        nodeOptions: major >= 24 ? [] : ["--experimental-loader=ts-node/esm"],
       },
-      // Fallback to `ts-node/esm` for old Node.js versions
-      // We don't use Node.js@22 because it doesn't allow to load typescript files where we have `package.json` in root
-      nodeOptions: major >= 22 && minor >= 6 ? [] : ["--experimental-loader=ts-node/esm"],
-    });
+    );
 
-    expect(stderr).toBeFalsy();
+    expect(stderr).toBeFalsy(); // Deprecation warning logs on stderr
     expect(stdout).toBeTruthy();
     expect(exitCode).toBe(0);
     expect(existsSync(resolve(__dirname, "dist/foo.bundle.js"))).toBeTruthy();
diff --git a/test/build/config-format/typescript-using-nodejs/webpack.config.ts b/test/build/config-format/typescript-using-nodejs/webpack.config.ts
@@ -1,4 +1,4 @@
-const path = require("path");
+import * as path from "path";
 
 // cspell:ignore elopment
 const mode: string = "dev" + "elopment";
@@ -11,4 +11,4 @@ const config = {
   },
 };
 
-module.exports = config;
+export default config;

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,6 @@`
`3`	`3`	`"target": "esnext",`
`4`	`4`	`"allowImportingTsExtensions": true,`
`5`	`5`	`"rewriteRelativeImportExtensions": true,`
`6`		`- "verbatimModuleSyntax": true,`
`7`		`- "module": "commonjs"`
	`6`	`+ "module": "esnext"`
`8`	`7`	`}`
`9`	`8`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,3 @@`
`1`		`-/* eslint-disable node/no-unsupported-features/es-syntax */`
`2`	`1`	`import * as path from "path";`
`3`	`2`
`4`	`3`	`// cspell:ignore elopment`