onyx-resque-retry 0.1.4 → 0.1.5

data/lib/resque/failure/multiple_with_retry_suppression.rb

@@ -70,7 +70,7 @@ module Resque
       end
 
       def retry_key
-        klass.redis_retry_key(payload['args'])
+        klass.redis_retry_key(*payload['args'])
       end
 
       def failure_key

data/lib/resque/plugins/retry.rb

@@ -187,7 +187,7 @@ module Resque
           Resque.enqueue_in(retry_delay, retry_in_queue, *args_for_retry(*args))
         end
 
-        clean_retry_key(args) if @retry_job_class
+        clean_retry_key(*args) if @retry_job_class
       end
 
       def clean_retry_key(*args)

data/test/multiple_failure_test.rb

@@ -26,6 +26,13 @@ class MultipleFailureTest < Test::Unit::TestCase
     assert Resque.redis.exists(key)
   end
 
+  def test_retry_key_splatting_args
+    RetryDefaultsJob.expects(:redis_retry_key).with({"a" => 1, "b" => 2}).times(3)
+    Resque.enqueue(RetryDefaultsJob, {"a" => 1, "b" => 2})
+
+    perform_next_job(@worker)
+  end
+
   def test_last_failure_removed_from_redis_after_error_limit
     Resque.enqueue(LimitThreeJob)
     3.times do

data/test/retry_test.rb

@@ -126,6 +126,14 @@ class RetryTest < Test::Unit::TestCase
     assert_equal nil, Resque.redis.get(JobWithRetryQueue.redis_retry_key('arg1'))
   end
 
+  def test_clean_retry_key_should_splat_args
+    JobWithRetryQueue.expects(:clean_retry_key).once.with({"a" => 1, "b" => 2})
+
+    Resque.enqueue(JobWithRetryQueue, {"a" => 1, "b" => 2})
+
+    perform_next_job(@worker)
+  end
+
   def test_retry_delayed_failed_jobs_in_separate_queue
     Resque.enqueue(DelayedJobWithRetryQueue, 'arg1')
     Resque.expects(:enqueue_in).with(1, JobRetryQueue, 'arg1')

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: onyx-resque-retry
 version: !ruby/object:Gem::Version 
-  hash: 19
-  prerelease: false
+  hash: 17
+  prerelease: 
   segments: 
   - 0
   - 1
-  - 4
-  version: 0.1.4
+  - 5
+  version: 0.1.5
 platform: ruby
 authors: 
 - Luke Antins
@@ -16,7 +16,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-10-18 00:00:00 -04:00
+date: 2011-10-31 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -136,19 +136,19 @@ files:
 - Rakefile
 - README.md
 - HISTORY.md
-- test/redis-test.conf
+- test/retry_criteria_test.rb
+- test/retry_test.rb
+- test/test_helper.rb
+- test/exponential_backoff_test.rb
 - test/multiple_failure_test.rb
 - test/retry_inheriting_checks_test.rb
+- test/redis-test.conf
 - test/resque_test.rb
 - test/test_jobs.rb
-- test/retry_criteria_test.rb
-- test/retry_test.rb
-- test/exponential_backoff_test.rb
-- test/test_helper.rb
+- lib/resque-retry.rb
 - lib/resque/plugins/retry.rb
 - lib/resque/plugins/exponential_backoff.rb
 - lib/resque/failure/multiple_with_retry_suppression.rb
-- lib/resque-retry.rb
 - lib/resque-retry/server/views/retry.erb
 - lib/resque-retry/server/views/retry_timestamp.erb
 - lib/resque-retry/server.rb
@@ -182,7 +182,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.3.7
+rubygems_version: 1.5.2
 signing_key: 
 specification_version: 3
 summary: A resque plugin; provides retry, delay and exponential backoff support for resque jobs.