inst-jobs 3.1.8 → 3.1.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4ecc79cbf0af3c0205febacfb4124bf1b86183132fd28f1330b0fb9c097efe62
-  data.tar.gz: 31f7cb19c7ecc6b71833dcc31b998642fec44e06d1bfbaefc2ed868fe331688a
+  metadata.gz: abb4fa28665e7fbe5edcc244f1261e9be5b9b473749739ac21477f4a15f0b9a3
+  data.tar.gz: e20096c0cd70ce6e1b231a24c761a89f50cee44ccb7bd80cad567366f9d027f4
 SHA512:
-  metadata.gz: 1d76db82af500258ac736d6b3819fffcccc0afe81a4c44cf01c540252695f3a3b2d120c00576f4ee02459e0808ddcb1d19c25ddb16c41945fc103718fa8011e3
-  data.tar.gz: 6f6efc00e1a707f5e984455db881281f56eac2b06fd17ee75a5a574c883fcdcb7c99b1483d3d3cadb6d501d6ee4c98b031b997e285bdf598bbc257cf6fb83364
+  metadata.gz: 7249c4fab5ee3113d50f37991878669703918b984b64e56986958eb0798b233a24d94489534e2dd0c00a252fe62a379957a7f7346306e06e4f93c5f0a411bea0
+  data.tar.gz: 5c84ddeace70c4c8cdad94d3b5967025f162e76e4d468fc064b1dd2d9196c442705a6a4b922d45738a8dcc1f733c10ef886d052a839244c3a2fa049efb2d8ca3

data/db/migrate/20200330230722_add_id_to_get_delayed_jobs_index.rb

@@ -9,7 +9,8 @@ class AddIdToGetDelayedJobsIndex < ActiveRecord::Migration[4.2]
 
   def up
     rename_index :delayed_jobs, "get_delayed_jobs_index", "get_delayed_jobs_index_old"
-    add_index :delayed_jobs, %i[queue priority run_at id],
+    add_index :delayed_jobs,
+              %i[queue priority run_at id],
               algorithm: :concurrently,
               where: "locked_at IS NULL AND next_in_strand",
               name: "get_delayed_jobs_index"
@@ -18,7 +19,8 @@ class AddIdToGetDelayedJobsIndex < ActiveRecord::Migration[4.2]
 
   def down
     rename_index :delayed_jobs, "get_delayed_jobs_index", "get_delayed_jobs_index_old"
-    add_index :delayed_jobs, %i[priority run_at queue],
+    add_index :delayed_jobs,
+              %i[priority run_at queue],
               algorithm: :concurrently,
               where: "locked_at IS NULL AND next_in_strand",
               name: "get_delayed_jobs_index"

data/db/migrate/20200825011002_add_strand_order_override.rb

@@ -10,7 +10,8 @@ class AddStrandOrderOverride < ActiveRecord::Migration[4.2]
   def up
     add_column :delayed_jobs, :strand_order_override, :integer, default: 0, null: false
     add_column :failed_jobs, :strand_order_override, :integer, default: 0, null: false
-    add_index :delayed_jobs, %i[strand strand_order_override id],
+    add_index :delayed_jobs,
+              %i[strand strand_order_override id],
               algorithm: :concurrently,
               where: "strand IS NOT NULL",
               name: "next_in_strand_index"

data/db/migrate/20210809145804_add_n_strand_index.rb

@@ -4,7 +4,8 @@ class AddNStrandIndex < ActiveRecord::Migration[5.2]
   disable_ddl_transaction!
 
   def change
-    add_index :delayed_jobs, %i[strand next_in_strand id],
+    add_index :delayed_jobs,
+              %i[strand next_in_strand id],
               name: "n_strand_index",
               where: "strand IS NOT NULL",
               algorithm: :concurrently

data/lib/delayed/backend/active_record.rb

@@ -115,7 +115,7 @@ module Delayed
             # but we don't need to lock when inserting into Delayed::Failed
             if values["strand"] && instance_of?(Job)
               fn_name = connection.quote_table_name("half_md5_as_bigint")
-              quoted_strand = connection.quote(Rails.version < "7.0" ? values["strand"] : values["strand"].value)
+              quoted_strand = connection.quote((Rails.version < "7.0") ? values["strand"] : values["strand"].value)
               sql = "SELECT pg_advisory_xact_lock(#{fn_name}(#{quoted_strand})); #{sql}"
             end
             result = connection.execute(sql, "#{self.class} Create")
@@ -253,7 +253,7 @@ module Delayed
                            offset = 0,
                            query = nil)
           scope = scope_for_flavor(flavor, query)
-          order = flavor.to_s == "future" ? "run_at" : "id desc"
+          order = (flavor.to_s == "future") ? "run_at" : "id desc"
           scope.order(order).limit(limit).offset(offset).to_a
         end
 
@@ -607,7 +607,12 @@ module Delayed
           end
 
           def requeue!
-            attrs = attributes.except("id", "last_error", "locked_at", "failed_at", "locked_by", "original_job_id",
+            attrs = attributes.except("id",
+                                      "last_error",
+                                      "locked_at",
+                                      "failed_at",
+                                      "locked_by",
+                                      "original_job_id",
                                       "requeued_job_id")
             self.class.transaction do
               job = nil

data/lib/delayed/backend/base.rb

@@ -116,7 +116,7 @@ module Delayed
         # effectively balancing the load during queueing
         # overwritten in ActiveRecord::Job to use triggers to balance at run time
         def n_strand_options(strand_name, num_strands)
-          strand_num = num_strands > 1 ? rand(num_strands) + 1 : 1
+          strand_num = (num_strands > 1) ? rand(num_strands) + 1 : 1
           strand_name += ":#{strand_num}" if strand_num > 1
           { strand: strand_name }
         end
@@ -217,7 +217,7 @@ module Delayed
       def failed?
         failed_at
       end
-      alias failed failed?
+      alias_method :failed, :failed?
 
       def expired?
         expires_at && (self.class.db_time_now >= expires_at)

data/lib/delayed/cli.rb

@@ -49,7 +49,7 @@ module Delayed
         daemon.daemonize!
         start
       when nil
-        puts option_parser.to_s
+        puts option_parser.help
       else
         raise("Unknown command: #{command.inspect}")
       end
@@ -82,7 +82,8 @@ module Delayed
         opts.on("-c", "--config [CONFIG_PATH]", "Use alternate config file (default #{@options[:config_file]})") do |c|
           @options[:config_file] = c
         end
-        opts.on("-p", "--pid [PID_PATH]",
+        opts.on("-p",
+                "--pid [PID_PATH]",
                 "Use alternate folder for PID files (default #{@options[:pid_folder]})") do |p|
           @options[:pid_folder] = p
         end

data/lib/delayed/lifecycle.rb

@@ -46,7 +46,7 @@ module Delayed
       missing_callback(event) unless @callbacks.key?(event)
 
       unless EVENTS[event].size == args.size
-        raise ArgumentError, "Callback #{event} expects #{EVENTS[event].size} parameter(s): #{EVENTS[event].join(', ')}"
+        raise ArgumentError, "Callback #{event} expects #{EVENTS[event].size} parameter(s): #{EVENTS[event].join(", ")}"
       end
 
       @callbacks[event].execute(*args, &block)

data/lib/delayed/log_tailer.rb

@@ -5,8 +5,8 @@ module Delayed
     def run
       if Rails.logger.respond_to?(:log_path)
         log_path = Rails.logger.log_path
-      elsif Rails.logger.instance_variable_get("@logdev").try(:instance_variable_get, "@dev").try(:path)
-        log_path = Rails.logger.instance_variable_get("@logdev").instance_variable_get("@dev").path
+      elsif Rails.logger.instance_variable_get(:@logdev).try(:instance_variable_get, "@dev").try(:path)
+        log_path = Rails.logger.instance_variable_get(:@logdev).instance_variable_get(:@dev).path
       else
         return
       end

data/lib/delayed/message_sending.rb

@@ -19,7 +19,7 @@ module Delayed
                          @object.protected_methods.exclude?(method) &&
                          @object.private_methods.exclude?(method)
 
-        sender_is_object = @sender == @object
+        sender_is_object = @sender.equal?(@object)
         sender_is_class = @sender.is_a?(@object.class)
 
         # even if the call is async, if the call is _going_ to generate an error, we make it synchronous
@@ -54,8 +54,10 @@ module Delayed
 
           if ::Delayed::Job != ::Delayed::Backend::ActiveRecord::Job || connection != ::Delayed::Job.connection
             connection.after_transaction_commit do
-              ::Delayed::Job.enqueue(::Delayed::PerformableMethod.new(@object, method,
-                                                                      args: args, kwargs: kwargs,
+              ::Delayed::Job.enqueue(::Delayed::PerformableMethod.new(@object,
+                                                                      method,
+                                                                      args: args,
+                                                                      kwargs: kwargs,
                                                                       on_failure: on_failure,
                                                                       on_permanent_failure: on_permanent_failure,
                                                                       sender: @sender),
@@ -65,7 +67,8 @@ module Delayed
           end
         end
 
-        result = ::Delayed::Job.enqueue(::Delayed::PerformableMethod.new(@object, method,
+        result = ::Delayed::Job.enqueue(::Delayed::PerformableMethod.new(@object,
+                                                                         method,
                                                                          args: args,
                                                                          kwargs: kwargs,
                                                                          on_failure: on_failure,

data/lib/delayed/performable_method.rb

@@ -21,6 +21,8 @@ module Delayed
       self.permanent_fail_cb = on_permanent_failure
       self.sender_is_object = sender.equal?(object)
       self.sender_is_class = sender.is_a?(object.class)
+      # bypass visibility checks (see MessageSending#__calculate_sender_for_delay)
+      self.sender_is_object = true if sender.nil?
     end
 
     def display_name
@@ -36,7 +38,7 @@ module Delayed
       kwargs = self.kwargs || {}
 
       # back-compat for jobs queued before we assigned these in initialize
-      self.sender_is_object = sender == object if sender_is_object.nil?
+      self.sender_is_object = sender.equal?(object) if sender_is_object.nil?
       self.sender_is_class = sender.is_a?(object.class) if sender_is_class.nil?
 
       if sender_is_object || (sender_is_class && object.protected_methods.include?(method))
@@ -63,9 +65,9 @@ module Delayed
     def deep_de_ar_ize(arg)
       case arg
       when Hash
-        "{#{arg.map { |k, v| "#{deep_de_ar_ize(k)} => #{deep_de_ar_ize(v)}" }.join(', ')}}"
+        "{#{arg.map { |k, v| "#{deep_de_ar_ize(k)} => #{deep_de_ar_ize(v)}" }.join(", ")}}"
       when Array
-        "[#{arg.map { |a| deep_de_ar_ize(a) }.join(', ')}]"
+        "[#{arg.map { |a| deep_de_ar_ize(a) }.join(", ")}]"
       when ActiveRecord::Base
         "#{arg.class}.find(#{arg.id})"
       else
@@ -77,7 +79,7 @@ module Delayed
       obj_name = object.is_a?(ActiveRecord::Base) ? "#{object.class}.find(#{object.id}).#{method}" : display_name
       kgs = kwargs || {}
       kwargs_str = kgs.map { |(k, v)| ", #{k}: #{deep_de_ar_ize(v)}" }.join
-      "#{obj_name}(#{args.map { |a| deep_de_ar_ize(a) }.join(', ')}#{kwargs_str})"
+      "#{obj_name}(#{args.map { |a| deep_de_ar_ize(a) }.join(", ")}#{kwargs_str})"
     end
   end
 end

data/lib/delayed/periodic.rb

@@ -83,7 +83,7 @@ module Delayed
     def tag
       "periodic: #{@name}"
     end
-    alias display_name tag
+    alias_method :display_name, :tag
 
     def self.now
       Time.zone.now

data/lib/delayed/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Delayed
-  VERSION = "3.1.8"
+  VERSION = "3.1.10"
 end

data/lib/delayed/work_queue/parent_process/server.rb

@@ -163,7 +163,7 @@ module Delayed
                 forced_latency: forced_latency
               )
               logger.debug(
-                "Fetched and locked #{response.values.flatten.size} new jobs for workers (#{response.keys.join(', ')})."
+                "Fetched and locked #{response.values.flatten.size} new jobs for workers (#{response.keys.join(", ")})."
               )
               response.each do |(worker_name, locked_jobs)|
                 if worker_name == prefetch_owner
@@ -267,7 +267,7 @@ module Delayed
         end
 
         def prefetch_owner
-          "prefetch:#{Socket.gethostname rescue 'X'}"
+          "prefetch:#{Socket.gethostname rescue "X"}"
         end
 
         def parent_exited?

data/lib/delayed/worker/process_helper.rb

@@ -9,9 +9,9 @@ module Delayed
       ALIVE_CHECK_LINUX = '[ -d "/proc/$WORKER_PID" ]'
       ALIVE_CHECK_MAC = "ps -p $WORKER_PID > /dev/null"
       ALIVE_CHECK = RUBY_PLATFORM.include?("darwin") ? ALIVE_CHECK_MAC : ALIVE_CHECK_LINUX
-      SCRIPT_TEMPLATE = <<-BASH
-        WORKER_PID="%<pid>d" # an example, filled from ruby when the check is created
-        ORIGINAL_MTIME="%<mtime>s" # an example, filled from ruby when the check is created
+      SCRIPT_TEMPLATE = <<~SH
+        WORKER_PID="%{pid}" # an example, filled from ruby when the check is created
+        ORIGINAL_MTIME="%{mtime}" # an example, filled from ruby when the check is created
 
         if #{ALIVE_CHECK}; then
             CURRENT_MTIME=$(#{STAT})
@@ -26,7 +26,7 @@ module Delayed
         else
             exit 255 # The process is no more, trigger a "critical" state.
         fi
-      BASH
+      SH
 
       def self.mtime(pid)
         if RUBY_PLATFORM.include?("darwin")

data/lib/delayed/worker.rb

@@ -77,7 +77,7 @@ module Delayed
     end
 
     def name
-      @name ||= "#{Socket.gethostname rescue 'X'}:#{id}"
+      @name ||= "#{Socket.gethostname rescue "X"}:#{id}"
     end
 
     def process_name=(new_name)
@@ -100,7 +100,7 @@ module Delayed
     def start
       logger.info "Starting worker"
       self.process_name =
-        "start:#{Settings.worker_procname_prefix}#{@queue_name}:#{min_priority || 0}:#{max_priority || 'max'}"
+        "start:#{Settings.worker_procname_prefix}#{@queue_name}:#{min_priority || 0}:#{max_priority || "max"}"
       @self_pipe = IO.pipe
       work_queue.init
 
@@ -170,7 +170,7 @@ module Delayed
 
       self.class.lifecycle.run_callbacks(:loop, self) do
         self.process_name =
-          "pop:#{Settings.worker_procname_prefix}#{@queue_name}:#{min_priority || 0}:#{max_priority || 'max'}"
+          "pop:#{Settings.worker_procname_prefix}#{@queue_name}:#{min_priority || 0}:#{max_priority || "max"}"
         job = self.class.lifecycle.run_callbacks(:pop, self) do
           work_queue.get_and_lock_next_available(name, config)
         end
@@ -196,7 +196,7 @@ module Delayed
           end
         else
           self.process_name =
-            "wait:#{Settings.worker_procname_prefix}#{@queue_name}:#{min_priority || 0}:#{max_priority || 'max'}"
+            "wait:#{Settings.worker_procname_prefix}#{@queue_name}:#{min_priority || 0}:#{max_priority || "max"}"
           sleep(Settings.sleep_delay + (rand * Settings.sleep_delay_stagger)) unless exit?
         end
       end
@@ -220,7 +220,7 @@ module Delayed
             end
             job.destroy
           end
-          logger.info("Completed #{log_job(job, :short)} #{format('%.0fms', (runtime * 1000))}")
+          logger.info("Completed #{log_job(job, :short)} #{format("%.0fms", (runtime * 1000))}")
         end
       rescue ::Delayed::RetriableError => e
         can_retry = job.attempts + 1 < job.inferred_max_attempts
@@ -279,7 +279,7 @@ module Delayed
       previous_tmpdir = ENV.fetch("TMPDIR", nil)
 
       self.class.running_job(job) do
-        dir = Dir.mktmpdir("job-#{job.id}-#{name.gsub(/[^\w.]/, '.')}-")
+        dir = Dir.mktmpdir("job-#{job.id}-#{name.gsub(/[^\w.]/, ".")}-")
         begin
           ENV["TMPDIR"] = dir
           yield

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: inst-jobs
 version: !ruby/object:Gem::Version
-  version: 3.1.8
+  version: 3.1.10
 platform: ruby
 authors:
 - Cody Cutrer
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-04-18 00:00:00.000000000 Z
+date: 2023-05-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -285,33 +285,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3.10'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: rubocop-inst
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.19'
+        version: '1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.19'
-- !ruby/object:Gem::Dependency
-  name: rubocop-performance
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.12.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.12.0
+        version: '1'
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement