From be44a258256ec9ba12876f56449078fcbb1431e0 Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Wed, 6 Dec 2023 16:03:46 +0000 Subject: Add latest changes from gitlab-org/gitlab@16-6-stable-ee --- .../users/deactivate_dormant_users_worker_spec.rb | 91 +++++++++++++++------- 1 file changed, 61 insertions(+), 30 deletions(-) (limited to 'spec') 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 -- cgit v1.2.3