diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-03-26 09:07:52 +0000 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-03-26 09:07:52 +0000 |
commit | 7e019504f5ac6decde690565857238e7e59aa034 (patch) | |
tree | fab8832b40e25fc9bc1ae54b9303b95ea146b5d5 /spec/services/members | |
parent | 116d4e56e83a1f408afe710ce070e699ba206475 (diff) | |
download | gitlab-ce-7e019504f5ac6decde690565857238e7e59aa034.tar.gz |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec/services/members')
-rw-r--r-- | spec/services/members/update_highest_role_service_spec.rb | 42 |
1 files changed, 42 insertions, 0 deletions
diff --git a/spec/services/members/update_highest_role_service_spec.rb b/spec/services/members/update_highest_role_service_spec.rb new file mode 100644 index 00000000000..b56a51f83f9 --- /dev/null +++ b/spec/services/members/update_highest_role_service_spec.rb @@ -0,0 +1,42 @@ +# frozen_string_literal: true + +require 'spec_helper' +require 'sidekiq/testing' + +describe Members::UpdateHighestRoleService, :clean_gitlab_redis_shared_state do + include ExclusiveLeaseHelpers + + let_it_be(:user) { create(:user) } + let_it_be(:lease_key) { "update_highest_role:#{user.id}" } + let(:service) { described_class.new(user.id) } + + describe '#perform' do + subject { service.execute } + + context 'when lease is obtained' do + it 'takes the lease but does not release it', :aggregate_failures do + expect_to_obtain_exclusive_lease(lease_key, 'uuid', timeout: described_class::LEASE_TIMEOUT) + + subject + + expect(service.exclusive_lease.exists?).to be_truthy + end + + it 'schedules a job' do + Sidekiq::Testing.fake! do + expect { subject }.to change(UpdateHighestRoleWorker.jobs, :size).by(1) + end + end + end + + context 'when lease cannot be obtained' do + it 'only schedules one job' do + Sidekiq::Testing.fake! do + stub_exclusive_lease_taken(lease_key, timeout: described_class::LEASE_TIMEOUT) + + expect { subject }.not_to change(UpdateHighestRoleWorker.jobs, :size) + end + end + end + end +end |