Prevent duplicated events when AJ adapter has a specific integration. (#815)

In rails 5, the job class’s `queue_adapter` property is not a class (such as `ActiveJob::QueueAdapters::SidekiqAdapter`) like it was in rails 4, but an instance (of a class such as `ActiveJob::QueueAdapters::SidekiqAdapter`).

The specs were previously not testing the logic in `already_supported_by_specific_integration?` which is why there wasn’t a failing spec.

Author: BrentWheeldon

lib/raven/integrations/rails/active_job.rb

@@ -28,7 +28,11 @@ def capture_and_reraise_with_sentry(job, block)
       end
 
       def already_supported_by_specific_integration?(job)
-        ALREADY_SUPPORTED_SENTRY_ADAPTERS.include?(job.class.queue_adapter.to_s)
+        if ::Rails.version.to_f < 5.0
+          ALREADY_SUPPORTED_SENTRY_ADAPTERS.include?(job.class.queue_adapter.to_s)
+        else
+          ALREADY_SUPPORTED_SENTRY_ADAPTERS.include?(job.class.queue_adapter.class.to_s)
+        end
       end
 
       def raven_context(job)

spec/raven/integrations/rails/activejob_spec.rb

@@ -2,7 +2,6 @@
 
 if defined? ActiveJob
   class MyActiveJob < ActiveJob::Base
-    self.queue_adapter = :inline
     self.logger = nil
 
     class TestError < RuntimeError
@@ -30,6 +29,7 @@ def rescue_callback(error)
 
   before(:each) do
     Raven.client.transport.events = []
+    MyActiveJob.queue_adapter = :inline
   end
 
   it_should_behave_like "Raven default capture behavior" do
@@ -68,10 +68,8 @@ def rescue_callback(error)
 
   context "when we are using an adapter which has a specific integration" do
     it "does not trigger sentry and re-raises" do
+      MyActiveJob.queue_adapter = :sidekiq
       job = MyActiveJob.new
-      def job.already_supported_by_specific_integration?(*)
-        true
-      end
 
       expect { job.perform_now }.to raise_error(MyActiveJob::TestError)