test: cleanup (#2688)

test/build/basic/basic.test.js

@@ -5,147 +5,135 @@ const { run } = require('../../utils/test-utils');
 
 describe('bundle command', () => {
     it('should work without command (default command)', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, [], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, []);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work without command and options (default command)', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['--mode', 'development'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['--mode', 'development']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with multiple entries syntax without command (default command)', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['./src/index.js', './src/other.js'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['./src/index.js', './src/other.js']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with multiple entries syntax without command with options (default command)', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['./src/index.js', './src/other.js', '--mode', 'development'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['./src/index.js', './src/other.js', '--mode', 'development']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with multiple entries syntax without command with options #2 (default command)', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['--mode', 'development', './src/index.js', './src/other.js'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['--mode', 'development', './src/index.js', './src/other.js']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with multiple entries syntax without command with options #3 (default command)', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['./src/index.js', './src/other.js', '--entry', './src/again.js'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['./src/index.js', './src/other.js', '--entry', './src/again.js']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with and override entries from the configuration', async () => {
-        const { exitCode, stderr, stdout } = await run(
-            __dirname,
-            ['./src/index.js', './src/other.js', '--config', './entry.config.js'],
-            false,
-        );
+        const { exitCode, stderr, stdout } = await run(__dirname, ['./src/index.js', './src/other.js', '--config', './entry.config.js']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with the "build" alias', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['build'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['build']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with "bundle" alias', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['bundle'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['bundle']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with the "b" alias', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['b'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['b']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with entries syntax using the "build" alias', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['build', './src/index.js'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['build', './src/index.js']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with entries syntax using the "bundle" alias', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['bundle', './src/index.js'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['bundle', './src/index.js']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with entries syntax using the "b" alias', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['b', './src/index.js'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['b', './src/index.js']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with multiple entries syntax using the "build" alias', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['build', './src/index.js', './src/other.js'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['build', './src/index.js', './src/other.js']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with multiple entries syntax using the "build" alias and options', async () => {
-        const { exitCode, stderr, stdout } = await run(
-            __dirname,
-            ['build', './src/index.js', './src/other.js', '--mode', 'development'],
-            false,
-        );
+        const { exitCode, stderr, stdout } = await run(__dirname, ['build', './src/index.js', './src/other.js', '--mode', 'development']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work with multiple entries syntax using the "build" alias and options', async () => {
-        const { exitCode, stderr, stdout } = await run(
-            __dirname,
-            ['build', '--mode', 'development', './src/index.js', './src/other.js'],
-            false,
-        );
+        const { exitCode, stderr, stdout } = await run(__dirname, ['build', '--mode', 'development', './src/index.js', './src/other.js']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should log error and suggest right name on the "buil" command', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['buil'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['buil']);
 
         expect(exitCode).toBe(2);
         expect(stderr).toContain("Unknown command or entry 'buil'");

test/build/build-variable/build-variable.test.js

@@ -4,7 +4,7 @@ const { run } = require('../../utils/test-utils');
 
 describe('bundle variable', () => {
     it('compiles without flags and export variable', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, [], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, []);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();

test/build/bundle-variable/bundle-variable.test.js

@@ -4,7 +4,7 @@ const { run } = require('../../utils/test-utils');
 
 describe('bundle variable', () => {
     it('compiles without flags and export variable', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, [], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, []);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();

test/build/config-format/commonjs-default/commonjs-default.test.js

@@ -2,7 +2,7 @@ const { run } = require('../../../utils/test-utils');
 
 describe('webpack cli', () => {
     it('should support CommonJS file', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['-c', 'webpack.config.cjs'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['-c', 'webpack.config.cjs']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();

test/build/config-format/commonjs/commonjs.test.js

@@ -2,7 +2,7 @@ const { run } = require('../../../utils/test-utils');
 
 describe('webpack cli', () => {
     it('should support CommonJS file', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['-c', 'webpack.config.cjs'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['-c', 'webpack.config.cjs']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();

test/build/config-lookup/dotfolder-array/dotfolder-array.test.js

@@ -5,7 +5,7 @@ const { run } = require('../../../utils/test-utils');
 
 describe('dotfolder array config lookup', () => {
     it('should find a webpack array configuration in a dotfolder', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, [], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, []);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();

test/build/config-lookup/dotfolder-single/dotfolder-single.test.js

@@ -7,7 +7,7 @@ const { run } = require('../../../utils/test-utils');
 
 describe('dotfolder single config lookup', () => {
     it('should find a webpack configuration in a dotfolder', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, [], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, []);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();

test/build/config-lookup/relative/basic-config.test.js

@@ -4,15 +4,15 @@ const { run } = require('../../../utils/test-utils');
 
 describe('relative path to config', () => {
     it('should work', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['-c', 'webpack.config.js', '--output-path', './binary/a'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['-c', 'webpack.config.js', '--output-path', './binary/a']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();
     });
 
     it('should work #2', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['-c', './webpack.config.js', '--output-path', './binary/b'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['-c', './webpack.config.js', '--output-path', './binary/b']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();

test/build/config-name/config-name.test.js

@@ -4,7 +4,7 @@ const { run } = require('../../utils/test-utils');
 
 describe('--config-name flag', () => {
     it('should select only the config whose name is passed with --config-name', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['--config-name', 'first'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['--config-name', 'first']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
@@ -14,7 +14,7 @@ describe('--config-name flag', () => {
     });
 
     it('should work with multiple values for --config-name', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['--config-name', 'first', '--config-name', 'third'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['--config-name', 'first', '--config-name', 'third']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
@@ -39,15 +39,15 @@ describe('--config-name flag', () => {
     });
 
     it('should log error if invalid config name is provided', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['--config-name', 'test'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['--config-name', 'test']);
 
         expect(exitCode).toBe(2);
         expect(stderr).toContain('Configuration with the name "test" was not found.');
         expect(stdout).toBeFalsy();
     });
 
     it('should log error if multiple configurations are not found', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['--config-name', 'test', '-c', 'single-config.js'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['--config-name', 'test', '-c', 'single-config.js']);
 
         expect(exitCode).toBe(2);
         expect(stderr).toContain('Configuration with the name "test" was not found.');
@@ -80,7 +80,7 @@ describe('--config-name flag', () => {
     });
 
     it('should work with config as a function', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['--config', 'function-config.js', '--config-name', 'first'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['--config', 'function-config.js', '--config-name', 'first']);
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
@@ -104,7 +104,7 @@ describe('--config-name flag', () => {
     });
 
     it('should log error if invalid config name is provided ', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['--config', 'function-config.js', '--config-name', 'test'], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['--config', 'function-config.js', '--config-name', 'test']);
 
         expect(exitCode).toBe(2);
         expect(stderr).toContain('Configuration with the name "test" was not found.');

test/build/config/absent/config-absent.test.js

@@ -5,7 +5,7 @@ const { run } = require('../../../utils/test-utils');
 
 describe('Config:', () => {
     it('supplied config file is absent', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['-c', path.resolve(__dirname, 'webpack.config.js')], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['-c', path.resolve(__dirname, 'webpack.config.js')]);
 
         // should throw with correct exit code
         expect(exitCode).toBe(2);

test/build/config/basic/basic-config.test.js

@@ -5,11 +5,12 @@ const { run } = require('../../../utils/test-utils');
 
 describe('basic config file', () => {
     it('is able to understand and parse a very basic configuration file', async () => {
-        const { exitCode, stderr, stdout } = await run(
-            __dirname,
-            ['-c', resolve(__dirname, 'webpack.config.js'), '--output-path', './binary'],
-            false,
-        );
+        const { exitCode, stderr, stdout } = await run(__dirname, [
+            '-c',
+            resolve(__dirname, 'webpack.config.js'),
+            '--output-path',
+            './binary',
+        ]);
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();

test/build/config/defaults/basic-config/default-js-config.test.js

@@ -4,7 +4,7 @@ const { run, isWebpack5 } = require('../../../../utils/test-utils');
 
 describe('Zero Config', () => {
     it('runs when config is present but not supplied via flag', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, [], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, []);
 
         expect(exitCode).toEqual(0);
         expect(stderr).toBeFalsy();

test/build/config/defaults/cjs-config/default-cjs-config.test.js

@@ -4,7 +4,7 @@ const { run, isWebpack5 } = require('../../../../utils/test-utils');
 
 describe('Default Config:', () => {
     it('Should be able to pick cjs config by default', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, [], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, []);
 
         expect(exitCode).toEqual(0);
         expect(stderr).toBeFalsy();

test/build/config/defaults/dot-webpack-directory-webpackfile/multiple-location-config.test.js

@@ -5,7 +5,7 @@ const { run } = require('../../../../utils/test-utils');
 
 describe('multiple dev config files with webpack.config.js', () => {
     it('Uses webpack.config.development.js', async () => {
-        const { stdout, stderr, exitCode } = await run(__dirname, [], false);
+        const { stdout, stderr, exitCode } = await run(__dirname, []);
         expect(exitCode).toEqual(0);
         expect(stderr).toBeFalsy();
         expect(stdout).not.toBe(undefined);

test/build/config/defaults/dot-webpack-directory/dev-none-config.test.js

@@ -5,7 +5,7 @@ const { run } = require('../../../../utils/test-utils');
 
 describe('multiple config files', () => {
     it('Uses dev config when both dev and none are present', async () => {
-        const { stdout, stderr, exitCode } = await run(__dirname, [], false);
+        const { stdout, stderr, exitCode } = await run(__dirname, []);
         expect(exitCode).toEqual(0);
         expect(stderr).toBeFalsy();
         expect(stdout).not.toBe(undefined);

test/build/config/defaults/with-mode/multiple-config.test.js

@@ -5,7 +5,7 @@ const { run } = require('../../../../utils/test-utils');
 
 describe('multiple config files', () => {
     it('Uses dev config when development mode is supplied', async () => {
-        const { stdout, stderr, exitCode } = await run(__dirname, ['--mode', 'development'], false);
+        const { stdout, stderr, exitCode } = await run(__dirname, ['--mode', 'development']);
         expect(exitCode).toEqual(0);
         expect(stderr).toBeFalsy();
         expect(stdout).not.toBe(undefined);

test/build/config/invalid-path/invalid-path.test.js

@@ -4,7 +4,7 @@ const { run } = require('../../../utils/test-utils');
 
 describe('basic config file', () => {
     it('is able to understand and parse a very basic configuration file', async () => {
-        const { exitCode, stderr, stdout } = await run(__dirname, ['-c', path.resolve(__dirname, 'invalid-webpack.config.js')], false);
+        const { exitCode, stderr, stdout } = await run(__dirname, ['-c', path.resolve(__dirname, 'invalid-webpack.config.js')]);
 
         expect(exitCode).toBe(2);
         expect(stderr).toContain(`Failed to load '${path.resolve(__dirname, 'invalid-webpack.config.js')}' config`);

test/build/config/multiple-with-one-compilation/multiple-with-one-compilation.test.js

@@ -5,11 +5,12 @@ const { run } = require('../../../utils/test-utils');
 
 describe('basic config file', () => {
     it('is able to understand and parse a very basic configuration file', async () => {
-        const { exitCode, stderr, stdout } = await run(
-            __dirname,
-            ['-c', resolve(__dirname, 'webpack.config.js'), '--output-path', './binary'],
-            false,
-        );
+        const { exitCode, stderr, stdout } = await run(__dirname, [
+            '-c',
+            resolve(__dirname, 'webpack.config.js'),
+            '--output-path',
+            './binary',
+        ]);
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
         expect(stdout).toBeTruthy();