fix: make debug mode actually work (#1305)

Author: kamilogorek

src/raven.js

@@ -2135,7 +2135,11 @@ Raven.prototype = {
   },
 
   _logDebug: function(level) {
-    if (this._originalConsoleMethods[level] && this.debug) {
+    // We allow `Raven.debug` and `Raven.config(DSN, { debug: true })` to not make backward incompatible API change
+    if (
+      this._originalConsoleMethods[level] &&
+      (this.debug || this._globalOptions.debug)
+    ) {
       // In IE<10 console methods do not have their own 'apply' method
       Function.prototype.apply.call(
         this._originalConsoleMethods[level],

test/raven.test.js

@@ -262,6 +262,13 @@ describe('globals', function() {
       assert.isTrue(originalConsoleMethods[level].calledOnce);
     });
 
+    it('should write to console when Raven.config(DSN, { debug: true })', function() {
+      Raven._globalOptions.debug = true;
+      this.sinon.stub(originalConsoleMethods, level);
+      Raven._logDebug(level, message);
+      assert.isTrue(originalConsoleMethods[level].calledOnce);
+    });
+
     it('should handle variadic arguments', function() {
       Raven.debug = true;
       this.sinon.stub(originalConsoleMethods, level);