acidic_job 0.7.1 → 0.7.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 970e9128c668bd3d5950a517eae9c215ee6c85d4d56a69fc8862afc622046df2
-  data.tar.gz: e04595bedefa12976e37661b34cb04544966af203c92df88a943e11667bab23f
+  metadata.gz: 0ce6a143a80bd88434c13eb9efe757511d5bc5a4eebdb60cd7b00cab5e9ccf91
+  data.tar.gz: 299938e0eac7f16ecf8b2090ccd2f3c8e019be091558c5774b6b3673990eeb11
 SHA512:
-  metadata.gz: d95c1a6446e528300fb0d61ce45c897796ecca97af639a9f76a7375b39bd4332e7011d93663738a8eae08f93ae896b5663522ca372eddbf93d106eec79430db5
-  data.tar.gz: 550c9c53c6bde0b1b572ed3e0e98d470b78af0cdc0b1e2a50294ec7c3432421de64b01389df9c43f2e2fe24430d38de94c67d3be8f58660fc182130453ae73a5
+  metadata.gz: e0ee67654ff2ab8aaa4739372d1692cc98c7cfb9c5a0ce1be1383f02421af3f6833ef7622c3a61a0677a37cdb3f1f38665bd07ccaa7adc204ed0e0e7f07f514f
+  data.tar.gz: 7b82050bb354ce05e4f0ee769b690f30fcd4a29dbb2382d9328b59fe32d9fec8b61b2e79e3bf4a508b5a59e5f6265024cb0c9e8dcfa85e861f1b3e3a2e8d5ea5

data/.github/workflows/main.yml

@@ -13,6 +13,6 @@ jobs:
         ruby-version: 2.7.1
     - name: Run the default task
       run: |
-        gem install bundler -v 2.2.5
+        gem install bundler -v 2.2.31
         bundle install
         bundle exec rake

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    acidic_job (0.7.1)
+    acidic_job (0.7.5)
       activerecord (>= 4.0.0)
       activesupport
 
@@ -62,8 +62,6 @@ GEM
     nokogiri (1.12.3)
       mini_portile2 (~> 2.6.1)
       racc (~> 1.4)
-    nokogiri (1.12.3-x86_64-darwin)
-      racc (~> 1.4)
     parallel (1.20.1)
     parser (3.0.1.1)
       ast (~> 2.4.1)

data/README.md

@@ -57,14 +57,14 @@ It provides a suite of functionality that empowers you to create complex, robust
 
 ### Transactional Steps
 
-The first and foundational feature `acidic_job` provides is the `idempotently` method, which takes a block of transactional step methods (defined via the `step`) method:
+The first and foundational feature `acidic_job` provides is the `with_acidity` method, which takes a block of transactional step methods (defined via the `step`) method:
 
 ```ruby
 class RideCreateJob < ActiveJob::Base
   include AcidicJob
 
   def perform(ride_params)
-    idempotently with: { user: current_user, params: ride_params, ride: nil } do
+    with_acidity given: { user: current_user, params: ride_params, ride: nil } do
       step :create_ride_and_audit_record
       step :create_stripe_charge
       step :send_receipt
@@ -85,20 +85,20 @@ class RideCreateJob < ActiveJob::Base
 end
 ```
 
-`idempotently` takes only the `with:` named parameter and a block where you define the steps of this operation. `step` simply takes the name of a method available in the job. That's all!
+`with_acidity` takes only the `given:` named parameter and a block where you define the steps of this operation. `step` simply takes the name of a method available in the job. That's all!
 
 Now, each execution of this job will find or create an `AcidicJob::Key` record, which we leverage to wrap every step in a database transaction. Moreover, this database record allows `acidic_job` to ensure that if your job fails on step 3, when it retries, it will simply jump right back to trying to execute the method defined for the 3rd step, and won't even execute the first two step methods. This means your step methods only need to be idempotent on failure, not on success, since they will never be run again if they succeed.
 
 ### Persisted Attributes
 
-Any objects passed to the `with` option on the `idempotently` method are not just made available to each of your step methods, they are made available across retries. This means that you can set an attribute in step 1, access it in step 2, have step 2 fail, have the job retry, jump directly back to step 2 on retry, and have that object still accessible. This is done by serializing all objects to a field on the `AcidicJob::Key` and manually providing getters and setters that sync with the database record.
+Any objects passed to the `given` option on the `with_acidity` method are not just made available to each of your step methods, they are made available across retries. This means that you can set an attribute in step 1, access it in step 2, have step 2 fail, have the job retry, jump directly back to step 2 on retry, and have that object still accessible. This is done by serializing all objects to a field on the `AcidicJob::Key` and manually providing getters and setters that sync with the database record.
 
 ```ruby
 class RideCreateJob < ActiveJob::Base
   include AcidicJob
 
   def perform(ride_params)
-    idempotently with: { ride: nil } do
+    with_acidity given: { ride: nil } do
       step :create_ride_and_audit_record
       step :create_stripe_charge
       step :send_receipt
@@ -132,7 +132,7 @@ class RideCreateJob < ActiveJob::Base
   include AcidicJob
 
   def perform(ride_params)
-    idempotently with: { user: current_user, params: ride_params, ride: nil } do
+    with_acidity given: { user: current_user, params: ride_params, ride: nil } do
       step :create_ride_and_audit_record
       step :create_stripe_charge
       step :send_receipt

data/lib/acidic_job/deliver_transactionally_extension.rb

@@ -2,7 +2,8 @@
 
 module AcidicJob
   module DeliverTransactionallyExtension
-    def deliver_transactionally(options = {})
+    # rubocop:disable Metrics/MethodLength
+    def deliver_transactionally(_options = {})
       job = delivery_job_class
 
       attributes = {
@@ -11,14 +12,15 @@ module AcidicJob
       }
 
       job_args = if job <= ActionMailer::Parameterized::MailDeliveryJob
-        [@mailer_class.name, @action.to_s, "deliver_now", {params: @params, args: @args}]
-      else
-        [@mailer_class.name, @action.to_s, "deliver_now", @params, *@args]
-      end
+                   [@mailer_class.name, @action.to_s, "deliver_now", { params: @params, args: @args }]
+                 else
+                   [@mailer_class.name, @action.to_s, "deliver_now", @params, *@args]
+                 end
 
       attributes[:job_args] = job.new(job_args).serialize
 
       AcidicJob::Staged.create!(attributes)
     end
+    # rubocop:enable Metrics/MethodLength
   end
 end

data/lib/acidic_job/perform_wrapper.rb

@@ -18,7 +18,6 @@ module AcidicJob
 
     private
 
-    # rubocop:disable Metrics/AbcSize
     def set_arguments_for_perform(*args, **kwargs)
       # store arguments passed into `perform` so that we can later persist
       # them to `AcidicJob::Key#job_args` for both ActiveJob and Sidekiq::Worker
@@ -32,6 +31,5 @@ module AcidicJob
                                  []
                                end
     end
-    # rubocop:enable Metrics/AbcSize
   end
 end

data/lib/acidic_job/version.rb

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

data/lib/acidic_job.rb

@@ -25,9 +25,7 @@ module AcidicJob
     # Extend ActiveJob with `perform_transactionally` class method
     klass.include PerformTransactionallyExtension
 
-    if defined?(ActionMailer)
-      ActionMailer::Parameterized::MessageDelivery.include DeliverTransactionallyExtension
-    end
+    ActionMailer::Parameterized::MessageDelivery.include DeliverTransactionallyExtension if defined?(ActionMailer)
 
     # Ensure our `perform` method always runs first to gather parameters
     klass.prepend PerformWrapper
@@ -63,7 +61,7 @@ module AcidicJob
   IDEMPOTENCY_KEY_LOCK_TIMEOUT = 90
 
   # takes a block
-  def idempotently(with:)
+  def with_acidity(given:)
     # execute the block to gather the info on what steps are defined for this job workflow
     steps = yield || []
 
@@ -78,11 +76,12 @@ module AcidicJob
     # close proximity, one of the two will be aborted by Postgres because we're
     # using a transaction with SERIALIZABLE isolation level. It may not look
     # it, but this code is safe from races.
-    key = ensure_idempotency_key_record(idempotency_key_value, workflow, with)
+    key = ensure_idempotency_key_record(idempotency_key_value, workflow, given)
 
     # begin the workflow
     process_key(key)
   end
+  alias_method :idempotently, :with_acidity
 
   def process_key(key)
     @key = key
@@ -95,7 +94,7 @@ module AcidicJob
       recovery_point = @key.recovery_point.to_s
       current_step = @key.workflow[recovery_point]
 
-      if recovery_point == Key::RECOVERY_POINT_FINISHED.to_s
+      if recovery_point == Key::RECOVERY_POINT_FINISHED.to_s # rubocop:disable Style/GuardClause
         break
       elsif current_step.nil?
         raise UnknownRecoveryPoint, "Defined workflow does not reference this step: #{recovery_point}"
@@ -182,7 +181,7 @@ module AcidicJob
         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?
+        key.update!(last_run_at: Time.current, locked_at: Time.current, workflow: workflow) unless key.finished?
       else
         key = Key.create!(
           idempotency_key: key_val,

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: acidic_job
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.7.5
 platform: ruby
 authors:
 - fractaledmind
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-11-16 00:00:00.000000000 Z
+date: 2021-11-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord