airbrake-ruby 2.2.4 → 2.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 076ef6e377b4e2fbc3a5cce600393a13390da43c
-  data.tar.gz: 8b31cc49491997971856b5c2d1480c00881f5058
+  metadata.gz: 2c6bcda11cc4df213ee240b89afd624859204780
+  data.tar.gz: 50f609e5e47c2129750b43f7f98bebf13794ef52
 SHA512:
-  metadata.gz: b65105fb50ef5098b1bbfa489387e48f785fcdd8ba25dc7ac306c817385719b8fb9af684b11c447a98740dcc1a2cee17fc65376bc6934150f7f26e268c8ea88d
-  data.tar.gz: 8ef30b2ef9aa0a652de4c9ef1a3a0a9f8b7d60018f9441bae0e7ebe16c79fe0a793290b666132b4adf447bfcdfc4ec9c6610459471f02d9ec0d1cd5b0cabc1ad
+  metadata.gz: daa0b464c5ca8818a006e0a10cec1ff185ffbf2b3602b0b57f1fda4b69032e17d3dbb512054b5a89a15ef0571d698526570c4a8c538c2d7ee1120b1b58158b9c
+  data.tar.gz: 845cd20d7caf62ab809db418cc38178e4dbf257e3436181250ffc14af851087be444315cd0af26830f0b3223e09df8d7a99aeaacc386d13b408766d62f621768

data/lib/airbrake-ruby/notifier.rb

@@ -118,7 +118,7 @@ module Airbrake
 
       notice = build_notice(exception, params)
       @filter_chain.refine(notice)
-      yield notice if block_given?
+      yield notice if block_given? && !notice.ignored?
 
       if notice.ignored?
         return promise.reject("#{notice} was marked as ignored")

data/lib/airbrake-ruby/version.rb

@@ -4,5 +4,5 @@
 module Airbrake
   ##
   # @return [String] the library version
-  AIRBRAKE_RUBY_VERSION = '2.2.4'.freeze
+  AIRBRAKE_RUBY_VERSION = '2.2.5'.freeze
 end

data/spec/notifier_spec.rb

@@ -380,9 +380,29 @@ RSpec.describe Airbrake::Notifier do
     end
 
     describe "block argument" do
-      it "yields a notice" do
-        @airbrake.notify_sync(ex) { |notice| notice[:params][:bingo] = :bango }
-        expect_a_request_with_body(/params":{.*"bingo":"bango".*}/)
+      context "when a notice is not ignored" do
+        it "yields the notice" do
+          @airbrake.notify_sync(ex) { |notice| notice[:params][:bingo] = :bango }
+          expect_a_request_with_body(/params":{.*"bingo":"bango".*}/)
+        end
+      end
+
+      context "when a notice is ignored before entering the block" do
+        it "doesn't call the given block" do
+          @airbrake.add_filter(&:ignore!)
+          @airbrake.notify_sync(ex) { |n| n[:params][:bingo] = :bango }
+          expect(
+            a_request(:post, endpoint).
+            with(body: /params":{.*"bingo":"bango".*}/)
+          ).not_to have_been_made
+        end
+      end
+
+      context "and when a notice is ignored inside the block" do
+        it "doesn't send the notice" do
+          @airbrake.notify_sync(ex, &:ignore!)
+          expect(a_request(:post, endpoint)).not_to have_been_made
+        end
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake-ruby
 version: !ruby/object:Gem::Version
-  version: 2.2.4
+  version: 2.2.5
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-17 00:00:00.000000000 Z
+date: 2017-05-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec