Merge pull request #312 from acdha/capture-message-ignore-errors

captureMessage() uses global ignoreErrors config

Author: mattrobenolt

src/raven.js

@@ -261,6 +261,13 @@ var Raven = {
      * @return {Raven}
      */
     captureMessage: function(msg, options) {
+        // 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:
+        if (!!globalOptions.ignoreErrors.test && globalOptions.ignoreErrors.test(msg)) {
+            return;
+        }
+
         // Fire away!
         send(
             objectMerge({

test/raven.test.js

@@ -1594,6 +1594,18 @@ describe('Raven (public API)', function() {
             Raven.captureMessage('lol');
             assert.equal(Raven.lastEventId(), 'abc123');
         });
+
+        it('should respect `ignoreErrors`', function() {
+            this.sinon.stub(window, 'send');
+
+            globalOptions.ignoreErrors = joinRegExp(['e1', 'e2']);
+            Raven.captureMessage('e1');
+            assert.isFalse(window.send.called);
+            Raven.captureMessage('e2');
+            assert.isFalse(window.send.called);
+            Raven.captureMessage('Non-ignored error');
+            assert.isTrue(window.send.calledOnce);
+        });
     });
 
     describe('.captureException', function() {