diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2023-02-07 15:10:09 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2023-02-07 15:10:09 +0300 |
commit | 75f809a2ff829574ab91628407993187d55e14a4 (patch) | |
tree | e781d4cdc7bc7785e2d55e5186ccda1992763c93 /spec/services/user_project_access_changed_service_spec.rb | |
parent | 9f99bf3b0e3516471806f3595e1619b8e9af2a80 (diff) |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec/services/user_project_access_changed_service_spec.rb')
-rw-r--r-- | spec/services/user_project_access_changed_service_spec.rb | 34 |
1 files changed, 24 insertions, 10 deletions
diff --git a/spec/services/user_project_access_changed_service_spec.rb b/spec/services/user_project_access_changed_service_spec.rb index 9aa7dcf09bc..356675d55f2 100644 --- a/spec/services/user_project_access_changed_service_spec.rb +++ b/spec/services/user_project_access_changed_service_spec.rb @@ -2,7 +2,7 @@ require 'spec_helper' -RSpec.describe UserProjectAccessChangedService do +RSpec.describe UserProjectAccessChangedService, feature_category: :authentication_and_authorization do describe '#execute' do it 'permits high-priority operation' do expect(AuthorizedProjectsWorker).to receive(:bulk_perform_async) @@ -11,16 +11,30 @@ RSpec.describe UserProjectAccessChangedService do described_class.new([1, 2]).execute end - it 'permits low-priority operation' do - expect(AuthorizedProjectUpdate::UserRefreshFromReplicaWorker).to( - receive(:bulk_perform_in).with( - described_class::DELAY, - [[1], [2]], - { batch_delay: 30.seconds, batch_size: 100 } - ) - ) + context 'for low priority operation' do + context 'when the feature flag `do_not_run_safety_net_auth_refresh_jobs` is disabled' do + before do + stub_feature_flags(do_not_run_safety_net_auth_refresh_jobs: false) + end + + it 'permits low-priority operation' do + expect(AuthorizedProjectUpdate::UserRefreshFromReplicaWorker).to( + receive(:bulk_perform_in).with( + described_class::DELAY, + [[1], [2]], + { batch_delay: 30.seconds, batch_size: 100 } + ) + ) + + described_class.new([1, 2]).execute(priority: described_class::LOW_PRIORITY) + end + end + + it 'does not perform low-priority operation' do + expect(AuthorizedProjectUpdate::UserRefreshFromReplicaWorker).not_to receive(:bulk_perform_in) - described_class.new([1, 2]).execute(priority: described_class::LOW_PRIORITY) + described_class.new([1, 2]).execute(priority: described_class::LOW_PRIORITY) + end end it 'permits medium-priority operation' do |