refactor: CLI args and more tests (#1679)

Author: evilebottnawi

jest.config.json

@@ -1,6 +1,7 @@
 {
   "testURL": "http://localhost/",
   "collectCoverage": true,
+  "coveragePathIgnorePatterns": ["test"],
   "moduleFileExtensions": ["js", "json"],
   "testMatch": ["**/test/**/*.test.js"],
   "setupFilesAfterEnv": ["<rootDir>/setupTest.js"]

lib/utils/createConfig.js

@@ -14,12 +14,12 @@ function createConfig(config, argv, { port }) {
     options.bonjour = true;
   }
 
-  if (argv.host !== 'localhost' || !options.host) {
+  if (argv.host && (argv.host !== 'localhost' || !options.host)) {
     options.host = argv.host;
   }
 
-  if (argv['allowed-hosts']) {
-    options.allowedHosts = argv['allowed-hosts'].split(',');
+  if (argv.allowedHosts) {
+    options.allowedHosts = argv.allowedHosts.split(',');
   }
 
   if (argv.public) {
@@ -47,11 +47,11 @@ function createConfig(config, argv, { port }) {
     }
   }
 
-  if (!options.filename) {
+  if (!options.filename && firstWpOpt.output && firstWpOpt.output.filename) {
     options.filename = firstWpOpt.output && firstWpOpt.output.filename;
   }
 
-  if (!options.watchOptions) {
+  if (!options.watchOptions && firstWpOpt.watchOptions) {
     options.watchOptions = firstWpOpt.watchOptions;
   }
 
@@ -64,38 +64,42 @@ function createConfig(config, argv, { port }) {
     process.stdin.resume();
   }
 
+  // TODO https://github.com/webpack/webpack-dev-server/issues/616 (v4)
+  // We should prefer CLI arg under config, now we always prefer `hot` from `devServer`
   if (!options.hot) {
     options.hot = argv.hot;
   }
 
+  // TODO https://github.com/webpack/webpack-dev-server/issues/616 (v4)
+  // We should prefer CLI arg under config, now we always prefer `hotOnly` from `devServer`
   if (!options.hotOnly) {
-    options.hotOnly = argv['hot-only'];
+    options.hotOnly = argv.hotOnly;
   }
 
-  if (!options.clientLogLevel) {
-    options.clientLogLevel = argv['client-log-level'];
+  // TODO https://github.com/webpack/webpack-dev-server/issues/616 (v4)
+  // We should prefer CLI arg under config, now we always prefer `clientLogLevel` from `devServer`
+  if (!options.clientLogLevel && argv.clientLogLevel) {
+    options.clientLogLevel = argv.clientLogLevel;
   }
 
-  // eslint-disable-next-line
-  if (options.contentBase === undefined) {
-    if (argv['content-base']) {
-      options.contentBase = argv['content-base'];
+  if (argv.contentBase) {
+    options.contentBase = argv.contentBase;
 
-      if (Array.isArray(options.contentBase)) {
-        options.contentBase = options.contentBase.map((p) => path.resolve(p));
-      } else if (/^[0-9]$/.test(options.contentBase)) {
-        options.contentBase = +options.contentBase;
-      } else if (!/^(https?:)?\/\//.test(options.contentBase)) {
-        options.contentBase = path.resolve(options.contentBase);
-      }
-      // It is possible to disable the contentBase by using
-      // `--no-content-base`, which results in arg["content-base"] = false
-    } else if (argv['content-base'] === false) {
-      options.contentBase = false;
+    if (Array.isArray(options.contentBase)) {
+      options.contentBase = options.contentBase.map((p) => path.resolve(p));
+    } else if (/^[0-9]$/.test(options.contentBase)) {
+      options.contentBase = +options.contentBase;
+    } else if (!/^(https?:)?\/\//.test(options.contentBase)) {
+      options.contentBase = path.resolve(options.contentBase);
     }
+    // It is possible to disable the contentBase by using
+    // `--no-content-base`, which results in arg["content-base"] = false
+  } else if (argv.contentBase === false) {
+    // TODO doesn't work need fix, `false` in this case is string
+    options.contentBase = false;
   }
 
-  if (argv['watch-content-base']) {
+  if (argv.watchContentBase) {
     options.watchContentBase = true;
   }
 
@@ -108,7 +112,8 @@ function createConfig(config, argv, { port }) {
 
   if (
     typeof options.stats === 'object' &&
-    typeof options.stats.colors === 'undefined'
+    typeof options.stats.colors === 'undefined' &&
+    argv.color
   ) {
     options.stats = Object.assign({}, options.stats, { colors: argv.color });
   }
@@ -117,10 +122,12 @@ function createConfig(config, argv, { port }) {
     options.lazy = true;
   }
 
+  // TODO remove in `v4`
   if (!argv.info) {
     options.noInfo = true;
   }
 
+  // TODO remove in `v4`
   if (argv.quiet) {
     options.quiet = true;
   }
@@ -129,29 +136,29 @@ function createConfig(config, argv, { port }) {
     options.https = true;
   }
 
-  if (argv['pfx-passphrase']) {
-    options.pfxPassphrase = argv['pfx-passphrase'];
+  if (argv.pfxPassphrase) {
+    options.pfxPassphrase = argv.pfxPassphrase;
   }
 
   if (argv.inline === false) {
     options.inline = false;
   }
 
-  if (argv['history-api-fallback']) {
+  if (argv.historyApiFallback) {
     options.historyApiFallback = true;
   }
 
   if (argv.compress) {
     options.compress = true;
   }
 
-  if (argv['disable-host-check']) {
+  if (argv.disableHostCheck) {
     options.disableHostCheck = true;
   }
 
-  if (argv['open-page']) {
+  if (argv.openPage) {
     options.open = true;
-    options.openPage = argv['open-page'];
+    options.openPage = argv.openPage;
   }
 
   if (typeof argv.open !== 'undefined') {