TypeError when payload class is already a constant

CHANGELOG.md

@@ -8,6 +8,7 @@
 
 - Fix argument serialization for ranges that consist of ActiveSupport::TimeWithZone ([#2548](https://github.com/getsentry/sentry-ruby/pull/2548))
 - Prevent starting Vernier in nested transactions ([#2528](https://github.com/getsentry/sentry-ruby/pull/2528))
+- Fix TypeError when Resque.inline == true ([#2564] https://github.com/getsentry/sentry-ruby/pull/2564)
 
 ### Miscellaneous
 

sentry-resque/lib/sentry/resque.rb

@@ -40,16 +40,7 @@ def record(queue, worker, payload, &block)
 
               finish_transaction(transaction, 200)
             rescue Exception => exception
-              klass = if payload["class"].respond_to?(:constantize)
-                payload["class"].constantize
-              else
-                Object.const_get(payload["class"])
-              end
-
-              raise if Sentry.configuration.resque.report_after_job_retries &&
-                       defined?(::Resque::Plugins::Retry) == "constant" &&
-                       klass.is_a?(::Resque::Plugins::Retry) &&
-                       !klass.retry_limit_reached?
+              raise_if_retries_remain(payload["class"], exception)
 
               ::Sentry::Resque.capture_exception(exception, hint: { background: false })
               finish_transaction(transaction, 500)
@@ -93,6 +84,17 @@ def finish_transaction(transaction, status)
           transaction.set_http_status(status)
           transaction.finish
         end
+
+        private
+
+        def raise_if_retries_remain(klass, exception)
+          return unless Sentry.configuration.resque.report_after_job_retries &&
+            defined?(::Resque::Plugins::Retry) == "constant"
+          klass = klass.constantize if klass.respond_to?(:constantize) # Needed for Ruby < 2.7 && Rails < 7.0
+          klass = Object.const_get(klass) unless klass.is_a?(Class)
+
+          raise exception if klass.is_a?(::Resque::Plugins::Retry) && !klass.retry_limit_reached?
+        end
       end
     end
   end

sentry-resque/spec/sentry/resque_spec.rb

@@ -208,6 +208,24 @@ def self.perform(msg)
         expect(event[:tags]).to eq({ "resque.queue" => "default" })
       end
     end
+
+    context "with Resque.inline = true" do
+      around do |example|
+        Resque.inline = true
+        example.run
+        Resque.inline = false
+      end
+
+      it 'reports the class properly' do
+        expect do
+          Resque::Job.create(:default, FailedRetriableJob)
+          process_job(worker)
+        end.not_to raise_error(TypeError)
+
+        event = transport.events.last.to_hash
+        expect(event.dig(:exception, :values, 0, :type)).to eq("ZeroDivisionError")
+      end
+    end
   end
 
   rails_gems = begin