Fixes: #472

If there is every an error within the payload object itself, within delayed jobs, rollbar does not propagate that error, since delayed job fails to call the :invoke_job lifecycle hook and instead calls the :failure lifecycle hook.

lib/rollbar/plugins/delayed_job/job_data.rb

@@ -32,6 +32,8 @@ def handler_data
         return payload_object unless payload_object.respond_to?(:object)
 
         object_data(payload_object.object)
+      rescue
+        payload_object = job
       end
 
       def object_data(object)

lib/rollbar/plugins/delayed_job/plugin.rb

@@ -10,6 +10,9 @@ class << self
     class RollbarPlugin < ::Delayed::Plugin
       callbacks do |lifecycle|
         lifecycle.around(:invoke_job, &Delayed::invoke_job_callback)
+        lifecycle.after(:failure) do |worker, job, *args, &block|
+          Delayed.report(job.last_error, job)
+        end
       end
     end
 

spec/rollbar/plugins/delayed_job_spec.rb

@@ -34,6 +34,15 @@ def do_job_please!(a, b)
     end
   end
 
+  context 'with failed deserialization' do
+    it 'sends the exception' do
+      expect(Rollbar).to receive(:scope).with(kind_of(Hash)).and_call_original
+      allow_any_instance_of(Delayed::Backend::Base).to receive(:payload_object).and_raise(Delayed::DeserializationError)
+      expect_any_instance_of(Rollbar::Notifier).to receive(:error)
+
+      FailingJob.new.delay.do_job_please!(:foo, :bar)
+    end
+  end
 
   describe '.build_job_data' do
     let(:job) { double(:payload_object => {}) }