Skip to content

Remove `Sidekiq::Testing.fake!`

Laurent Arnoud requested to merge (removed):fix-22992 into master

What does this MR do?

Fix https://gitlab.com/gitlab-org/gitlab-ce/issues/22992

Are there points in the code the reviewer needs to double check?

Its not possible to use perform_enqueued_jobs because the gitlab workers does not use ActiveJob API, gitlab sidekiq workers have to first migrate to ActiveJob, cf https://github.com/mperham/sidekiq/wiki/Active-Job

Why was this MR needed?

See https://gitlab.com/gitlab-org/gitlab-ce/issues/22992

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ce/issues/22992

Merge request reports

Loading