chore: resolve "use strict" TODO (#19705)

.editorconfig

@@ -12,14 +12,14 @@ max_line_length = 80
 indent_style = space
 indent_size = 2
 
-[test/cases/parsing/bom/bomfile.{css,js}]
-charset = utf-8-bom
-
-[test/configCases/css/no-extra-runtime-in-js/source.text]
-insert_final_newline = false
-
 [*.md]
 trim_trailing_whitespace = false
 
 [*.snap]
 trim_trailing_whitespace = false
+
+[test/cases/parsing/bom/bomfile.{css,js}]
+charset = utf-8-bom
+
+[test/configCases/css/no-extra-runtime-in-js/source.text]
+insert_final_newline = false

eslint.config.mjs

@@ -151,9 +151,6 @@ export default defineConfig([
 	{
 		files: ["test/**/*.js"],
 		rules: {
-			// TODO enable me
-			strict: "off",
-
 			// Some our tests contain `package.json` without `engines`, but tests should work on Node.js@10, so let's disable it
 			"n/prefer-node-protocol": "off",
 

test/BinaryMiddleware.unittest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const BinaryMiddleware = require("../lib/serialization/BinaryMiddleware");
 const SerializerMiddleware = require("../lib/serialization/SerializerMiddleware");
 

test/Cli.basictest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { getArguments, processArguments } = require("../").cli;
 
 describe("Cli", () => {

test/ConfigCacheTestCases.longtest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./ConfigTestCases.template");
 
 describeCases({

test/ConfigTestCases.basictest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./ConfigTestCases.template");
 
 describeCases({

test/Defaults.unittest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 require("./helpers/warmup-webpack");
 
 const path = require("path");

test/HotTestCasesAsyncNode.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./HotTestCases.template");
 
 describe("HotTestCases", () => {

test/HotTestCasesNode.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./HotTestCases.template");
 
 describe("HotTestCases", () => {

test/HotTestCasesWeb.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./HotTestCases.template");
 
 describe("HotTestCases", () => {

test/HotTestCasesWebWorker.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./HotTestCases.template");
 
 describe("HotTestCases", () => {

test/LazySet.unittest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const LazySet = require("../lib/util/LazySet");
 
 describe("LazySet", () => {

test/PersistentCaching.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 require("./helpers/warmup-webpack");
 
 const fs = require("fs");

test/Queue.unittest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const Queue = require("../lib/util/Queue");
 
 describe("Queue", () => {

test/TestCasesAllCombined.longtest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesCachePack.longtest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const path = require("path");
 const { describeCases } = require("./TestCases.template");
 

test/TestCasesDevelopment.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesDevtoolCheapSourceMap.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesDevtoolEval.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesDevtoolEvalCheapModuleSourceMap.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesDevtoolEvalCheapSourceMap.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesDevtoolEvalDeterministicModuleIds.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesDevtoolEvalNamedModules.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesDevtoolEvalSourceMap.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesDevtoolInlineCheapSourceMap.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesDevtoolInlineSourceMap.longtest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesDevtoolSourceMap.longtest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesHot.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const webpack = require("..");
 const { describeCases } = require("./TestCases.template");
 

test/TestCasesMinimizedSourceMap.longtest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesModule.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const vm = require("vm");
 const { describeCases } = require("./TestCases.template");
 

test/TestCasesNormal.basictest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/TestCasesProdGlobalUsed.test.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCasesProdGlobalUsed", () => {

test/TestCasesProduction.longtest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./TestCases.template");
 
 describe("TestCases", () => {

test/URLAbsoluteSpecifier.unittest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { getProtocol, getScheme } = require("../lib/util/URLAbsoluteSpecifier");
 
 /**

test/WasmHashes.unittest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { createHash, randomBytes } = require("crypto");
 
 const wasmHashes = {

test/WatchCacheUnaffectedTestCases.longtest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./WatchTestCases.template");
 
 describeCases({

test/WatchTestCases.longtest.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const { describeCases } = require("./WatchTestCases.template");
 
 describeCases({

test/benchmarkCases/cache-filesystem/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index",

test/benchmarkCases/context-commonjs/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index"

test/benchmarkCases/context-esm/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index"

test/benchmarkCases/devtool-eval-source-map/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index",

test/benchmarkCases/devtool-eval/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index",

test/benchmarkCases/devtool-source-map/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index",

test/benchmarkCases/future-defaults/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index",

test/benchmarkCases/lodash/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index"

test/benchmarkCases/many-chunks-commonjs/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index"

test/benchmarkCases/many-chunks-esm/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index"

test/benchmarkCases/many-modules-commonjs/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index"

test/benchmarkCases/many-modules-esm/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index"

test/benchmarkCases/minimal/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index"

test/benchmarkCases/react/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index"

test/benchmarkCases/three-long/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index"

test/benchmarkCases/typescript-long-on-schedule/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 /** @type {import("../../../").Configuration} */
 module.exports = {
 	entry: "./index",

test/cases/async-modules/micro-ticks-parents/test.filter.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = () =>
 	// In node 10 v8 has a bug which inserts an additional micro-tick into async functions
 	!process.version.startsWith("v10.");

test/cases/chunks/destructuring-assignment/test.filter.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = function filter(config) {
 	// This test can't run in development mode
 	return config.mode !== "development";

test/cases/chunks/destructuring-assignment/warnings.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[/`webpackExports` could not be used with destructuring assignment./]
 ];

test/cases/chunks/runtime/test.filter.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = config =>
 	// This test can't run in development mode as it depends on the flagIncludedChunks optimization
 	config.mode !== "development";

test/cases/chunks/runtime/webpack.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = {
 	optimization: {
 		moduleIds: "named"

test/cases/chunks/weird-reference-to-entry/errors.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[
 		/It's not allowed to load an initial chunk on demand\. The chunk name "main" is already used by an entrypoint\./

test/cases/cjs-interop/non-existing-export/warnings.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[
 		/export 'named' \(imported as 'named'\) was not found in '\.\/module' \(possible exports: default\)/

test/cases/cjs-tree-shaking/mutate/warnings.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[/export 'a' \(imported as 'a'\) was not found/],
 	[/export 'a' \(imported as 'a'\) was not found/]

test/cases/compile/error-hide-stack/errors.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[/Module build failed( \(from [^)]+\))?:\nMessage/, { details: /Stack/ }]
 ];

test/cases/compile/error-hide-stack/infrastructure-log.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	/^Pack got invalid because of write to: Compilation\/modules.+loader\.js!$/
 ];

test/cases/context/issue-5750/warnings.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[/Critical dependency: Contexts can't use RegExps with the 'g' or 'y' flags/]
 ];

test/cases/errors/case-sensitive/test.filter.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const fs = require("fs");
 const path = require("path");
 

test/cases/errors/case-sensitive/warnings.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[
 		/There are multiple modules with names that only differ in casing/,

test/cases/errors/crash-missing-import/errors.js

@@ -1 +1,3 @@
+"use strict";
+
 module.exports = [[/Module not found/]];

test/cases/errors/harmony-import-missing/errors.js

@@ -1 +1,3 @@
+"use strict";
+
 module.exports = [[/Can't resolve '.\/missing'/]];

test/cases/errors/harmony-import-missing2/errors.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[/Can't resolve '.\/missing1'/],
 	[/Can't resolve '.\/missing2'/]

test/cases/errors/load-module-error/errors.js

@@ -1 +1,3 @@
+"use strict";
+
 module.exports = [[/err: abc/], [/The loaded module contains errors/]];

test/cases/errors/load-module-error/infrastructure-log.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	/^Pack got invalid because of write to: Compilation\/modules|json.+error-loader\.js!/
 ];

test/cases/errors/loader-error-warning/errors.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[/abc/, /Emitted value instead of an instance of Error/, /error-loader\.js/],
 	[/def/, /Emitted value instead of an instance of Error/, /error-loader\.js/]

test/cases/errors/loader-error-warning/warnings.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[/xyz/, /Emitted value instead of an instance of Error/, /warning-loader\.js/]
 ];

test/cases/errors/mjs-non-fully-specified/errors.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[/Can't resolve '.\/dependency'/, /Did you mean 'dependency\.js'\?/]
 ];

test/cases/errors/module-request/errors.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[/Can't resolve 'dependency\.js'/, /Did you mean '\.\/dependency\.js'\?/]
 ];

test/cases/esm/import-meta/test.filter.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const supportsRequireInModule = require("../../../helpers/supportsRequireInModule");
 
 module.exports = config => !config.module || supportsRequireInModule();

test/cases/esm/import-meta/warnings.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[
 		/Accessing import.meta directly is unsupported \(only property access or destructuring is supported\)/

test/cases/inner-graph/class-dynamic-props/test.filter.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const supportsClassFields = require("../../../helpers/supportsClassFields");
 
 module.exports = () => supportsClassFields();

test/cases/inner-graph/extend-class/test.filter.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const supportsClassStaticBlock = require("../../../helpers/supportsClassStaticBlock");
 
 module.exports = () => supportsClassStaticBlock();

test/cases/inner-graph/extend-class2/test.filter.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const supportsClassStaticBlock = require("../../../helpers/supportsClassStaticBlock");
 
 module.exports = () => supportsClassStaticBlock();

test/cases/json/import-assertions-type-json/errors.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[{ moduleName: /data.poison/, message: /Unexpected token .+ JSON/ }]
 ];

test/cases/json/import-assertions-type-json/infrastructure-log.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	/^Pack got invalid because of write to: Compilation\/modules|json.+json\/data\/poison$/
 ];

test/cases/json/import-by-name-with-concatenation/warnings.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[
 		/Should not import the named export '2' \(imported as 'c'\) from default-exporting module \(only default export is available soon\)/

test/cases/json/import-by-name/warnings.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[
 		/Should not import the named export '2' \(imported as 'c'\) from default-exporting module \(only default export is available soon\)/

test/cases/json/import-with-type-json/errors.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[{ moduleName: /data.poison/, message: /Unexpected token .+ JSON/ }]
 ];

test/cases/json/import-with-type-json/infrastructure-log.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	/^Pack got invalid because of write to: Compilation\/modules|json.+json\/data\/poison$/
 ];

test/cases/json/reexport/warnings.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[
 		/Should not import the named export 'named' \(reexported as 'fNamed'\) from default-exporting module \(only default export is available soon\)/

test/cases/large/big-assets/test.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = {
 	timeout: 120000
 };

test/cases/large/big-assets/test.filter.js

@@ -1 +1,3 @@
+"use strict";
+
 module.exports = () => !process.env.CI;

test/cases/large/many-replacements/test.config.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = {
 	timeout: 120000
 };

test/cases/large/many-replacements/test.filter.js

@@ -1 +1,3 @@
+"use strict";
+
 module.exports = () => !process.env.CI;

test/cases/loaders/_esm-loader-type/test.filter.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = () =>
 	// TODO need fix in v8 https://github.com/nodejs/node/issues/35889
 	// TODO otherwise this test case cause segment fault

test/cases/loaders/context/test.filter.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const supportsRequireInModule = require("../../../helpers/supportsRequireInModule");
 
 module.exports = config => !config.module || supportsRequireInModule();

test/cases/loaders/emit-file/test.filter.js

@@ -1 +1,3 @@
+"use strict";
+
 module.exports = config => !config.module;

test/cases/loaders/import-module/test.filter.js

@@ -1 +1,3 @@
+"use strict";
+
 module.exports = config => !config.module;

test/cases/loaders/no-string/errors.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[
 		{

test/cases/loaders/no-string/infrastructure-log.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	/^Pack got invalid because of write to: Compilation\/modules.+no-string[/\\]loader\.js!.+no-string[/\\]file\.js$/
 ];

test/cases/loaders/pug-loader/test.filter.js

@@ -1,3 +1,5 @@
+"use strict";
+
 const supportsRequireInModule = require("../../../helpers/supportsRequireInModule");
 
 module.exports = config => !config.module || supportsRequireInModule();

test/cases/mjs/non-enumerable/errors.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[
 		/export '__esModule' \(imported as 'm1'\) was not found in '\.\/analyzable-module\.js'/

test/cases/mjs/non-enumerable/warnings.js

@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = [
 	[
 		/export '__esModule' \(imported as 'm1'\) was not found in '\.\/analyzable-module\.js'/