mperham/sidekiq — pull request #512 by jackrg (2012-11-09 00:56:57+00:00)
Title: Fix two issues with retry_jobs

1) Specifying 'worker.sidekiq_options(retry: 2)' ends up setting retry to true in the request
2) Specifying 'worker.sidekiq_options(retry: 2)' attempts the job 4 times (one try plus three retries) instead of 3 times (one try plus two retries).

I added tests for these two changes but was unable to get the test suite to run (./test/helper.rb:12:in `require': no such file to load -- minitest/unit (LoadError)) so I don't know that they work, but they are at least syntactically valid :-). If you'd like me to run the tests I'm happy to do so but I need some help with the error (btw, I did 'gem install minitest' and verified that it was installed,but the issue is at least a little confused by the use of RBENV).

Also, since this is my first pull request (everyone has a first one!), if I've messed up the protocol on this feel free to let me know -- I've got a thick skin.