fix: verbose flag functionality

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

@@ -13,16 +13,12 @@ class StatsGroup extends GroupHelper {
     }
 
     resolveOptions() {
-        if (this.args.verbose && this.args.stats) {
+        if (this.args.verbose && process.argv.includes('--stats')) {
             logger.warn('Conflict between "verbose" and "stats" options. Using verbose.');
-            this.opts.option.stats = {
-                verbose: true,
-            };
+            this.opts.options.stats = 'verbose';
         } else {
             if (this.args.verbose) {
-                this.opts.option.stats = {
-                    verbose: true,
-                };
+                this.opts.options.stats = 'verbose';
             } else if (!StatsGroup.validOptions().includes(this.args.stats)) {
                 logger.warn(`'${this.args.stats}' is invalid value for stats. Using 'normal' option for stats`);
                 this.opts.options.stats = 'normal';

test/stats/stats.test.js

@@ -45,6 +45,13 @@ describe('stats flag', () => {
         expect(stdout).toBeTruthy();
     });
 
+    it('should warn when --verbose & --stats are passed together', () => {
+        const { stderr, stdout } = run(__dirname, ['--verbose', '--stats', 'normal']);
+        expect(stderr).toBeTruthy();
+        expect(stderr).toContain('Conflict between "verbose" and "stats" options');
+        expect(stdout).toBeTruthy();
+    });
+
     it('should warn when an unknown flag stats value is passed', () => {
         const { stderr, stdout } = run(__dirname, ['--stats', 'foo']);
         expect(stderr).toBeTruthy();

test/verbose/.gitignore

@@ -0,0 +1,3 @@
+yarn.lock
+package-lock.json
+node_modules/

test/verbose/index.js

@@ -0,0 +1,4 @@
+require('react');
+require('react-dom');
+require('redux');
+require('react-redux');

test/verbose/package.json

@@ -0,0 +1,8 @@
+{
+  "dependencies": {
+    "react": "^16.13.0",
+    "react-dom": "^16.13.0",
+    "redux": "^4.0.5",
+    "react-redux": "^7.2.0"
+  }
+}

test/verbose/verbose-flag.test.js

@@ -0,0 +1,11 @@
+'use strict';
+// eslint-disable-next-line node/no-unpublished-require
+const { run } = require('../utils/test-utils');
+
+describe('verbose flag', () => {
+    it('should accept --verbose', () => {
+        const { stderr, stdout } = run(__dirname, ['--verbose']);
+        expect(stderr).toBeFalsy();
+        expect(stdout).toBeTruthy();
+    });
+});