acidic_job 0.5.2 → 0.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7b6e76d892b0953452781a38155801969acf50d81897fb78645b561280620ddc
-  data.tar.gz: 6640c17792135307d21d20f05b49136512368ff84ed748fb2b1ad98d552fc8fb
+  metadata.gz: e4041a344bd53943f281696cafdd0b0a7fa05969b33ac849475f39efd9333018
+  data.tar.gz: 73a7b1175df2b8e8b7ce6e89cbc13bce208e575b662a31ea03f11ed66fbd0961
 SHA512:
-  metadata.gz: c489d047845c6fcd27482dfe1a3c0cae89a326cd3c81d4816ca9758c7db9e0edd09d954e719eedcc0d8b2a689137e21486482571bd05b7518af795152fc4bc61
-  data.tar.gz: ddc07f0514be3ad3f8fdec4ffd4bf350a039a4ad95f6b28984f1cb73f29d51ed3511df94eb628f8f336b81d4edae3d9b1c9af3a8b55ab5ac08efc015775c9249
+  metadata.gz: bb042126f12c372d0d3767d785d095aa68b48bb71faa10866fa7de6216b47ff7e885ef55611b10cd1d65f9d53aff515a7723fc429ee3095dc3b6ac749e0f761d
+  data.tar.gz: 8cbf5b168781b8ea1125382b998492e6f6d38cf1b46dc8c766b432e93f19e786547dc5dfc8f5437b5623d33970d67d090857abdcd58a5eb5e80804fc682efd36

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    acidic_job (0.5.2)
+    acidic_job (0.5.3)
       activerecord (>= 4.0.0)
       activesupport
 

data/lib/acidic_job/errors.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module AcidicJob
   class Error < StandardError; end
 

data/lib/acidic_job/key.rb

@@ -10,6 +10,7 @@ module AcidicJob
 
     serialize :error_object
     serialize :job_args
+    store :attr_accessors
 
     validates :idempotency_key, presence: true, uniqueness: { scope: %i[job_name job_args] }
     validates :job_name, presence: true

data/lib/acidic_job/perform_transactionally_extension.rb

@@ -7,25 +7,27 @@ module AcidicJob
     extend ActiveSupport::Concern
 
     class_methods do
+      # rubocop:disable Metrics/MethodLength
       def perform_transactionally(*args)
         attributes = if self < ActiveJob::Base
-          {
-            adapter: "activejob",
-            job_name: self.name,
-            job_args: job_or_instantiate(*args).serialize
-          }
-        elsif self.include? Sidekiq::Worker
-          {
-            adapter: "sidekiq",
-            job_name: self.name,
-            job_args: args
-          }
-        else
-          raise UnknownJobAdapter
-        end
+                       {
+                         adapter: "activejob",
+                         job_name: name,
+                         job_args: job_or_instantiate(*args).serialize
+                       }
+                     elsif include? Sidekiq::Worker
+                       {
+                         adapter: "sidekiq",
+                         job_name: name,
+                         job_args: args
+                       }
+                     else
+                       raise UnknownJobAdapter
+                     end
 
         AcidicJob::Staged.create!(attributes)
       end
+      # rubocop:enable Metrics/MethodLength
     end
   end
 end

data/lib/acidic_job/perform_wrapper.rb

@@ -6,14 +6,14 @@ module AcidicJob
       # store arguments passed into `perform` so that we can later persist
       # them to `AcidicJob::Key#job_args` for both ActiveJob and Sidekiq::Worker
       @arguments_for_perform = if args.any? && kwargs.any?
-        args + [kwargs]
-      elsif args.any? && kwargs.none?
-        args
-      elsif args.none? && kwargs.any?
-        [kwargs]
-      else
-        []
-      end
+                                 args + [kwargs]
+                               elsif args.any? && kwargs.none?
+                                 args
+                               elsif args.none? && kwargs.any?
+                                 [kwargs]
+                               else
+                                 []
+                               end
 
       super
     end

data/lib/acidic_job/staged.rb

@@ -15,10 +15,11 @@ module AcidicJob
     after_create_commit :enqueue_job
 
     def enqueue_job
-      if adapter == "activejob"
+      case adapter
+      when "activejob"
         job = ActiveJob::Base.deserialize(job_args)
         job.enqueue
-      elsif adapter == "sidekiq"
+      when "sidekiq"
         Sidekiq::Client.push("class" => job_name, "args" => job_args)
       else
         raise UnknownJobAdapter.new(adapter: adapter)

data/lib/acidic_job/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module AcidicJob
-  VERSION = "0.5.2"
+  VERSION = "0.5.3"
 end

data/lib/acidic_job.rb

@@ -33,6 +33,7 @@ module AcidicJob
   class_methods do
     def inherited(subclass)
       AcidicJob.wire_everything_up(subclass)
+      super
     end
   end
 
@@ -44,10 +45,6 @@ module AcidicJob
 
   # takes a block
   def idempotently(with:)
-    # set accessors for each argument passed in to ensure they are available
-    # to the step methods the job will have written
-    define_accessors_for_passed_arguments(with)
-
     # execute the block to gather the info on what phases are defined for this job
     defined_steps = yield
     # [:create_ride_and_audit_record, :create_stripe_charge, :send_receipt]
@@ -68,6 +65,10 @@ module AcidicJob
     # if the key record is already marked as finished, immediately return its result
     return @key.succeeded? if @key.finished?
 
+    # set accessors for each argument passed in to ensure they are available
+    # to the step methods the job will have written
+    define_accessors_for_passed_arguments(with, @key)
+
     # otherwise, we will enter a loop to process each required step of the job
     phases.size.times do
       # our `phases` hash uses Symbols for keys
@@ -115,16 +116,16 @@ module AcidicJob
       rescued_error = e
       raise e
     ensure
-      return unless rescued_error
-
-      # If we're leaving under an error condition, try to unlock the idempotency
-      # key right away so that another request can try again.3
-      begin
-        key.update_columns(locked_at: nil, error_object: rescued_error)
-      rescue StandardError => e
-        # We're already inside an error condition, so swallow any additional
-        # errors from here and just send them to logs.
-        puts "Failed to unlock key #{key.id} because of #{e}."
+      if rescued_error
+        # If we're leaving under an error condition, try to unlock the idempotency
+        # key right away so that another request can try again.3
+        begin
+          key.update_columns(locked_at: nil, error_object: rescued_error)
+        rescue StandardError => e
+          # We're already inside an error condition, so swallow any additional
+          # errors from here and just send them to logs.
+          puts "Failed to unlock key #{key.id} because of #{e}."
+        end
       end
     end
   end
@@ -143,15 +144,11 @@ module AcidicJob
       if @key
         # Programs enqueuing multiple jobs with different parameters but the
         # same idempotency key is a bug.
-        if @key.job_args != @arguments_for_perform
-          raise MismatchedIdempotencyKeyAndJobArguments
-        end
+        raise MismatchedIdempotencyKeyAndJobArguments if @key.job_args != @arguments_for_perform
 
         # Only acquire a lock if the key is unlocked or its lock has expired
         # because the original job was long enough ago.
-        if @key.locked_at && @key.locked_at > Time.current - IDEMPOTENCY_KEY_LOCK_TIMEOUT
-          raise LockedIdempotencyKey
-        end
+        raise LockedIdempotencyKey if @key.locked_at && @key.locked_at > Time.current - IDEMPOTENCY_KEY_LOCK_TIMEOUT
 
         # Lock the key and update latest run unless the job is already finished.
         @key.update!(last_run_at: Time.current, locked_at: Time.current) unless @key.finished?
@@ -168,12 +165,24 @@ module AcidicJob
     end
   end
 
-  def define_accessors_for_passed_arguments(passed_arguments)
+  def define_accessors_for_passed_arguments(passed_arguments, key)
+    # first, ensure that `Key#attr_accessors` is populated with initial values
+    key.update_column(:attr_accessors, passed_arguments)
+
     passed_arguments.each do |accessor, value|
       # the reader method may already be defined
       self.class.attr_reader accessor unless respond_to?(accessor)
       # but we should always update the value to match the current value
       instance_variable_set("@#{accessor}", value)
+      # and we overwrite the setter to ensure any updates to an accessor update the `Key` stored value
+      # Note: we must define the singleton method on the instance to avoid overwriting setters on other
+      # instances of the same class
+      define_singleton_method("#{accessor}=") do |current_value|
+        instance_variable_set("@#{accessor}", current_value)
+        key.attr_accessors[accessor] = current_value
+        key.save!(validate: false)
+        current_value
+      end
     end
 
     true
@@ -201,7 +210,7 @@ module AcidicJob
     return job_id if defined?(job_id) && !job_id.nil?
     return jid if defined?(jid) && !jid.nil?
 
-    require 'securerandom'
+    require "securerandom"
     SecureRandom.hex
   end
 end

data/lib/generators/templates/create_acidic_job_keys_migration.rb.erb

@@ -8,6 +8,7 @@ class CreateAcidicJobKeys < <%= migration_class %>
       t.datetime :locked_at, null: true
       t.string :recovery_point, null: false
       t.text :error_object
+      t.text :attr_accessors
       t.timestamps
 
       t.index %i[idempotency_key job_name job_args],

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: acidic_job
 version: !ruby/object:Gem::Version
-  version: 0.5.2
+  version: 0.5.3
 platform: ruby
 authors:
 - fractaledmind
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-09-29 00:00:00.000000000 Z
+date: 2021-10-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord