fix(cli): show help flag when defaults fail

bin/cli.js

@@ -4,6 +4,7 @@
 	MIT License http://www.opensource.org/licenses/mit-license.php
 	Author Tobias Koppers @sokra
 */
+const chalk = require("chalk");
 
 (function() {
 	// wrap in IIFE to be able to use return
@@ -496,6 +497,21 @@ For more information, see https://webpack.js.org/api/cli/.`);
 					);
 				} else if (stats.hash !== lastHash) {
 					lastHash = stats.hash;
+					if (stats.compilation && stats.compilation.errors.length !== 0) {
+						const errors = stats.compilation.errors;
+						if (errors[0].name === "EntryModuleNotFoundError") {
+							stdout.write("\n" + chalk.red(errors[0].error.toString()) + "\n");
+							stdout.write(
+								"\n" + chalk.blue("ERROR ") + "Insufficient number of arguments provided " +
+								"\n" + chalk.blue("INFO ") + "Alternatively, run `webpack(-cli) --help` for usage info." +
+								"\n\n"
+							);
+							process.exit(0);
+						}
+					} else {
+						const statsString = stats.toString(outputOptions);
+						if (statsString) stdout.write(statsString + "\n");
+					}
 					const statsString = stats.toString(outputOptions);
 					const delimiter = outputOptions.buildDelimiter
 						? `${outputOptions.buildDelimiter}\n`

test/binCases/no-options/development/stdin.js

@@ -1,13 +1,9 @@
 "use strict";
 
 module.exports = function testAssertions(code, stdout, stderr) {
-	expect(code).toBe(2);
+	expect(code).toBe(0);
 	expect(stdout).toEqual(expect.anything());
-	expect(stdout[0]).toContain("Hash: ");
-	expect(stdout[1]).toContain("Version: ");
-	expect(stdout[2]).toContain("Time: ");
-	expect(stdout[4]).toContain("");
-	expect(stdout[5]).toContain("ERROR in Entry module not found");
-	expect(stdout[6]).toContain("");
+	expect(stdout[3]).toContain("Insufficient number of arguments provided");
+	expect(stdout[4]).toContain("Alternatively, run `webpack(-cli) --help` for usage info");
 	expect(stderr).toHaveLength(0);
 };

test/binCases/no-options/none/stdin.js

@@ -1,16 +1,9 @@
 "use strict";
 
 module.exports = function testAssertions(code, stdout, stderr) {
-	expect(code).toBe(2);
+	expect(code).toBe(0);
 	expect(stdout).toEqual(expect.anything());
-	expect(stdout[0]).toContain("Hash: ");
-	expect(stdout[1]).toContain("Version: ");
-	expect(stdout[2]).toContain("Time: ");
-	expect(stdout[4]).toContain("");
-	expect(stdout[5]).toContain("WARNING");
-	expect(stdout[6]).toContain("The \'mode\' option has not been set");
-	expect(stdout[7]).toContain("You can also set it to \'none\'");
-	expect(stdout[8]).toContain("");
-	expect(stdout[9]).toContain("ERROR in Entry module not found");
+	expect(stdout[3]).toContain("Insufficient number of arguments provided");
+	expect(stdout[4]).toContain("Alternatively, run `webpack(-cli) --help` for usage info");
 	expect(stderr).toHaveLength(0);
 };

test/binCases/no-options/production/stdin.js

@@ -1,13 +1,9 @@
 "use strict";
 
 module.exports = function testAssertions(code, stdout, stderr) {
-	expect(code).toBe(2);
+	expect(code).toBe(0);
 	expect(stdout).toEqual(expect.anything());
-	expect(stdout[0]).toContain("Hash: ");
-	expect(stdout[1]).toContain("Version: ");
-	expect(stdout[2]).toContain("Time: ");
-	expect(stdout[4]).toContain("");
-	expect(stdout[5]).toContain("ERROR in Entry module not found");
-	expect(stdout[6]).toContain("");
+	expect(stdout[3]).toContain("Insufficient number of arguments provided");
+	expect(stdout[4]).toContain("Alternatively, run `webpack(-cli) --help` for usage info");
 	expect(stderr).toHaveLength(0);
 };

test/binCases/no-options/stdin.js

@@ -0,0 +1,5 @@
+"use strict";
+
+module.exports = function testAssertions(code, stdout, stderr) {
+	expect(stdout[3]).toContain("Insufficient number of arguments provided");
+};

test/binCases/watch/single-config-watch-opt/stdin.js

@@ -4,7 +4,6 @@ module.exports = function testAssertions(stdout, stderr, done) {
 	expect(stdout).toEqual(expect.anything());
 	expect(stdout[0]).toContain("");
 	expect(stdout[1]).toContain("Webpack is watching the files…");
-
 	expect(stderr).toHaveLength(0);
 	done();
 };