diff options
Diffstat (limited to 'spec/services')
-rw-r--r-- | spec/services/members/invitation_reminder_email_service_spec.rb | 4 | ||||
-rw-r--r-- | spec/services/notification_service_spec.rb | 14 | ||||
-rw-r--r-- | spec/services/users/reject_service_spec.rb | 54 |
3 files changed, 70 insertions, 2 deletions
diff --git a/spec/services/members/invitation_reminder_email_service_spec.rb b/spec/services/members/invitation_reminder_email_service_spec.rb index 88280869476..7cb30662152 100644 --- a/spec/services/members/invitation_reminder_email_service_spec.rb +++ b/spec/services/members/invitation_reminder_email_service_spec.rb @@ -11,7 +11,7 @@ RSpec.describe Members::InvitationReminderEmailService do context 'when the experiment is disabled' do before do - allow(Gitlab::Experimentation).to receive(:enabled_for_attribute?).and_return(false) + allow(Gitlab::Experimentation).to receive(:in_experiment_group?).and_return(false) invitation.expires_at = frozen_time + 2.days end @@ -26,7 +26,7 @@ RSpec.describe Members::InvitationReminderEmailService do context 'when the experiment is enabled' do before do - allow(Gitlab::Experimentation).to receive(:enabled_for_attribute?).and_return(true) + allow(Gitlab::Experimentation).to receive(:in_experiment_group?).and_return(true) invitation.expires_at = frozen_time + expires_at_days.days if expires_at_days end diff --git a/spec/services/notification_service_spec.rb b/spec/services/notification_service_spec.rb index 34a95c0505d..28577821231 100644 --- a/spec/services/notification_service_spec.rb +++ b/spec/services/notification_service_spec.rb @@ -2326,6 +2326,20 @@ RSpec.describe NotificationService, :mailer do end end + describe '#user_admin_rejection', :deliver_mails_inline do + let_it_be(:user) { create(:user, :blocked_pending_approval) } + + before do + reset_delivered_emails! + end + + it 'sends the user a rejection email' do + notification.user_admin_rejection(user.name, user.email) + + should_only_email(user) + end + end + describe 'GroupMember', :deliver_mails_inline do let(:added_user) { create(:user) } diff --git a/spec/services/users/reject_service_spec.rb b/spec/services/users/reject_service_spec.rb new file mode 100644 index 00000000000..07863d1a290 --- /dev/null +++ b/spec/services/users/reject_service_spec.rb @@ -0,0 +1,54 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe Users::RejectService do + let_it_be(:current_user) { create(:admin) } + let(:user) { create(:user, :blocked_pending_approval) } + + subject(:execute) { described_class.new(current_user).execute(user) } + + describe '#execute' do + context 'failures' do + context 'when the executor user is not allowed to reject users' do + let(:current_user) { create(:user) } + + it 'returns error result' do + expect(subject[:status]).to eq(:error) + expect(subject[:message]).to match(/You are not allowed to reject a user/) + end + end + + context 'when the executor user is an admin in admin mode', :enable_admin_mode do + context 'when user is not in pending approval state' do + let(:user) { create(:user, state: 'active') } + + it 'returns error result' do + expect(subject[:status]).to eq(:error) + expect(subject[:message]) + .to match(/This user does not have a pending request/) + end + end + end + end + + context 'success' do + context 'when the executor user is an admin in admin mode', :enable_admin_mode do + it 'deletes the user', :sidekiq_inline do + subject + + expect(subject[:status]).to eq(:success) + expect { User.find(user.id) }.to raise_error(ActiveRecord::RecordNotFound) + end + + it 'emails the user on rejection' do + expect_next_instance_of(NotificationService) do |notification| + allow(notification).to receive(:user_admin_rejection).with(user.name, user.notification_email) + end + + subject + end + end + end + end +end |