resolving review requests from #186

Author: shellscape

lib/Shared.js

@@ -54,13 +54,13 @@ module.exports = function Shared(context) {
 					}
 				}
 				if(!options.noInfo && !options.quiet) {
+					var msg = "Compiled successfully.";
 					if(stats.hasErrors()) {
-						options.error("webpack: Failed to compile.");
-					} else if(stats.hasWarnings()) {
-						options.warn("webpack: Compiled with warnings.");
-					} else {
-						options.log("webpack: Compiled successfully.");
-					}
+						msg = "Failed to compile.";
+					  } else if(stats.hasWarnings()) {
+						msg = "Compiled with warnings.";
+					  }
+					options.log("webpack: " + msg);
 				}
 			} else {
 				options.log("webpack: Compiling...");

test/Reporter.test.js

@@ -71,10 +71,10 @@ describe("Reporter", function() {
 
 			plugins.done(errorStats);
 			setTimeout(function() {
-				should.strictEqual(console.log.callCount, 0);
+				should.strictEqual(console.log.callCount, 1);
 				should.strictEqual(console.warn.callCount, 0);
-				should.strictEqual(console.error.callCount, 2);
-				should.strictEqual(console.error.calledWith("webpack: Failed to compile."), true);
+				should.strictEqual(console.error.callCount, 1);
+				should.strictEqual(console.log.calledWith("webpack: Failed to compile."), true);
 				done();
 			});
 		});
@@ -84,10 +84,10 @@ describe("Reporter", function() {
 
 			plugins.done(warningStats);
 			setTimeout(function() {
-				should.strictEqual(console.log.callCount, 0);
-				should.strictEqual(console.warn.callCount, 2);
+				should.strictEqual(console.log.callCount, 1);
+				should.strictEqual(console.warn.callCount, 1);
 				should.strictEqual(console.error.callCount, 0);
-				should.strictEqual(console.warn.calledWith("webpack: Compiled with warnings."), true);
+				should.strictEqual(console.log.calledWith("webpack: Compiled with warnings."), true);
 				done();
 			});
 		});