Fixes issue resque#406: supports calling perform hooks when Resque.inline = true.

Author: jonhyman

lib/resque/job.rb

@@ -44,7 +44,8 @@ def self.create(queue, klass, *args)
       Resque.validate(klass, queue)
 
       if Resque.inline?
-        constantize(klass).perform(*decode(encode(args)))
+        # Instantiating a Resque::Job and calling perform on it so callbacks run
+        new(:inline, {'class' => klass, 'args' => args}).perform
       else
         Resque.push(queue, :class => klass.to_s, :args => args)
       end

test/job_hooks_test.rb

@@ -420,4 +420,45 @@ def self.on_failure_record_history(exception, history)
       "oh no"
     ]
   end
+
+  class ::CallbacksInline
+    @queue = :callbacks_inline
+
+    def self.before_perform_record_history(history, count)
+      history << :before_perform
+      count['count'] += 1
+    end
+
+    def self.after_perform_record_history(history, count)
+      history << :after_perform
+      count['count'] += 1
+    end
+
+    def self.around_perform_record_history(history, count)
+      history << :start_around_perform
+      count['count'] += 1
+      yield
+      history << :finish_around_perform
+      count['count'] += 1
+    end
+
+    def self.perform(history, count)
+      history << :perform
+      $history = history
+      $count = count
+    end
+  end
+
+  test "it runs callbacks when inline is true" do
+    begin
+      Resque.inline = true
+      # Sending down two parameters that can be passed and updated by reference
+      result = Resque.enqueue(CallbacksInline, [], {'count' => 0})
+      assert_equal true, result, "perform returned true"
+      assert_equal $history, [:before_perform, :start_around_perform, :perform, :finish_around_perform, :after_perform]
+      assert_equal 4, $count['count']
+    ensure
+      Resque.inline = false
+    end
+  end
 end