Merge pull request #10 from james-west/master

Explicitly check whether console is defined to prevent IE7 from choking ...

src/logger.js

@@ -150,7 +150,7 @@
  // Configure and example a Default implementation which writes to the `window.console` (if present).
  Logger.useDefaults = function(defaultLevel) {
  // Check for the presence of a logger.
- if (!console) {
+ if (typeof console === "undefined") {
  return;
  }
 

test-src/index.html

@@ -5,6 +5,9 @@
  <!-- Qunit mcgubbins -->
  <script type="text/javascript" src="vendor/qunit-1.14.0.js"></script>
 
+ <!-- Sinon -->
+ <script type="text/javascript" src="vendor/sinon-1.10.3.js"></script>
+
  <!-- src -->
  <script type="text/javascript" src="../src/logger.js"></script>
 

test-src/loggertests.js

@@ -132,4 +132,27 @@ test("Logger.setLevel - Modify log filter level of all named loggers", function
  named.debug("debug message");
 
  ok(this.calls.length === 1, "Logger.setLevel() sets log filter level for all named loggers");
-});
+});
+
+test("Logger.useDefaults logs to console", function () {
+ var logger = this.logger;
+
+ var sandbox = sinon.sandbox.create();
+ sandbox.stub(console, "log");
+ sandbox.stub(console, "info");
+ sandbox.stub(console, "warn");
+ sandbox.stub(console, "error");
+
+ logger.useDefaults();
+ logger.debug("debug message");
+ logger.info("info message");
+ logger.warn("warning message");
+ logger.error("error message");
+
+ ok(console.log.calledOnce, "logger.debug calls console.log");
+ ok(console.info.calledOnce, "logger.info calls console.info");
+ ok(console.warn.calledOnce, "logger.warn calls console.warn");
+ ok(console.error.calledOnce, "logger.error calls console.error");
+
+ sandbox.restore();
+});