Use Airbrake#notify_or_ignore

kiennt · Sep 6, 2012 · 22381aa · 22381aa
1 parent 3c69685
commit 22381aa
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/Changes.md b/Changes.md
@@ -5,6 +5,7 @@
 - Upgrade Twitter Bootstrap to 2.1.0
 - Rescue more Exceptions
 - Change Job ID to be Hex, rather than Base64, for HTTP safety
+- Use `Airbrake#notify_or_ignore`
 
 2.2.1
 -----------

diff --git a/lib/sidekiq/exception_handler.rb b/lib/sidekiq/exception_handler.rb
@@ -13,7 +13,7 @@ def handle_exception(ex, msg)
     private
 
     def send_to_airbrake(msg, ex)
-      ::Airbrake.notify(ex, :parameters => msg)
+      ::Airbrake.notify_or_ignore(ex, :parameters => msg)
     end
 
     def send_to_exceptional(msg, ex)

diff --git a/test/test_exception_handler.rb b/test/test_exception_handler.rb
@@ -49,7 +49,7 @@ class TestExceptionHandler < MiniTest::Unit::TestCase
     end
 
     it "notifies Airbrake" do
-      ::Airbrake.expect(:notify,nil,[TEST_EXCEPTION,:parameters => { :a => 1 }])
+      ::Airbrake.expect(:notify_or_ignore,nil,[TEST_EXCEPTION,:parameters => { :a => 1 }])
       Component.new.invoke_exception(:a => 1)
       ::Airbrake.verify
     end