kotovalexarian-likes-github/mperham--sidekiq
1
0
Fork 0
mirror of https://github.com/mperham/sidekiq.git synced 2022-11-09 13:52:34 -05:00
Code Releases Activity

The test for throwing away after too many retries didn't actually test anything!

Browse source
This commit is contained in:
James Le Cuirot 2012-10-16 14:55:04 +01:00
parent d3499a3609
commit 1de18da247
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
test/test_retry.rb
View file

@ -90,14 +90,16 @@ class TestRetry < MiniTest::Unit::TestCase
it 'throws away old messages after too many retries' do
now = Time.now.utc
msg = {"class"=>"Bob", "args"=>[1, 2, "foo"], "queue"=>"default", "error_message"=>"kerblammo!", "error_class"=>"RuntimeError", "failed_at"=>now, "retry_count"=>25}
msg = {"class"=>"Bob", "args"=>[1, 2, "foo"], "queue"=>"default", "error_message"=>"kerblammo!", "error_class"=>"RuntimeError", "failed_at"=>now, "retry"=>true, "retry_count"=>25}
@redis.expect :zadd, 1, [ 'retry', String, String ]
handler = Sidekiq::Middleware::Server::RetryJobs.new
assert_raises RuntimeError do
handler.call('', msg, 'default') do
raise "kerblammo!"
end
end
@redis.verify
# MiniTest can't assert that a method call did NOT happen!?
assert_raises(MockExpectationError) { @redis.verify }
end
end