Merge pull request #593 from taiki45/no-system-stack-error

No SystemStackError with any active jobs

lib/airbrake/rails/active_job.rb

@@ -16,7 +16,7 @@ module ActiveJob
             notice[:context][:component] = 'active_job'
             notice[:context][:action] = self.class.name
 
-            notice[:params] = as_json
+            notice[:params] = serialize
 
             # We special case Resque because it kills our workers by forking, so
             # we use synchronous delivery instead.

spec/apps/rails/dummy_app.rb

@@ -58,7 +58,14 @@ def raise_error_after_rollback
   class BingoJob < ActiveJob::Base
     queue_as :bingo
 
+    class BingoWrapper
+      def initialize(bingo)
+        @bingo = bingo
+      end
+    end
+
     def perform(*_args)
+      @wrapper = BingoWrapper.new(self)
       raise AirbrakeTestError, 'active_job error'
     end
   end

spec/integration/rails/rails_spec.rb

@@ -112,6 +112,16 @@
         ).to have_been_made.at_least_once
       end
 
+      it "does not raise SystemStackError" do
+        get '/active_job'
+        sleep 2
+
+        wait_for(
+          a_request(:post, endpoint).
+          with(body: /"type":"SystemStackError"/)
+        ).not_to have_been_made
+      end
+
       context "when Airbrake is not configured" do
         it "doesn't report errors" do
           allow(Airbrake).to receive(:build_notice).and_return(nil)