invites_controller_spec.rb 6.41 KB
Newer Older
1 2
# frozen_string_literal: true

3 4
require 'spec_helper'

5
RSpec.describe InvitesController do
6
  let_it_be(:user) { create(:user) }
7
  let_it_be(:member, reload: true) { create(:project_member, :invited, invite_email: user.email) }
8
  let(:raw_invite_token) { member.raw_invite_token }
9
  let(:project_members) { member.source.users }
10
  let(:md5_member_global_id) { Digest::MD5.hexdigest(member.to_global_id.to_s) }
11
  let(:params) { { id: raw_invite_token } }
12

13 14 15 16 17 18 19 20 21 22
  shared_examples 'invalid token' do
    context 'when invite token is not valid' do
      let(:params) { { id: '_bogus_token_' } }

      it 'renders the 404 page' do
        request

        expect(response).to have_gitlab_http_status(:not_found)
      end
    end
23 24
  end

25
  describe 'GET #show' do
26 27
    subject(:request) { get :show, params: params }

28 29 30 31
    context 'when logged in' do
      before do
        sign_in(user)
      end
32

33 34 35 36
      it 'accepts user if invite email matches signed in user' do
        expect do
          request
        end.to change { project_members.include?(user) }.from(false).to(true)
37

38 39 40
        expect(response).to have_gitlab_http_status(:found)
        expect(flash[:notice]).to include 'You have been granted'
      end
41

42
      it 'forces re-confirmation if email does not match signed in user' do
43
        member.update!(invite_email: 'bogus@email.com')
44

45 46 47
        expect do
          request
        end.not_to change { project_members.include?(user) }
48

49 50
        expect(response).to have_gitlab_http_status(:ok)
        expect(flash[:notice]).to be_nil
51 52
      end

53
      it_behaves_like 'invalid token'
54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75

      context 'when invite comes from the initial email invite' do
        let(:params) { { id: raw_invite_token, invite_type: Members::InviteEmailExperiment::INVITE_TYPE } }

        it 'tracks via experiment', :aggregate_failures do
          experiment = double(track: true)
          allow(controller).to receive(:experiment).and_return(experiment)

          request

          expect(experiment).to have_received(:track).with(:opened)
          expect(experiment).to have_received(:track).with(:accepted)
        end
      end

      context 'when invite does not come from initial email invite' do
        it 'does not track via experiment' do
          expect(controller).not_to receive(:experiment)

          request
        end
      end
76 77
    end

78 79
    context 'when not logged in' do
      context 'when inviter is a member' do
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 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152
        context 'when instance allows sign up' do
          it 'indicates an account can be created in notice' do
            request

            expect(flash[:notice]).to include('or create an account')
          end

          context 'when user exists with the invited email' do
            it 'is redirected to a new session with invite email param' do
              request

              expect(response).to redirect_to(new_user_session_path(invite_email: member.invite_email))
            end
          end

          context 'when user exists with the invited email as secondary email' do
            before do
              secondary_email = create(:email, user: user, email: 'foo@example.com')
              member.update!(invite_email: secondary_email.email)
            end

            it 'is redirected to a new session with invite email param' do
              request

              expect(response).to redirect_to(new_user_session_path(invite_email: member.invite_email))
            end
          end

          context 'when user does not exist with the invited email' do
            before do
              member.update!(invite_email: 'bogus_email@example.com')
            end

            it 'indicates an account can be created in notice' do
              request

              expect(flash[:notice]).to include('create an account or sign in')
            end

            it 'is redirected to a new registration with invite email param' do
              request

              expect(response).to redirect_to(new_user_registration_path(invite_email: member.invite_email))
            end
          end
        end

        context 'when instance does not allow sign up' do
          before do
            stub_application_setting(allow_signup?: false)
          end

          it 'does not indicate an account can be created in notice' do
            request

            expect(flash[:notice]).not_to include('or create an account')
          end

          context 'when user exists with the invited email' do
            it 'is redirected to a new session with invite email param' do
              request

              expect(response).to redirect_to(new_user_session_path(invite_email: member.invite_email))
            end
          end

          context 'when user does not exist with the invited email' do
            before do
              member.update!(invite_email: 'bogus_email@example.com')
            end

            it 'is redirected to a new session with invite email param' do
              request
153

154 155 156
              expect(response).to redirect_to(new_user_session_path(invite_email: member.invite_email))
            end
          end
157 158
        end
      end
159

160 161 162 163 164 165 166 167
      context 'when inviter is not a member' do
        let(:params) { { id: '_bogus_token_' } }

        it 'is redirected to a new session' do
          request

          expect(response).to redirect_to(new_user_session_path)
        end
168 169 170 171 172
      end
    end
  end

  describe 'POST #accept' do
173 174 175
    before do
      sign_in(user)
    end
176 177 178

    subject(:request) { post :accept, params: params }

179
    it_behaves_like 'invalid token'
180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198

    context 'when invite comes from the initial email invite' do
      it 'tracks via experiment' do
        experiment = double(track: true)
        allow(controller).to receive(:experiment).and_return(experiment)

        post :accept, params: params, session: { invite_type: Members::InviteEmailExperiment::INVITE_TYPE }

        expect(experiment).to have_received(:track).with(:accepted)
      end
    end

    context 'when invite does not come from initial email invite' do
      it 'does not track via experiment' do
        expect(controller).not_to receive(:experiment)

        request
      end
    end
199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218
  end

  describe 'POST #decline for link in UI' do
    before do
      sign_in(user)
    end

    subject(:request) { post :decline, params: params }

    it_behaves_like 'invalid token'
  end

  describe 'GET #decline for link in email' do
    before do
      sign_in(user)
    end

    subject(:request) { get :decline, params: params }

    it_behaves_like 'invalid token'
219
  end
220
end