fix(cli): show help flag when defaults fail (#466)

* fixed test cases

* remove unwanted console

* fix(254) - addressed PR comments

* fix(254) - added requested changes at right place

* fix(254) - exit after error

exit webpack if defaults are not found
fix test cases for no options

* fix: require chalk

* refactor - addressed PR comments

* refactor - addressed PR comments

* refactor : remove unwanted keywords

* chore(review): format

bin/cli.js

@@ -496,6 +496,17 @@ 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") {
+							console.error(
+								"\n\u001b[1m\u001b[31mInsufficient number of arguments or no entry found."
+							);
+							console.error(
+								"\u001b[1m\u001b[31mAlternatively, run 'webpack(-cli) --help' for usage info.\u001b[39m\u001b[22m\n"
+							);
+						}
+					}
 					const statsString = stats.toString(outputOptions);
 					const delimiter = outputOptions.buildDelimiter
 						? `${outputOptions.buildDelimiter}\n`

packages/utils/recursive-parser.test.js

@@ -2,6 +2,7 @@
 
 const defineTest = require("./defineTest");
 
+
 defineTest(
 	__dirname,
 	"init",

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");
+};