delayed 0.5.2 → 0.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ac86a8d4576b55da967457fefbe6576b1cb2aa894d1015bd4e9883beb25a14cf
-  data.tar.gz: 1db2668da101214ff8ad304931de54a5b348a54dee991eac6535fd117a14af90
+  metadata.gz: 1a6428c0614b28cf06d2921310bea48cc8e0bcc891a9772b7aed44238dbf1533
+  data.tar.gz: fa61f990945d382d21dd856170ce10dc52ecf44f2360a5be8ac9ec3463f9b5ac
 SHA512:
-  metadata.gz: b07f547dbc8087465a79dbad0e75170d720d7f942b573dab337666f244ace82f2de08b52aee12e20038ec88594a103385de21abf3f439aabe4e2404bdf45d75a
-  data.tar.gz: d2144160713def9d49398a99cb99e3def26cc5beea91d9f0ee832533ba91569b9c9642e00de44ce23eb4e85318c3bbaf34b02f4f478fb3dd85340d0eff09f627
+  metadata.gz: 46152e349a8b725978c3ef8a281789e288301e1d605b67fe4a8562733dac1742d5a9783a329c1569dce69910e73249b1134f8a5a1b19956ccb2b3a3a355dddd9
+  data.tar.gz: f6bd7af40fd73a4d4d9fe5c2af5eb307241d627a43ecf972cc7fb6510ba1a0d2ad0b2e065fdf935bb2bc3f03ffa912f91d2cead31cd01a2ab5aa6e70d446203a

data/lib/delayed/active_job_adapter.rb

@@ -14,7 +14,7 @@ module Delayed
       opts.merge!({ queue: job.queue_name, priority: job.priority }.compact)
         .merge!(job.provider_attributes || {})
 
-      Delayed::Job.enqueue(JobWrapper.new(job.serialize), opts).tap do |dj|
+      Delayed::Job.enqueue(JobWrapper.new(job), opts).tap do |dj|
         job.provider_job_id = dj.id
       end
     end

data/lib/delayed/job_wrapper.rb

@@ -4,8 +4,16 @@ module Delayed
 
     delegate_missing_to :job
 
-    def initialize(job_data)
-      @job_data = job_data
+    def initialize(job_or_data)
+      # During enqueue the job instance is passed in directly, saves us deserializing
+      # it to find out how to queue the job.
+      # During load from the db, we get a data hash passed in so deserialize lazily.
+      if job_or_data.is_a?(ActiveJob::Base)
+        @job = job_or_data
+        @job_data = job_or_data.serialize
+      else
+        @job_data = job_or_data
+      end
     end
 
     def display_name

data/lib/delayed/monitor.rb

@@ -34,7 +34,7 @@ module Delayed
     attr_reader :jobs
 
     def emit_metric!(metric)
-      send("#{metric}_grouped").reverse_merge(default_results).each do |(priority, queue), value|
+      send(:"#{metric}_grouped").reverse_merge(default_results).each do |(priority, queue), value|
         ActiveSupport::Notifications.instrument(
           "delayed.job.#{metric}",
           default_tags.merge(priority: Priority.new(priority).to_s, queue: queue, value: value),

data/spec/delayed/active_job_adapter_spec.rb

@@ -23,6 +23,16 @@ RSpec.describe Delayed::ActiveJobAdapter do
     ActiveJob::Base.queue_adapter = adapter_was
   end
 
+  it "does not invoke #deserialize during enqueue" do # rubocop:disable RSpec/NoExpectationExample
+    JobClass.include(Module.new do
+      def deserialize(*)
+        raise "uh oh, deserialize called during enqueue!"
+      end
+    end)
+
+    JobClass.perform_later
+  end
+
   it 'serializes a JobWrapper in the handler with expected fields' do
     Timecop.freeze('2023-01-20T18:52:29Z') do
       JobClass.perform_later

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: delayed
 version: !ruby/object:Gem::Version
-  version: 0.5.2
+  version: 0.5.3
 platform: ruby
 authors:
 - Nathan Griffith
@@ -19,7 +19,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-10-19 00:00:00.000000000 Z
+date: 2024-01-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord