fix: move progress option to client.progress

Author: ylemkimon

bin/cli-flags.js

@@ -21,10 +21,15 @@ module.exports = {
       negative: true,
     },
     {
-      name: 'progress',
+      name: 'client-progress',
       type: Boolean,
-      describe: 'Print compilation progress in percentage',
+      describe: 'Print compilation progress in percentage in the browser',
       group: BASIC_GROUP,
+      processor(opts) {
+        opts.client = opts.client || {};
+        opts.client.progress = opts.clientProgress;
+        delete opts.clientProgress;
+      },
     },
     {
       name: 'hot-only',

examples/cli/progress/webpack.config.js

@@ -7,7 +7,7 @@ const { setup } = require('../../util');
 module.exports = setup({
   context: __dirname,
   entry: './app.js',
-  devServer: {
+  client: {
     progress: true,
   },
 });

lib/Server.js

@@ -58,7 +58,7 @@ class Server {
       this.options
     );
 
-    if (this.options.progress) {
+    if (this.options.client.progress) {
       this.setupProgressPlugin();
     }
     this.setupHooks();
@@ -83,7 +83,6 @@ class Server {
   }
 
   setupProgressPlugin() {
-    // for browser console output
     new webpack.ProgressPlugin((percent, msg, addInfo) => {
       percent = Math.floor(percent * 100);
 
@@ -564,8 +563,8 @@ class Server {
         this.sockWrite([connection], 'liveReload');
       }
 
-      if (this.options.progress) {
-        this.sockWrite([connection], 'progress', this.options.progress);
+      if (this.options.client.progress) {
+        this.sockWrite([connection], 'progress', this.options.client.progress);
       }
 
       if (this.options.clientOverlay) {

lib/options.json

@@ -83,6 +83,9 @@
         },
         "logging": {
           "enum": ["none", "error", "warn", "info", "log", "verbose"]
+        },
+        "progress": {
+          "type": "boolean"
         }
       },
       "additionalProperties": false
@@ -292,9 +295,6 @@
         }
       ]
     },
-    "progress": {
-      "type": "boolean"
-    },
     "proxy": {
       "anyOf": [
         {
@@ -405,7 +405,6 @@
       "openPage": "should be {String|Array} (https://webpack.js.org/configuration/dev-server/#devserveropenpage)",
       "overlay": "should be {Boolean|Object} (https://webpack.js.org/configuration/dev-server/#devserveroverlay)",
       "port": "should be {Number|String|Null} (https://webpack.js.org/configuration/dev-server/#devserverport)",
-      "progress": "should be {Boolean} (https://webpack.js.org/configuration/dev-server/#devserverprogress---cli-only)",
       "proxy": "should be {Object|Array} (https://webpack.js.org/configuration/dev-server/#devserverproxy)",
       "public": "should be {String} (https://webpack.js.org/configuration/dev-server/#devserverpublic)",
       "setupExitSignals": "should be {Boolean} (https://webpack.js.org/configuration/dev-server/#devserversetupexitsignals)",

test/__snapshots__/Validation.test.js.snap

@@ -39,5 +39,5 @@ exports[`Validation validation should fail validation for invalid \`static\` con
 exports[`Validation validation should fail validation for no additional properties 1`] = `
 "Invalid configuration object. Object has been initialized using a configuration object that does not match the API schema.
  - configuration has an unknown property 'additional'. These properties are valid:
-   object { bonjour?, client?, compress?, dev?, firewall?, headers?, historyApiFallback?, host?, hot?, http2?, https?, injectClient?, injectHot?, liveReload?, onAfterSetupMiddleware?, onBeforeSetupMiddleware?, onListening?, open?, openPage?, overlay?, port?, progress?, proxy?, public?, setupExitSignals?, static?, stdin?, transportMode?, useLocalIp? }"
+   object { bonjour?, client?, compress?, dev?, firewall?, headers?, historyApiFallback?, host?, hot?, http2?, https?, injectClient?, injectHot?, liveReload?, onAfterSetupMiddleware?, onBeforeSetupMiddleware?, onListening?, open?, openPage?, overlay?, port?, proxy?, public?, setupExitSignals?, static?, stdin?, transportMode?, useLocalIp? }"
 `;

test/cli/cli.test.js

@@ -98,31 +98,6 @@ runCLITest('CLI', () => {
       .catch(done);
   });
 
-  it('--progress', (done) => {
-    testBin('--progress')
-      .then((output) => {
-        expect(output.exitCode).toEqual(0);
-        expect(output.stderr).toContain('100%');
-        // should not profile
-        expect(output.stderr).not.toContain(
-          'ms after chunk modules optimization'
-        );
-        done();
-      })
-      .catch(done);
-  });
-
-  it('--progress --profile', (done) => {
-    testBin('--progress --profile')
-      .then((output) => {
-        expect(output.exitCode).toEqual(0);
-        // should profile
-        expect(output.stderr).toContain('after chunk modules optimization');
-        done();
-      })
-      .catch(done);
-  });
-
   it('--bonjour', (done) => {
     testBin('--bonjour')
       .then((output) => {

test/e2e/Progress.test.js

@@ -37,7 +37,9 @@ describe('client progress', () => {
         port,
         host: '0.0.0.0',
         hot: true,
-        progress: true,
+        client: {
+          progress: true,
+        },
       };
 
       // we need a delay between file writing and the start

test/fixtures/schema/webpack.config.no-dev-stats.js

@@ -8,7 +8,6 @@ module.exports = {
   devServer: {
     host: '_foo',
     public: '_public',
-    progress: '_progress',
     publicPath: '_publicPath',
     hot: '_hot',
     clientLogging: '_clientLogging',

test/options.test.js

@@ -202,6 +202,11 @@ describe('options', () => {
               port: null,
             },
           },
+          {
+            client: {
+              progress: false,
+            },
+          },
         ],
         failure: [
           'whoops!',
@@ -227,6 +232,11 @@ describe('options', () => {
               logging: 'silent',
             },
           },
+          {
+            client: {
+              progress: '',
+            },
+          },
         ],
       },
       compress: {
@@ -355,10 +365,6 @@ describe('options', () => {
         success: ['', 0, null],
         failure: [false],
       },
-      progress: {
-        success: [false],
-        failure: [''],
-      },
       proxy: {
         success: [
           {