Skip to content

Commit

Permalink
Fix activejob integration for Sidekiq 6
Browse files Browse the repository at this point in the history
Sidekiq::MAJOR was added in sidekiq/sidekiq@b4092e3, which is only
included in 7.0.0+ so we can't use it to check the version for
Sidekiq 6.

Since the test code is written to support both, the condition should
also support both.
  • Loading branch information
skipkayhil committed Jan 7, 2023
1 parent 7f2ece8 commit 862dc5b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion activejob/test/support/integration/adapters/sidekiq.rb
Expand Up @@ -54,7 +54,7 @@ def start_workers

require "sidekiq/cli"
require "sidekiq/launcher"
if Sidekiq::MAJOR >= 7
if Gem::Version.new(Sidekiq::VERSION) >= Gem::Version.new("7")
config = Sidekiq.default_configuration
config.queues = ["integration_tests"]
config.concurrency = 1
Expand Down

0 comments on commit 862dc5b

Please sign in to comment.