faktory_worker_ruby 1.2.0 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 57f98beb553cb8d38e36d45b1721c15446201a30db101827d277ef7f3b9fb542
-  data.tar.gz: a0d82206d826d345bdcde2271ccf300c8af2ccb7ce4c2140900f00b0330ebeac
+  metadata.gz: abc45bf7c07e61197705a9283c38e6d399a81701f433d8a5eded1e6cb6b480e3
+  data.tar.gz: ac96de6c442ad806cf3cafa5fc839e5509454a8f315229f4014cddfcd9bbb28e
 SHA512:
-  metadata.gz: ed5afdc2c7d07f54b90eb1230d50b1d0bdfb219d3ac6f12d4c2f35c05eb7f0b73c3990c84d65c77567f127c1c082f6e858480165aa0bf1e697bef6aa81517075
-  data.tar.gz: 01ea007ff63b7e91a94cc7134f57d783680b9a2ba88be5677b93a505eb55d8e39d2cec9f53a3503b5270b2cfcf8a16909569082b19c59c6439d7474f402da7db
+  metadata.gz: ef6262279803c776c204d2efd58bb547617519bf8277a78e3a46f3789e39bff21220134556f8c209b3d7fd26eb85627b24062f721de3c87324acb0063ec778e4
+  data.tar.gz: 6de7bd208b4ab94ab57b3513159c5ab047f5144fb6152365d52535e3274f8306a416a1124f1c299fe0269542bdb61670958323b8be2285face085f8e3f53ae35

data/.standard.yml

@@ -2,11 +2,8 @@ ruby_version: 2.5.0
 fix: true
 parallel: true
 ignore:
-  - test/**/*
-  - examples/**/*
-  - myapp/**/*
-  - 'lib/faktory.rb':
-    - Lint/InheritException
   - 'lib/**/*':
-    - Lint/RescueException
+    - Style/GlobalVars
+  - 'test/**/*':
+    - Lint/ConstantDefinitionInBlock
     - Style/GlobalVars

data/Changes.md

@@ -1,5 +1,11 @@
 # Changes
 
+## 1.3.0
+
+- Fix `faktory/testing` [#74]
+- Provide more batch helper methods along with a more complex
+  workflow in `examples/complex_workflow.rb`.
+
 ## 1.2.0
 
 - Dup options before deep merging [#67]

data/Gemfile.lock

@@ -1,23 +1,23 @@
 PATH
   remote: .
   specs:
-    faktory_worker_ruby (1.2.0)
+    faktory_worker_ruby (1.3.0)
       connection_pool (~> 2.2, >= 2.2.2)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activejob (7.0.4.2)
-      activesupport (= 7.0.4.2)
+    activejob (7.0.4.3)
+      activesupport (= 7.0.4.3)
       globalid (>= 0.3.6)
-    activesupport (7.0.4.2)
+    activesupport (7.0.4.3)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
       tzinfo (~> 2.0)
     ast (2.4.2)
-    concurrent-ruby (1.2.0)
-    connection_pool (2.3.0)
+    concurrent-ruby (1.2.2)
+    connection_pool (2.4.0)
     docile (1.4.0)
     globalid (1.1.0)
       activesupport (>= 5.0)
@@ -25,42 +25,42 @@ GEM
       concurrent-ruby (~> 1.0)
     json (2.6.3)
     language_server-protocol (3.17.0.3)
-    minitest (5.17.0)
+    minitest (5.18.0)
     minitest-hooks (1.5.0)
       minitest (> 5.3)
     parallel (1.22.1)
-    parser (3.2.0.0)
+    parser (3.2.2.0)
       ast (~> 2.4.1)
     rainbow (3.1.1)
     rake (13.0.6)
-    regexp_parser (2.6.2)
+    regexp_parser (2.7.0)
     rexml (3.2.5)
-    rubocop (1.42.0)
+    rubocop (1.48.1)
       json (~> 2.3)
       parallel (~> 1.10)
-      parser (>= 3.1.2.1)
+      parser (>= 3.2.0.0)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml (>= 3.2.5, < 4.0)
-      rubocop-ast (>= 1.24.1, < 2.0)
+      rubocop-ast (>= 1.26.0, < 2.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.24.1)
-      parser (>= 3.1.1.0)
-    rubocop-performance (1.15.2)
+      unicode-display_width (>= 2.4.0, < 3.0)
+    rubocop-ast (1.28.0)
+      parser (>= 3.2.1.0)
+    rubocop-performance (1.16.0)
       rubocop (>= 1.7.0, < 2.0)
       rubocop-ast (>= 0.4.0)
-    ruby-progressbar (1.11.0)
+    ruby-progressbar (1.13.0)
     simplecov (0.22.0)
       docile (~> 1.1)
       simplecov-html (~> 0.11)
       simplecov_json_formatter (~> 0.1)
     simplecov-html (0.12.3)
     simplecov_json_formatter (0.1.4)
-    standard (1.22.1)
+    standard (1.25.3)
       language_server-protocol (~> 3.17.0.2)
-      rubocop (= 1.42.0)
-      rubocop-performance (= 1.15.2)
+      rubocop (~> 1.48.1)
+      rubocop-performance (~> 1.16.0)
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)
     unicode-display_width (2.4.2)

data/bin/faktory-worker

@@ -4,7 +4,7 @@
 # RUBYOPT=-w bundle exec faktory
 $TESTING = false
 
-require_relative '../lib/faktory/cli'
+require_relative "../lib/faktory/cli"
 
 begin
   cli = Faktory::CLI.instance
@@ -12,7 +12,7 @@ begin
   cli.run
 rescue => e
   raise e if $DEBUG
-  STDERR.puts e.message
-  STDERR.puts e.backtrace.join("\n")
+  warn e.message
+  warn e.backtrace.join("\n")
   exit 1
 end

data/lib/faktory/batch.rb

@@ -91,6 +91,10 @@ module Faktory
       @bid = bid
     end
 
+    def parent=(parent)
+      @parent_bid = parent.bid
+    end
+
     def success=(val)
       raise "Batch cannot be modified once created" if bid
       @success = to_callback(val)

data/lib/faktory/client.rb

@@ -163,7 +163,9 @@ module Faktory
     # Hash must contain "jid", "jobtype" and "args" elements at minimum.
     # Returned value will either be the JID String if successful OR
     # a symbol corresponding to an error.
+    # NB: aliased by faktory/testing
     def push(job)
+      job = job.transform_keys(&:to_s)
       job["jid"] ||= SecureRandom.hex(12)
       job["queue"] ||= "default"
       raise ArgumentError, "Missing `jobtype` attribute: #{job.inspect}" unless job["jobtype"]
@@ -246,6 +248,7 @@ module Faktory
       @location.scheme =~ /tls/
     end
 
+    # NB: aliased by faktory/testing
     def open_socket(timeout = DEFAULT_TIMEOUT)
       if tls?
         require "openssl"

data/lib/faktory/job.rb

@@ -23,6 +23,7 @@ module Faktory
   module Job
     attr_accessor :jid
     attr_accessor :bid
+    attr_accessor :_custom
 
     include Faktory::Trackable
 
@@ -37,12 +38,26 @@ module Faktory
       Setter.new(options)
     end
 
+    # this method is typically called within a batch job to reopen
+    # the batch in order to add more jobs.
     def batch
       if bid
         @batch ||= Faktory::Batch.new(bid)
       end
     end
 
+    # this method is typically called within a child batch callback to reopen
+    # the parent batch in order to add more jobs or workflow steps.
+    def parent_batch
+      if parent_bid
+        @parent_batch ||= Faktory::Batch.new(parent_bid)
+      end
+    end
+
+    def parent_bid
+      _custom["_pbid"]
+    end
+
     def logger
       Faktory.logger
     end

data/lib/faktory/job_logger.rb

@@ -5,7 +5,7 @@ module Faktory
       logger.info("start".freeze)
       yield
       logger.info("done: #{elapsed(start)} sec")
-    rescue Exception
+    rescue Exception # rubocop:disable Lint/RescueException
       logger.info("fail: #{elapsed(start)} sec")
       raise
     end

data/lib/faktory/processor.rb

@@ -73,7 +73,7 @@ module Faktory
       @mgr.processor_stopped(self)
     rescue Faktory::Shutdown
       @mgr.processor_stopped(self)
-    rescue Exception => ex
+    rescue Exception => ex # rubocop:disable Lint/RescueException
       @mgr.processor_died(self, ex)
     end
 
@@ -131,6 +131,7 @@ module Faktory
             klass = constantize(payload["jobtype"])
             jobinst = klass.new
             jobinst.jid = payload["jid"]
+            jobinst._custom = payload["custom"]
             yield jobinst
           end
         end
@@ -151,7 +152,7 @@ module Faktory
         # the timeout.  Fail it so we can release any locks server-side
         # and immediately restart it.
         work.fail(shut)
-      rescue Exception => ex
+      rescue Exception => ex # rubocop:disable Lint/RescueException
         handle_exception(ex, {context: "Job raised exception", job: work.job})
         work.fail(ex)
         raise ex

data/lib/faktory/testing.rb

@@ -76,7 +76,7 @@ module Faktory
 
   class Client
     alias_method :real_push, :push
-    alias_method :real_open, :open
+    alias_method :real_open_socket, :open_socket
 
     def push(job)
       if Faktory::Testing.inline?
@@ -94,9 +94,9 @@ module Faktory
       end
     end
 
-    def open(*args)
+    def open_socket(*args)
       unless Faktory::Testing.enabled?
-        real_open(*args)
+        real_open_socket(*args)
       end
     end
   end

data/lib/faktory/util.rb

@@ -15,7 +15,7 @@ module Faktory
 
     def watchdog(last_words)
       yield
-    rescue Exception => ex
+    rescue Exception => ex # rubocop:disable Lint/RescueException
       handle_exception(ex, {context: last_words})
       raise ex
     end

data/lib/faktory/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Faktory
-  VERSION = "1.2.0"
+  VERSION = "1.3.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: faktory_worker_ruby
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.3.0
 platform: ruby
 authors:
 - Mike Perham
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-02-02 00:00:00.000000000 Z
+date: 2023-03-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: connection_pool
@@ -154,7 +154,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.4
+rubygems_version: 3.4.7
 signing_key:
 specification_version: 4
 summary: Ruby worker for Faktory