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

@@ -232,10 +232,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);
 
@@ -267,10 +263,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:
@@ -699,8 +691,6 @@ function getHttpData() {
 }
 
 function send(data) {
- if (!isSetup()) return;
-
  var baseData = {
  project: globalProject,
  logger: globalOptions.logger,
@@ -752,7 +742,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

@@ -1086,6 +1086,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() {
@@ -1772,8 +1786,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');
+ });
  });
 
  });
@@ -1830,8 +1845,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'));
+ });
  });
  });