fleiss 0.4.4 → 0.4.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ee721d0d8b4ceb6a3308ded176bac0cbb64aafe602bf4aacda952d9440cff486
-  data.tar.gz: 93f4afd58038d95d24bb75640fc96b613a7eac6a421e18852ade1e25ad9df76b
+  metadata.gz: 4a8ecda4ed800fa1d15848bf37c3f7f1a49cba62be614d25eea8725c00baab0d
+  data.tar.gz: fade5ca7d61dad9e7a912b99125b1b43159ab61df2c68b02fb8ed21a73382854
 SHA512:
-  metadata.gz: '078d55bb3edf9c2419fbbd7c1906195019e50b356301cb2a4570645b5abae563975f6c0ea6d0632ce039f87c7faddb10b9a21170a0486e06011606a4c4039899'
-  data.tar.gz: cbe4badf9b18b68b9e664ed9bd9a0ec2e1021dfe5ea088ae6a404e6938e325f92fffdb1d24ee8c642b103cdf847272fa85ff2e057c120ee01ce6be8a6d78aed1
+  metadata.gz: ee3ee999372b9d214cd7261e115a0a5ceb7b0c3b266dc2924cb6c7f8c888fbd98140fd39a02a75e108af4f541bee564a67cef179ed62e38d4a0ae019f071f182
+  data.tar.gz: 70d922d556feeef102242c726c1812f4f4625ee25f19d01087074aa0725b677d0e74e043097f665b9626f3e16bc5a424f5441e8a3e96088e180cabb9d1a99b45

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    fleiss (0.4.4)
+    fleiss (0.4.5)
       activejob (>= 6.0)
       activerecord (>= 6.0)
       concurrent-ruby

data/README.md

@@ -38,6 +38,16 @@ class ExpringJob < ActiveJob::Base
 end
 ```
 
+Allow to subscribe on worker perform method and detect errors
+
+```ruby
+ActiveSupport::Notifications.subscribe('worker_perform.fleiss') do |event|
+  break unless event.payload.key?(:exception_object)
+
+  Raven.capture_exception(event.payload[:exception_object])
+end
+```
+
 Include the data migration:
 
 ```ruby

data/fleiss.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name          = 'fleiss'
-  s.version       = '0.4.4'
+  s.version       = '0.4.5'
   s.authors       = ['Black Square Media Ltd']
   s.email         = ['info@blacksquaremedia.com']
   s.summary       = %(Minimialist background jobs backed by ActiveJob and ActiveRecord.)

data/lib/fleiss/worker.rb

@@ -76,7 +76,8 @@ class Fleiss::Worker
 
     log(:info) { "Worker #{uuid} execute job ##{job.id} (by thread #{thread_id})" }
     finished = false
-    begin
+
+    ActiveSupport::Notifications.instrument('worker_perform.fleiss', id: job.id, uuid: uuid, thread_id: thread_id) do |payload|
       ActiveJob::Base.execute job.job_data
       finished = true
     rescue StandardError
@@ -84,6 +85,7 @@ class Fleiss::Worker
       raise
     ensure
       finished ? job.finish(owner) : job.reschedule(owner)
+      payload[:finished] = finished
     end
   end
 

data/spec/fleiss/worker_spec.rb

@@ -16,6 +16,13 @@ RSpec.describe Fleiss::Worker do
     runner.kill
   end
 
+  around do |example|
+    callback = ->(*args) { notifications.push ActiveSupport::Notifications::Event.new(*args) }
+    ActiveSupport::Notifications.subscribed(callback, 'worker_perform.fleiss') do
+      example.call
+    end
+  end
+
   def wait_for
     100.times do
       break if yield
@@ -43,5 +50,19 @@ RSpec.describe Fleiss::Worker do
     TestJob.perform_later('raise')
     wait_for { Fleiss.backend.not_finished.count.zero? }
     expect(Fleiss.backend.finished.count).to eq(1)
+
+    expect(notifications.size).to eq(1)
+    expect(notifications.first.payload).to have_key(:id)
+    expect(notifications.first.payload).to have_key(:uuid)
+    expect(notifications.first.payload).to have_key(:thread_id)
+    expect(notifications.first.payload[:finished]).to be_truthy
+    expect(notifications.first.payload[:exception_object]).to be_an_instance_of(RuntimeError)
+    expect(notifications.first.payload[:exception_object].message).to eq('Failing')
+  end
+
+  private
+
+  def notifications
+    @notifications ||= []
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fleiss
 version: !ruby/object:Gem::Version
-  version: 0.4.4
+  version: 0.4.5
 platform: ruby
 authors:
 - Black Square Media Ltd
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-09-29 00:00:00.000000000 Z
+date: 2021-10-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activejob