feat(webpack-cli): add --no-stats flag

packages/webpack-cli/README.md

@@ -56,6 +56,7 @@ Options
   -v, --version            Get current version
   --node-args string[]     NodeJS flags
   --stats string           It instructs webpack on how to treat the stats
+  --no-stats               Disables stats output
   --verbose                It tells webpack to output all the information
 ```
 

packages/webpack-cli/lib/groups/StatsGroup.js

@@ -5,7 +5,7 @@ const logger = require('../utils/logger');
  */
 class StatsGroup extends GroupHelper {
     static validOptions() {
-        return ['none', 'errors-only', 'minimal', 'normal', 'detailed', 'verbose', 'errors-warnings', true];
+        return ['none', 'errors-only', 'minimal', 'normal', 'detailed', 'verbose', 'errors-warnings', true, false];
     }
 
     constructor(options) {

packages/webpack-cli/lib/utils/cli-flags.js

@@ -245,6 +245,14 @@ module.exports = {
             description: 'It instructs webpack on how to treat the stats e.g. verbose',
             link: 'https://webpack.js.org/configuration/stats/#stats',
         },
+        {
+            name: 'no-stats',
+            usage: '--no-stats',
+            type: Boolean,
+            group: DISPLAY_GROUP,
+            description: 'Disables stats output',
+            link: 'https://webpack.js.org/configuration/stats/#stats',
+        },
         {
             name: 'verbose',
             usage: '--verbose',

test/no-stats/with-config/a.js

@@ -0,0 +1,4 @@
+require('url');
+require('path');
+
+console.log('--no-stats with config test');

test/no-stats/with-config/no-stats-with-config.test.js

@@ -0,0 +1,29 @@
+'use strict';
+// eslint-disable-next-line node/no-unpublished-require
+const { run } = require('../../utils/test-utils');
+// eslint-disable-next-line node/no-extraneous-require
+const { version } = require('webpack');
+
+describe('stats flag', () => {
+    it(`should use stats 'detailed' as defined in webpack config`, () => {
+        const { stderr, stdout } = run(__dirname, []);
+
+        expect(stderr).toBeFalsy();
+        if (version.startsWith('5')) {
+            expect(stdout).toContain(`stats: { preset: 'detailed' }`);
+        } else {
+            expect(stdout).toContain(`stats: 'detailed'`);
+        }
+    });
+
+    it(`should use --no-stats and override value in config`, () => {
+        const { stderr, stdout } = run(__dirname, ['--no-stats']);
+
+        expect(stderr).toBeFalsy();
+        if (version.startsWith('5')) {
+            expect(stdout).toContain(`stats: { preset: 'none' }`);
+        } else {
+            expect(stdout).toContain(`stats: false`);
+        }
+    });
+});

test/no-stats/with-config/webpack.config.js

@@ -0,0 +1,9 @@
+// eslint-disable-next-line node/no-unpublished-require
+const WebpackCLITestPlugin = require('../../utils/webpack-cli-test-plugin');
+
+module.exports = {
+    mode: 'development',
+    entry: './main.js',
+    stats: 'detailed',
+    plugins: [new WebpackCLITestPlugin()],
+};

test/no-stats/with-flags/main.js

@@ -0,0 +1,4 @@
+require('url');
+require('path');
+
+console.log('--no-stats test');

test/no-stats/with-flags/no-stats.test.js

@@ -0,0 +1,51 @@
+'use strict';
+// eslint-disable-next-line node/no-unpublished-require
+const { run } = require('../../utils/test-utils');
+// eslint-disable-next-line node/no-extraneous-require
+const { version } = require('webpack');
+
+describe('stats flag', () => {
+    it('should accept --no-stats as boolean', () => {
+        const { stderr, stdout } = run(__dirname, ['--no-stats']);
+
+        expect(stderr).toBeFalsy();
+        if (version.startsWith('5')) {
+            expect(stdout).toContain(`stats: { preset: 'none' }`);
+        } else {
+            expect(stdout).toContain('stats: false');
+        }
+    });
+
+    it('should warn and use --no-stats when stats and no-stats both are provided', () => {
+        const { stderr, stdout } = run(__dirname, ['--stats', 'verbose', '--no-stats']);
+
+        expect(stderr).toContain(`You provided both --stats and --no-stats. We will use only the last of these flags`);
+        if (version.startsWith('5')) {
+            expect(stdout).toContain(`stats: { preset: 'none' }`);
+        } else {
+            expect(stdout).toContain('stats: false');
+        }
+    });
+
+    it('should warn and use --stats when stats and no-stats both are provided', () => {
+        const { stderr, stdout } = run(__dirname, ['--no-stats', '--stats', 'verbose']);
+
+        expect(stderr).toContain(`You provided both --stats and --no-stats. We will use only the last of these flags`);
+        if (version.startsWith('5')) {
+            expect(stdout).toContain(`stats: { preset: 'verbose' }`);
+        } else {
+            expect(stdout).toContain(`stats: 'verbose'`);
+        }
+    });
+
+    it('should use --verbose over --no-stats', () => {
+        const { stderr, stdout } = run(__dirname, ['--no-stats', '--verbose']);
+
+        expect(stderr).toBeFalsy();
+        if (version.startsWith('5')) {
+            expect(stdout).toContain(`stats: { preset: 'verbose' }`);
+        } else {
+            expect(stdout).toContain(`stats: 'verbose'`);
+        }
+    });
+});

test/no-stats/with-flags/webpack.config.js

@@ -0,0 +1,8 @@
+// eslint-disable-next-line node/no-unpublished-require
+const WebpackCLITestPlugin = require('../../utils/webpack-cli-test-plugin');
+
+module.exports = {
+    mode: 'development',
+    entry: './main.js',
+    plugins: [new WebpackCLITestPlugin()],
+};