diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2023-12-05 18:10:08 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2023-12-05 18:10:08 +0300 |
commit | 6659634b2bdc3ba362574541985c6852ad1574a4 (patch) | |
tree | 495e2462917ef620d379d0b7296d44e6c4b84bbc /spec/workers | |
parent | b35f7ce1f3f12bf7b673c9d29002e14d0c83f35f (diff) |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec/workers')
-rw-r--r-- | spec/workers/users/deactivate_dormant_users_worker_spec.rb | 91 |
1 files changed, 61 insertions, 30 deletions
diff --git a/spec/workers/users/deactivate_dormant_users_worker_spec.rb b/spec/workers/users/deactivate_dormant_users_worker_spec.rb index c28be165fd7..574dc922a36 100644 --- a/spec/workers/users/deactivate_dormant_users_worker_spec.rb +++ b/spec/workers/users/deactivate_dormant_users_worker_spec.rb @@ -10,34 +10,27 @@ RSpec.describe Users::DeactivateDormantUsersWorker, feature_category: :seat_cost let_it_be(:inactive) { create(:user, last_activity_on: nil, created_at: User::MINIMUM_DAYS_CREATED.days.ago.to_date) } let_it_be(:inactive_recently_created) { create(:user, last_activity_on: nil, created_at: (User::MINIMUM_DAYS_CREATED - 1).days.ago.to_date) } - let(:admin_bot) { create(:user, :admin_bot) } - let(:deactivation_service) { instance_spy(Users::DeactivateService) } - - before do - allow(Users::DeactivateService).to receive(:new).and_return(deactivation_service) - end - subject(:worker) { described_class.new } it 'does not run for SaaS', :saas do - # Now makes a call to current settings to determine period of dormancy - worker.perform - expect(deactivation_service).not_to have_received(:execute) - end - - context 'when automatic deactivation of dormant users is enabled' do - before do - stub_application_setting(deactivate_dormant_users: true) + expect_any_instance_of(::Users::DeactivateService) do |deactivation_service| + expect(deactivation_service).not_to have_received(:execute) end + end - it 'deactivates dormant users' do - worker.perform - - expect(deactivation_service).to have_received(:execute).twice + shared_examples 'deactivates dormant users' do + specify do + expect { worker.perform } + .to change { dormant.reload.state } + .to('deactivated') + .and change { inactive.reload.state } + .to('deactivated') end + end + shared_examples 'deactivates certain user types' do where(:user_type, :expected_state) do :human | 'deactivated' :support_bot | 'active' @@ -52,33 +45,69 @@ RSpec.describe Users::DeactivateDormantUsersWorker, feature_category: :seat_cost end with_them do - it 'deactivates certain user types' do + specify do user = create(:user, user_type: user_type, state: :active, last_activity_on: Gitlab::CurrentSettings.deactivate_dormant_users_period.days.ago.to_date) worker.perform - if expected_state == 'deactivated' - expect(deactivation_service).to have_received(:execute).with(user) - else - expect(deactivation_service).not_to have_received(:execute).with(user) + expect_any_instance_of(::Users::DeactivateService) do |deactivation_service| + if expected_state == 'deactivated' + expect(deactivation_service).to receive(:execute).with(user).and_call_original + else + expect(deactivation_service).not_to have_received(:execute).with(user) + end end + + expect(user.reload.state).to eq expected_state end end + end - it 'does not deactivate non-active users' do + shared_examples 'does not deactivate non-active users' do + specify do human_user = create(:user, user_type: :human, state: :blocked, last_activity_on: Gitlab::CurrentSettings.deactivate_dormant_users_period.days.ago.to_date) service_user = create(:user, user_type: :service_user, state: :blocked, last_activity_on: Gitlab::CurrentSettings.deactivate_dormant_users_period.days.ago.to_date) worker.perform - expect(deactivation_service).not_to have_received(:execute).with(human_user) - expect(deactivation_service).not_to have_received(:execute).with(service_user) + expect_any_instance_of(::Users::DeactivateService) do |deactivation_service| + expect(deactivation_service).not_to have_received(:execute).with(human_user) + expect(deactivation_service).not_to have_received(:execute).with(service_user) + end end + end - it 'does not deactivate recently created users' do + shared_examples 'does not deactivate recently created users' do + specify do worker.perform - expect(deactivation_service).not_to have_received(:execute).with(inactive_recently_created) + expect_any_instance_of(::Users::DeactivateService) do |deactivation_service| + expect(deactivation_service).not_to have_received(:execute).with(inactive_recently_created) + end + end + end + + context 'when automatic deactivation of dormant users is enabled' do + before do + stub_application_setting(deactivate_dormant_users: true) + end + + context 'when admin mode is not enabled', :do_not_mock_admin_mode_setting do + include_examples 'deactivates dormant users' + include_examples 'deactivates certain user types' + include_examples 'does not deactivate non-active users' + include_examples 'does not deactivate recently created users' + end + + context 'when admin mode is enabled', :request_store do + before do + stub_application_setting(admin_mode: true) + end + + include_examples 'deactivates dormant users' + include_examples 'deactivates certain user types' + include_examples 'does not deactivate non-active users' + include_examples 'does not deactivate recently created users' end end @@ -90,7 +119,9 @@ RSpec.describe Users::DeactivateDormantUsersWorker, feature_category: :seat_cost it 'does nothing' do worker.perform - expect(deactivation_service).not_to have_received(:execute) + expect_any_instance_of(::Users::DeactivateService) do |deactivation_service| + expect(deactivation_service).not_to have_received(:execute) + end end end end |