diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2021-06-16 21:25:58 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2021-06-16 21:25:58 +0300 |
commit | a5f4bba440d7f9ea47046a0a561d49adf0a1e6d4 (patch) | |
tree | fb69158581673816a8cd895f9d352dcb3c678b1e /spec/services/user_project_access_changed_service_spec.rb | |
parent | d16b2e8639e99961de6ddc93909f3bb5c1445ba1 (diff) |
Add latest changes from gitlab-org/gitlab@14-0-stable-eev14.0.0-rc42
Diffstat (limited to 'spec/services/user_project_access_changed_service_spec.rb')
-rw-r--r-- | spec/services/user_project_access_changed_service_spec.rb | 35 |
1 files changed, 34 insertions, 1 deletions
diff --git a/spec/services/user_project_access_changed_service_spec.rb b/spec/services/user_project_access_changed_service_spec.rb index 070782992e7..4723619afd2 100644 --- a/spec/services/user_project_access_changed_service_spec.rb +++ b/spec/services/user_project_access_changed_service_spec.rb @@ -19,7 +19,7 @@ RSpec.describe UserProjectAccessChangedService do end it 'permits low-priority operation' do - expect(AuthorizedProjectUpdate::UserRefreshWithLowUrgencyWorker).to( + expect(AuthorizedProjectUpdate::UserRefreshFromReplicaWorker).to( receive(:bulk_perform_in).with( described_class::DELAY, [[1], [2]], @@ -31,4 +31,37 @@ RSpec.describe UserProjectAccessChangedService do priority: described_class::LOW_PRIORITY) end end + + context 'with load balancing enabled' do + let(:service) { UserProjectAccessChangedService.new([1, 2]) } + + before do + allow(Gitlab::Database::LoadBalancing).to receive(:enable?).and_return(true) + + expect(AuthorizedProjectsWorker).to receive(:bulk_perform_and_wait) + .with([[1], [2]]) + .and_return(10) + end + + it 'sticks all the updated users and returns the original result', :aggregate_failures do + expect(Gitlab::Database::LoadBalancing::Sticking).to receive(:bulk_stick).with(:user, [1, 2]) + + expect(service.execute).to eq(10) + end + + it 'avoids N+1 cached queries', :use_sql_query_cache, :request_store do + # Run this once to establish a baseline + control_count = ActiveRecord::QueryRecorder.new(skip_cached: false) do + service.execute + end + + service = UserProjectAccessChangedService.new([1, 2, 3, 4, 5]) + + allow(AuthorizedProjectsWorker).to receive(:bulk_perform_and_wait) + .with([[1], [2], [3], [4], [5]]) + .and_return(10) + + expect { service.execute }.not_to exceed_all_query_limit(control_count.count) + end + end end |