Improve messages when bundle is compiling and done with compiling (#164)

* Improve messages when bundle is compiling and done with compiling

* Add compiled with warnings msg

Author: SpaceK33z

lib/Shared.js

@@ -44,10 +44,16 @@ module.exports = function Shared(context) {
 					options.log(stats.toString(options.stats));
 				}
 				if(!options.noInfo && !options.quiet) {
-					options.log("webpack: bundle is now VALID.");
+					var msg = "Compiled successfully.";
+					if(stats.hasErrors()) {
+						msg = "Failed to compile.";
+					} else if(stats.hasWarnings()) {
+						msg = "Compiled with warnings.";
+					}
+					options.log("webpack: " + msg);
 				}
 			} else {
-				options.log("webpack: bundle is now INVALID.");
+				options.log("webpack: Compiling...");
 			}
 		},
 		handleRangeHeaders: function handleRangeHeaders(content, req, res) {

test/Reporter.test.js

@@ -15,6 +15,24 @@ var simpleStats = {
 	}
 };
 
+var errorStats = {
+	hasErrors: function() {
+		return true;
+	},
+	hasWarnings: function() {
+		return false;
+	}
+};
+
+var warningStats = {
+	hasErrors: function() {
+		return false;
+	},
+	hasWarnings: function() {
+		return true;
+	}
+};
+
 describe("Reporter", function() {
 	var plugins = {};
 	var compiler = {
@@ -33,13 +51,33 @@ describe("Reporter", function() {
 	});
 
 	describe("valid/invalid messages", function() {
-		it("should show valid message", function(done) {
+		it("should show compiled successfully message", function(done) {
 			middleware(compiler);
 
 			plugins.done(simpleStats);
 			setTimeout(function() {
 				should.strictEqual(console.log.callCount, 2);
-				should.strictEqual(console.log.calledWith("webpack: bundle is now VALID."), true);
+				should.strictEqual(console.log.calledWith("webpack: Compiled successfully."), true);
+				done();
+			});
+		});
+
+		it("should show compiled successfully message", function(done) {
+			middleware(compiler);
+
+			plugins.done(errorStats);
+			setTimeout(function() {
+				should.strictEqual(console.log.calledWith("webpack: Failed to compile."), true);
+				done();
+			});
+		});
+
+		it("should show compiled with warnings message", function(done) {
+			middleware(compiler);
+
+			plugins.done(warningStats);
+			setTimeout(function() {
+				should.strictEqual(console.log.calledWith("webpack: Compiled with warnings."), true);
 				done();
 			});
 		});
@@ -70,7 +108,7 @@ describe("Reporter", function() {
 			plugins.invalid();
 			setTimeout(function() {
 				should.strictEqual(console.log.callCount, 1);
-				should.strictEqual(console.log.calledWith("webpack: bundle is now INVALID."), true);
+				should.strictEqual(console.log.calledWith("webpack: Compiling..."), true);
 				done();
 			});
 		});