improve case sensitive warning

sokra · sokra · commit 8ee43c88b8ca · 2016-06-04T19:51:22.000+02:00
diff --git a/lib/CaseSensitiveModulesWarning.js b/lib/CaseSensitiveModulesWarning.js
@@ -7,7 +7,13 @@ function CaseSensitiveModulesWarning(modules) {
 	Error.captureStackTrace(this, CaseSensitiveModulesWarning);
 	this.name = "CaseSensitiveModulesWarning";
 	var modulesList = modules.map(function(m) {
-		return "* " + m.identifier();
+		var message = "* " + m.identifier();
+		var validReasons = m.reasons.filter(function(r) { return r.module; });
+		if(validReasons.length > 0) {
+			message += "\n    Used by " + validReasons.length + " module(s), i. e.";
+			message += "\n    " + validReasons[0].module.identifier();
+		}
+		return message;
 	}).join("\n");
 	this.message = "There are multiple modules with names that only differ in casing.\n" +
 		"This can lead to unexpected behavior when compiling on a filesystem with other case-semantic.\n" +
diff --git a/test/Errors.test.js b/test/Errors.test.js
@@ -81,7 +81,11 @@ describe("Errors", function() {
 				var lines = warnings[0].split("\n");
 				lines[0].should.match(/file\.js/);
 				lines[4].should.match(/file\.js/);
-				lines[5].should.match(/FILE\.js/);
+				lines[5].should.match(/Used by/);
+				lines[6].should.match(/case-sensitive/);
+				lines[7].should.match(/FILE\.js/);
+				lines[8].should.match(/Used by/);
+				lines[9].should.match(/case-sensitive/);
 			} else {
 				errors.length.should.be.eql(1);
 				warnings.length.should.be.eql(0);
