canvas-jobs 0.9.11 → 0.9.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 64df58c18931c32c09ae1e120ff915e0d7643f77
-  data.tar.gz: 7048135401a9e919daa209ab0c3f2944950e1b28
+  metadata.gz: ed7324f6c7cf621f56a91226d77c82c66d6917ab
+  data.tar.gz: 90589f8a1452f79e0d1fd9a7508e810918792515
 SHA512:
-  metadata.gz: b64cbe915819f26c25b5127447f3d68bdb5d564719d6a2b4c869eb6269861ff8f1e3fbd7d2f23e2058102e5c9872c13d1b4f50c44fb104b89010f18eafaade39
-  data.tar.gz: e7b5c0e267fa4f5281c936f0a826a7795ea725ddf432b252ab14271ea07aa870b915d25dea5f88143474324c8bd9ab293322776120e8973390efa5779040e9b4
+  metadata.gz: 2aea05a91a53f7bff8ddb7d61afefe0f1b0aaa757a287e3adf7616cd1b6433517ffb66f62940273a56188c0a0956e6b301b086ecba483be8aad20886ae95b9a2
+  data.tar.gz: 04e1e5ab0d1394bc0f354147a830081d5dea1602aa713a3166a9461f7a6e3e92c7324a2374b470e57aa97178184906e418f0b8e6f0ab8bb1085e86476cba5491

data/lib/delayed/lifecycle.rb

@@ -7,6 +7,7 @@ module Delayed
       :pop              => [:worker],
       :exceptional_exit => [:worker, :exception],
       :invoke_job       => [:job],
+      :error            => [:worker, :job, :exception],
     }
 
     def initialize

data/lib/delayed/version.rb

@@ -1,3 +1,3 @@
 module Delayed
-  VERSION = "0.9.11"
+  VERSION = "0.9.12"
 end

data/lib/delayed/worker.rb

@@ -136,7 +136,7 @@ class Worker
       runtime = Benchmark.realtime do
         if job.batch?
           # each job in the batch will have perform called on it, so we don't
-          # need a timeout around this 
+          # need a timeout around this
           count = perform_batch(job)
         else
           job.invoke_job
@@ -147,7 +147,9 @@ class Worker
     end
     count
   rescue Exception => e
-    handle_failed_job(job, e)
+    self.class.lifecycle.run_callbacks(:error, self, job, e) do
+      handle_failed_job(job, e)
+    end
     count
   end
 

data/spec/delayed/worker_spec.rb

@@ -0,0 +1,16 @@
+require_relative "../spec_helper"
+
+module Delayed
+  describe Worker do
+    describe "#perform" do
+      it "fires off an error callback when a job raises an exception" do
+        fired = false
+        Worker.lifecycle.before(:error) {|worker, exception| fired = true}
+        worker = Worker.new
+        job = double(:last_error= => nil, attempts: 1, reschedule: nil)
+        worker.perform(job)
+        expect(fired).to be_truthy
+      end
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: canvas-jobs
 version: !ruby/object:Gem::Version
-  version: 0.9.11
+  version: 0.9.12
 platform: ruby
 authors:
 - Tobias Luetke
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-14 00:00:00.000000000 Z
+date: 2015-04-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: after_transaction_commit
@@ -263,6 +263,7 @@ files:
 - lib/delayed/yaml_extensions.rb
 - lib/delayed_job.rb
 - spec/active_record_job_spec.rb
+- spec/delayed/worker_spec.rb
 - spec/gemfiles/32.gemfile
 - spec/gemfiles/32.gemfile.lock
 - spec/gemfiles/40.gemfile
@@ -304,6 +305,7 @@ specification_version: 4
 summary: Instructure-maintained fork of delayed_job
 test_files:
 - spec/active_record_job_spec.rb
+- spec/delayed/worker_spec.rb
 - spec/gemfiles/32.gemfile
 - spec/gemfiles/32.gemfile.lock
 - spec/gemfiles/40.gemfile
@@ -320,3 +322,4 @@ test_files:
 - spec/shared/worker.rb
 - spec/shared_jobs_specs.rb
 - spec/spec_helper.rb
+has_rdoc: