Merge pull request resque#681 from jeremy/yagni-mocha

Consolidate Thoughtbot-backed Failure tests and eliminate Mocha dependency

Author: steveklabnik

Gemfile

@@ -14,11 +14,8 @@ end
 
 group :test do
   gem "rack-test", "~> 0.5"
-  gem "mocha", "~> 0.9.7"
   gem "yajl-ruby", "~>0.8.2", :platforms => :mri
   gem "json", "~>1.5.3", :platforms => [:jruby, :rbx]
-  gem "hoptoad_notifier"
-  gem "airbrake"
   gem "i18n"
   gem "minitest"
 end

test/airbrake_test.rb

@@ -5,7 +5,6 @@
 $dir = File.dirname(File.expand_path(__FILE__))
 $LOAD_PATH.unshift $dir + '/../lib'
 $TESTING = true
-require 'mocha'
 require 'minitest/unit'
 require 'minitest/spec'
 require 'test/unit'

test/hoptoad_test.rb

@@ -0,0 +1,27 @@
+require 'test_helper'
+require 'minitest/mock'
+
+require 'resque/failure/thoughtbot'
+
+class ThoughtbotFailure < Resque::Failure::Base
+  include Resque::Failure::Thoughtbot
+end
+
+describe "Hoptoad and Airbrake failure class" do
+  it "should be notified of an error" do
+    exception = StandardError.new("BOOM")
+    worker = Resque::Worker.new(:test)
+    queue = "test"
+    payload = {'class' => Object, 'args' => 66}
+
+    mock = Minitest::Mock.new
+    mock.expect :notify_or_ignore, nil, [ exception,
+      { :parameters => { :payload_class => 'Object', :payload_args => '66' }} ]
+    ThoughtbotFailure.klass = mock
+
+    backend = ThoughtbotFailure.new(exception, worker, queue, payload)
+    backend.save
+
+    mock.verify
+  end
+end