summaryrefslogtreecommitdiff
path: root/spec/services/user_project_access_changed_service_spec.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/services/user_project_access_changed_service_spec.rb')
-rw-r--r--spec/services/user_project_access_changed_service_spec.rb35
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