acidic_job 0.5.0 → 0.5.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 943a9bc87b07b1e4a2ec7d0a83ffa8a23917124386cfc601780f95d36c62d23a
-  data.tar.gz: 2f534e9ebd0d533ae4b7572fb1da1451df9a1a2ed68fc231bc4f4de323de9e84
+  metadata.gz: 64a311e997f51bcd3ddf92571120c63976f32c35c9da22d3acd1e844ca4b152e
+  data.tar.gz: 68482f3ef456a7e677bfdd3bd9da93ccec5cf3029a9b21a78820cf0d1f210249
 SHA512:
-  metadata.gz: 59f4ecc2e773d72bbef2d386a223e9859ee6f14c657e706c593e7a3f4c994c86aae885ef0b2993cbb916a1380daf3231882255685b6ffa360ffc4bf5dee591d8
-  data.tar.gz: 2a8d540fabb8528a9c305b9d7cf7fe12588c14760557f08f4c380dfb6b45fdafec1232767234809101d6aa0e8f44f8ac895b120664dec8e533d180900ad06c6f
+  metadata.gz: c15a9b5155db86ea3fc43323bcb2ed37e82a2fff199077861a1c7698c998fb4ccd08b7ba70f3f9a190f1d2f5e14430caf880ce0a3d468d83f20941c3d9b41f8e
+  data.tar.gz: 8248faf77812deb4a073c4215d6796ac60932e97c0fc3efe36ce8a11eb79b5e06a41f7a7dab13c9759ffc79d296ad6d82a85340a59f388a7d19b966b5a6d631a

data/Gemfile.lock

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

data/bin/console

@@ -3,7 +3,8 @@
 
 require "bundler/setup"
 require "acidic_job"
-require_relative "../test/setup"
+require_relative "../test/support/setup"
+require_relative "../test/support/ride_create_job"
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.

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.0"
+  VERSION = "0.5.4"
 end

data/lib/acidic_job.rb

@@ -15,15 +15,26 @@ require "active_support/concern"
 module AcidicJob
   extend ActiveSupport::Concern
 
-  included do
-    attr_reader :key
-    attr_accessor :arguments_for_perform
+  def self.wire_everything_up(klass)
+    klass.attr_reader :key
+    klass.attr_accessor :arguments_for_perform
 
     # Extend ActiveJob with `perform_transactionally` class method
-    include PerformTransactionallyExtension
+    klass.include PerformTransactionallyExtension
 
     # Ensure our `perform` method always runs first to gather parameters
-    prepend PerformWrapper
+    klass.prepend PerformWrapper
+  end
+
+  included do
+    AcidicJob.wire_everything_up(self)
+  end
+
+  class_methods do
+    def inherited(subclass)
+      AcidicJob.wire_everything_up(subclass)
+      super
+    end
   end
 
   # Number of seconds passed which we consider a held idempotency key lock to be
@@ -34,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]
@@ -58,8 +65,12 @@ 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
-    100.times do
+    phases.size.times do
       # our `phases` hash uses Symbols for keys
       recovery_point = @key.recovery_point.to_sym
 
@@ -105,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
@@ -133,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?
@@ -158,12 +165,27 @@ module AcidicJob
     end
   end
 
-  def define_accessors_for_passed_arguments(passed_arguments)
-    passed_arguments.each do |accessor, value|
+  def define_accessors_for_passed_arguments(passed_arguments, key)
+    # first, get the current state of all accessors for both previously persisted and initialized values
+    current_accessors = passed_arguments.stringify_keys.merge(key.attr_accessors)
+
+    # next, ensure that `Key#attr_accessors` is populated with initial values
+    key.update_column(:attr_accessors, current_accessors)
+
+    current_accessors.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
@@ -191,7 +213,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],

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

@@ -4,6 +4,7 @@ class CreateStagedAcidicJobs < <%= migration_class %>
       t.string :adapter, null: false
       t.string :job_name, null: false
       t.text :job_args, null: true
+      t.timestamps
     end
   end
 end

metadata

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