Log to console what would be sent to the server when raven isn't configured. Helps with local debugging of applications using raven.js

Conflicts:
	src/raven.js

Author: benoitg

src/raven.js

@@ -231,10 +231,6 @@ var Raven = {
      * @return {Raven}
      */
     captureException: function(ex, options) {
-        if (!isSetup()) {
-            return Raven;
-        }
-
         // If not an Error is passed through, recall as a message instead
         if (!isError(ex)) return Raven.captureMessage(ex, options);
 
@@ -266,10 +262,6 @@ var Raven = {
      * @return {Raven}
      */
     captureMessage: function(msg, options) {
-        if (!isSetup()) {
-            return Raven;
-        }
-
         // config() automagically converts ignoreErrors from a list to a RegExp so we need to test for an
         // early call; we'll error on the side of logging anything called before configuration since it's
         // probably something you should see:
@@ -698,8 +690,6 @@ function getHttpData() {
 }
 
 function send(data) {
-    if (!isSetup()) return;
-
     var baseData = {
         project: globalProject,
         logger: globalOptions.logger,
@@ -751,7 +741,12 @@ function send(data) {
     // Set lastEventId after we know the error should actually be sent
     lastEventId = data.event_id || (data.event_id = uuid4());
 
-    makeRequest(data);
+    if (isSetup()) {
+      makeRequest(data);
+    }
+    else {
+      console.log("If configured, raven.js would send: ", data);
+    }
 }
 
 

test/raven.test.js

@@ -1078,6 +1078,20 @@ describe('globals', function() {
                 extra: {'session:duration': 100}
             });
         });
+        
+        it('should log to console if not configured', function() {
+            this.sinon.stub(window, 'isSetup').returns(false);
+            this.sinon.stub(console, 'log');
+            send({foo: 'bar'});
+            assert.isTrue(console.log.called);
+        });
+        
+        it('should NOT log to console if configured', function() {
+            this.sinon.stub(window, 'isSetup').returns(true);
+            this.sinon.stub(console, 'log');
+            send({foo: 'bar'});
+            assert.isFalse(console.log.called);
+        });
     });
 
     describe('makeRequest', function() {
@@ -1732,8 +1746,9 @@ describe('Raven (public API)', function() {
         it('should not throw an error if not configured', function() {
             this.sinon.stub(Raven, 'isSetup').returns(false);
             this.sinon.stub(window, 'send')
-            Raven.captureMessage('foo');
-            assert.isFalse(window.send.called);
+            assert.doesNotThrow(function() {
+                Raven.captureMessage('foo');
+            });
         });
 
     });
@@ -1790,8 +1805,9 @@ describe('Raven (public API)', function() {
         it('should not throw an error if not configured', function() {
             this.sinon.stub(Raven, 'isSetup').returns(false);
             this.sinon.stub(window, 'handleStackInfo')
-            Raven.captureException(new Error('err'));
-            assert.isFalse(window.handleStackInfo.called);
+            assert.doesNotThrow(function() {
+                Raven.captureException(new Error('err'));
+            });
         });
     });