summaryrefslogtreecommitdiff
path: root/spec/services/users/approve_service_spec.rb
blob: 50f2b6b0827f2a322ac6b68a6632011edd06cb22 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
# frozen_string_literal: true

require 'spec_helper'

RSpec.describe Users::ApproveService 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 approve 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 approve a user/)
        end
      end

      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(/The user you are trying to approve is not pending an approval/)
        end
      end

      context 'when user cannot be activated' do
        let(:user) do
          build(:user, state: 'blocked_pending_approval', email: 'invalid email')
        end

        it 'returns error result' do
          expect(subject[:status]).to eq(:error)
          expect(subject[:message]).to match(/Email is invalid/)
        end

        it 'does not change the state of the user' do
          expect { subject }.not_to change { user.state }
        end
      end
    end

    context 'success' do
      it 'activates the user' do
        expect(subject[:status]).to eq(:success)
        expect(user.reload).to be_active
      end

      context 'email confirmation status' do
        context 'user is unconfirmed' do
          let(:user) { create(:user, :blocked_pending_approval, :unconfirmed) }

          it 'sends confirmation instructions' do
            expect { subject }
              .to have_enqueued_mail(DeviseMailer, :confirmation_instructions)
          end
        end

        context 'user is confirmed' do
          it 'does not send a confirmation email' do
            expect { subject }
              .not_to have_enqueued_mail(DeviseMailer, :confirmation_instructions)
          end
        end
      end

      context 'pending invitiations' do
        let!(:project_member_invite) { create(:project_member, :invited, invite_email: user.email) }
        let!(:group_member_invite) { create(:group_member, :invited, invite_email: user.email) }

        context 'user is unconfirmed' do
          let(:user) { create(:user, :blocked_pending_approval, :unconfirmed) }

          it 'does not accept pending invites of the user' do
            expect(subject[:status]).to eq(:success)

            group_member_invite.reload
            project_member_invite.reload

            expect(group_member_invite).to be_invite
            expect(project_member_invite).to be_invite
          end
        end

        context 'user is confirmed' do
          it 'accepts pending invites of the user' do
            expect(subject[:status]).to eq(:success)

            group_member_invite.reload
            project_member_invite.reload

            expect(group_member_invite).not_to be_invite
            expect(project_member_invite).not_to be_invite
            expect(group_member_invite.user).to eq(user)
            expect(project_member_invite.user).to eq(user)
          end
        end
      end
    end
  end
end