Commit 58ac1179 authored by Arturo Herrero's avatar Arturo Herrero

Stop using argument in perform_async

After providing the default value to deprecate the Sidekiq argument
https://gitlab.com/gitlab-org/gitlab/-/merge_requests/40381

We can now stop using the argument in perform_async.
parent 4075754f
...@@ -20,7 +20,7 @@ module IntegrationsActions ...@@ -20,7 +20,7 @@ module IntegrationsActions
respond_to do |format| respond_to do |format|
format.html do format.html do
if saved if saved
PropagateIntegrationWorker.perform_async(integration.id, false) PropagateIntegrationWorker.perform_async(integration.id)
redirect_to scoped_edit_integration_path(integration), notice: success_message redirect_to scoped_edit_integration_path(integration), notice: success_message
else else
render 'shared/integrations/edit' render 'shared/integrations/edit'
......
...@@ -7,7 +7,7 @@ class PropagateIntegrationWorker ...@@ -7,7 +7,7 @@ class PropagateIntegrationWorker
idempotent! idempotent!
loggable_arguments 1 loggable_arguments 1
# Keep overwrite parameter for backwards compatibility. # TODO: Keep overwrite parameter for backwards compatibility. Remove after >= 14.0
def perform(integration_id, overwrite = nil) def perform(integration_id, overwrite = nil)
Admin::PropagateIntegrationService.propagate(Service.find(integration_id)) Admin::PropagateIntegrationService.propagate(Service.find(integration_id))
end end
......
...@@ -43,7 +43,7 @@ RSpec.describe Admin::IntegrationsController do ...@@ -43,7 +43,7 @@ RSpec.describe Admin::IntegrationsController do
end end
it 'calls to PropagateIntegrationWorker' do it 'calls to PropagateIntegrationWorker' do
expect(PropagateIntegrationWorker).to have_received(:perform_async).with(integration.id, false) expect(PropagateIntegrationWorker).to have_received(:perform_async).with(integration.id)
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment