Add configuration to disable ActiveJob job signature matching check

[smathieu]

Is your feature request related to a problem? Please describe.

The 7.0 version of RSpec Rails introduced #2745 by @odlp which is overall a great addition, but doesn't provide a configuration to disable this check.

This is a problem for one of our app that uses before_enqueue and around_perform callbacks that modifies the arguments passed to the job.

Describe the solution you'd like

I would like a way to disable this check.

Describe alternatives you've considered

I'm not sure there are other solutions short of updating all the jobs in our app.

Additional context

Here are the callbacks being used:

class ApplicationJob < ActiveJob::Base
  # Automatically retry jobs that encountered a deadlock
  # retry_on ActiveRecord::Deadlocked

  # Most jobs are safe to ignore if the underlying records are no longer available
  discard_on ActiveJob::DeserializationError

  before_enqueue do |job|
    if Actor.current?
      job.arguments << { actor: Actor.current }
    end
  end

  around_perform do |job, block|
    arg = job.arguments.find { |a| a.is_a?(Hash) && a.key?(:actor) }
    if arg
      job.arguments.delete(arg)
      actor = arg[:actor]
    end

    if actor
      Actor.with(actor) do
        block.call
      end
    else
      block.call
    end
  end
end

[os0x]

I agree with you.

FYI: My workaround is allow_any_instance_of(RSpec::Rails::Matchers::ActiveJob::Base).to receive(:detect_args_signature_mismatch).and_return(nil)

[odlp]

Sorry to hear the signature-match check is causing issues for you both – adding an escape hatch sounds reasonable.

FWIW one can also store arbitrary data on an ActiveJob job by overriding #serialize and #deserialize to store/retrieve an extra value on the job payload hash. This would allow you to avoid adding then deleting a job argument, and allow the signature-match check to pass.

So in the example above you'd override #serialize to store the actor, #deserialize to populate an @actor instance variable, which can then be referenced in your around_perform hook...

https://github.com/rails/rails/blob/a5d1e10449c60f731a37298ba1912dbd58b1a9c4/activejob/lib/active_job/core.rb#L105-L162

[smathieu]

Overriding #serialize is a much cleaner implementation and we'll switch over to that.

Given this is an edge cases and there are documented workaround in this ticket, I think we can close this.

[JonRowe]

I'm leaving this open because I think this should respect the existing configuration for verifying doubles, we should check to see if we should verify arguments.

[odlp]

I'm leaving this open because I think this should respect the existing configuration for verifying doubles, we should check to see if we should verify arguments.

I've taken a first pass at this over in #2808. If verify_partial_doubles is configured to be false, or #without_partial_double_verification is being used then we'll skip the signature matching check.