be_retryable not using global settings

wspurgin / rspec-sidekiq

RSpec for Sidekiq

https://github.com/wspurgin/rspec-sidekiq

Other

651 stars 133 forks source link

be_retryable not using global settings #166

Closed chrishough closed 2 months ago

chrishough commented 4 years ago

The matcher for it { is_expected.to be_retryable 10 } is not picking up the :max_retries: 10 in the config/sidekiq.yml file.

yskttm commented 1 year ago

I also encountered the same Issue. Is it unresolved?

wspurgin commented 2 months ago

The global config for Sidekiq doesn't actually set anything in the job specific options which is what be_retryable specifically tests. The Max Retries config is only leveraged directly in the job retry code in Sidekiq (and the job setting takes precedence naturally).