resque-retry 1.8.0 → 1.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e59965fb9d40a6d3f680291e5cbdec8edf04946ad50749175e984062a52e9d1c
-  data.tar.gz: c7962de9928657b3b4b2745efbb5e403a512f44a1b0cd5db5701aee87302dffa
+  metadata.gz: 562c945b75ddfc1a5136f6966bae9e9cce9b79167ec63c84c68a64d4ee0f9cad
+  data.tar.gz: 8e8069a54b444dd02b8c14ed8bd327a5bde8d33bde64529f3a0de570bcc7a938
 SHA512:
-  metadata.gz: 41c214710cdfb36737e8ed9cc598f625df4020e29a056f235fd8aa636cec7f2e220f63afbfa29090b944444cee0f938b66d676b23496671b65f7f1051f08b8e4
-  data.tar.gz: c78705b7a1a87c9b92c8eb6c9b72070dfb51f52dcab7b3cdc55e2a0a89940f34a1ef98255f11986a43011c6a56abeeee0d2f96493b41ee921ad90dab178ce76b
+  metadata.gz: ded662e20a75af089bf98d09b37b95ac3c5b81f4cb0e8f0d2241c5207007d738382baff53682284c1ba157fa50d44648a5e54325d0492418b065a81683dd7cbb
+  data.tar.gz: 6290a973a9576501f8654189ddad0da8df77711c0b3e3aafbfd581faf4de2aef15b1f15eaa0f6f3b43fcf368cd8a0e49ad099fa48b426be84c67e02c17828f0f

data/lib/resque-retry/server.rb

@@ -49,12 +49,21 @@ module ResqueRetry
 
       # gets the number of retry attempts for a job.
       def retry_attempts_for_job(job)
-        Resque.redis.get(retry_key_for_job(job))
+        retry_key = retry_key_for_job(job)
+        if retry_key
+          Resque.redis.get(retry_key)
+        else
+          nil
+        end
       end
 
       # gets the failure details hash for a job.
       def retry_failure_details(retry_key)
-        Resque.decode(Resque.redis.get("failure-#{retry_key}"))
+        if retry_key
+          Resque.decode(Resque.redis.get("failure-#{retry_key}"))
+        else
+          nil
+        end
       end
 
       # reads a 'local' template file.

data/lib/resque-retry/version.rb

@@ -1,3 +1,3 @@
 module ResqueRetry
-  VERSION = '1.8.0'
+  VERSION = '1.8.1'
 end

data/test/server_helpers_test.rb

@@ -19,4 +19,17 @@ class ServerHelpersTest < Minitest::Test
     job = Resque.delayed_timestamp_peek(timestamp, 0, 1).first
     assert_equal '0', @helpers.retry_attempts_for_job(job), 'should have 0 retry attempt'
   end
+
+  def test_retry_key_for_job_empty
+    Resque.enqueue(DelayedJobNoRetryKey)
+    perform_next_job(@worker)
+
+    timestamp = Resque.delayed_queue_peek(0, 1).first
+    job = Resque.delayed_timestamp_peek(timestamp, 0, 1).first
+    retry_key = @helpers.retry_key_for_job(job)
+
+    assert_nil retry_key, 'should be nil as the class does not respond to redis_retry_key'
+    assert_nil @helpers.retry_attempts_for_job(job), 'should have nil retry attempts as the key does not exist'
+    assert_nil @helpers.retry_failure_details(retry_key), 'should have nil failure details as the key does not exist'
+  end
 end

data/test/test_jobs.rb

@@ -198,6 +198,10 @@ class LimitThreeJobDelay1Hour < LimitThreeJob
   @retry_delay = 3600
 end
 
+class DelayedJobNoRetryKey
+  @queue = :testing
+end
+
 class FailFiveTimesJob < RetryDefaultsJob
   @queue = :testing
   @retry_limit = 6

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: resque-retry
 version: !ruby/object:Gem::Version
-  version: 1.8.0
+  version: 1.8.1
 platform: ruby
 authors:
 - Luke Antins
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-04-15 00:00:00.000000000 Z
+date: 2023-06-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: resque
@@ -229,7 +229,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.4.14
 signing_key:
 specification_version: 4
 summary: A resque plugin; provides retry, delay and exponential backoff support for