# frozen_string_literal: true require 'spec_helper' RSpec.describe ProjectMember do describe 'associations' do it { is_expected.to belong_to(:project).with_foreign_key(:source_id) } end describe 'validations' do it { is_expected.to allow_value('Project').for(:source_type) } it { is_expected.not_to allow_value('project').for(:source_type) } it { is_expected.to validate_inclusion_of(:access_level).in_array(Gitlab::Access.values) } end describe 'delegations' do it { is_expected.to delegate_method(:namespace_id).to(:project) } end describe '.access_level_roles' do it 'returns Gitlab::Access.options' do expect(described_class.access_level_roles).to eq(Gitlab::Access.options) end end describe '#real_source_type' do subject { create(:project_member).real_source_type } it { is_expected.to eq 'Project' } end describe "#destroy" do let(:owner) { create(:project_member, access_level: ProjectMember::MAINTAINER) } let(:project) { owner.project } let(:maintainer) { create(:project_member, project: project) } it "creates an expired event when left due to expiry" do expired = create(:project_member, project: project, expires_at: 1.day.from_now) travel_to(2.days.from_now) { expired.destroy! } expect(Event.recent.first).to be_expired_action end it "creates a left event when left due to leave" do maintainer.destroy! expect(Event.recent.first).to be_left_action end context 'for an orphaned member' do let!(:orphaned_project_member) do owner.tap { |member| member.update_column(:user_id, nil) } end it 'does not raise an error' do expect { orphaned_project_member.destroy! }.not_to raise_error end end end describe '.import_team' do before do @project_1 = create(:project) @project_2 = create(:project) @user_1 = create :user @user_2 = create :user @project_1.add_developer(@user_1) @project_2.add_reporter(@user_2) @status = @project_2.team.import(@project_1) end it { expect(@status).to be_truthy } describe 'project 2 should get user 1 as developer. user_2 should not be changed' do it { expect(@project_2.users).to include(@user_1) } it { expect(@project_2.users).to include(@user_2) } it { expect(Ability.allowed?(@user_1, :create_project, @project_2)).to be_truthy } it { expect(Ability.allowed?(@user_2, :read_project, @project_2)).to be_truthy } end describe 'project 1 should not be changed' do it { expect(@project_1.users).to include(@user_1) } it { expect(@project_1.users).not_to include(@user_2) } end end describe '.add_users_to_projects' do it 'adds the given users to the given projects' do projects = create_list(:project, 2) users = create_list(:user, 2) described_class.add_users_to_projects( [projects.first.id, projects.second.id], [users.first.id, users.second], described_class::MAINTAINER) expect(projects.first.users).to include(users.first) expect(projects.first.users).to include(users.second) expect(projects.second.users).to include(users.first) expect(projects.second.users).to include(users.second) end end describe '.truncate_teams' do before do @project_1 = create(:project) @project_2 = create(:project) @user_1 = create :user @user_2 = create :user @project_1.add_developer(@user_1) @project_2.add_reporter(@user_2) described_class.truncate_teams([@project_1.id, @project_2.id]) end it { expect(@project_1.users).to be_empty } it { expect(@project_2.users).to be_empty } end it_behaves_like 'members notifications', :project context 'access levels' do context 'with parent group' do it_behaves_like 'inherited access level as a member of entity' do let(:entity) { create(:project, group: parent_entity) } end end context 'with parent group and a subgroup' do it_behaves_like 'inherited access level as a member of entity' do let(:subgroup) { create(:group, parent: parent_entity) } let(:entity) { create(:project, group: subgroup) } end end end context 'refreshing project_authorizations' do let_it_be_with_refind(:project) { create(:project) } let_it_be_with_refind(:user) { create(:user) } let_it_be(:project_member) { create(:project_member, :guest, project: project, user: user) } context 'when the source project of the project member is destroyed' do it 'refreshes the authorization of user to the project in the group' do expect { project.destroy! }.to change { user.can?(:guest_access, project) }.from(true).to(false) end it 'refreshes the authorization without calling AuthorizedProjectUpdate::ProjectRecalculatePerUserService' do expect(AuthorizedProjectUpdate::ProjectRecalculatePerUserService).not_to receive(:new) project.destroy! end end context 'when the user of the project member is destroyed' do it 'refreshes the authorization of user to the project in the group' do expect(project.authorized_users).to include(user) user.destroy! expect(project.authorized_users).not_to include(user) end it 'refreshes the authorization without calling UserProjectAccessChangedService' do expect(UserProjectAccessChangedService).not_to receive(:new) user.destroy! end end context 'when importing' do it 'does not refresh' do expect(AuthorizedProjectUpdate::ProjectRecalculatePerUserService).not_to receive(:new) member = build(:project_member) member.importing = true member.save! end end end context 'authorization refresh on addition/updation/deletion' do let_it_be(:project) { create(:project) } let_it_be(:user) { create(:user) } shared_examples_for 'calls AuthorizedProjectUpdate::ProjectRecalculatePerUserService to recalculate authorizations' do it 'calls AuthorizedProjectUpdate::ProjectRecalculatePerUserService' do expect_next_instance_of(AuthorizedProjectUpdate::ProjectRecalculatePerUserService, project, user) do |service| expect(service).to receive(:execute) end action end end shared_examples_for 'calls AuthorizedProjectUpdate::UserRefreshFromReplicaWorker with a delay to update project authorizations' do it 'calls AuthorizedProjectUpdate::UserRefreshFromReplicaWorker' do expect(AuthorizedProjectUpdate::UserRefreshFromReplicaWorker).to( receive(:bulk_perform_in) .with(1.hour, [[user.id]], batch_delay: 30.seconds, batch_size: 100) ) action end end context 'on create' do let(:action) { project.add_user(user, Gitlab::Access::GUEST) } it 'changes access level' do expect { action }.to change { user.can?(:guest_access, project) }.from(false).to(true) end it_behaves_like 'calls AuthorizedProjectUpdate::ProjectRecalculatePerUserService to recalculate authorizations' it_behaves_like 'calls AuthorizedProjectUpdate::UserRefreshFromReplicaWorker with a delay to update project authorizations' end context 'on update' do let(:action) { project.members.find_by(user: user).update!(access_level: Gitlab::Access::DEVELOPER) } before do project.add_user(user, Gitlab::Access::GUEST) end it 'changes access level' do expect { action }.to change { user.can?(:developer_access, project) }.from(false).to(true) end it_behaves_like 'calls AuthorizedProjectUpdate::ProjectRecalculatePerUserService to recalculate authorizations' it_behaves_like 'calls AuthorizedProjectUpdate::UserRefreshFromReplicaWorker with a delay to update project authorizations' end context 'on destroy' do let(:action) { project.members.find_by(user: user).destroy! } before do project.add_user(user, Gitlab::Access::GUEST) end it 'changes access level', :sidekiq_inline do expect { action }.to change { user.can?(:guest_access, project) }.from(true).to(false) end it 'calls AuthorizedProjectUpdate::ProjectRecalculatePerUserWorker to recalculate authorizations' do expect(AuthorizedProjectUpdate::ProjectRecalculatePerUserWorker).to receive(:perform_async).with(project.id, user.id) action end it_behaves_like 'calls AuthorizedProjectUpdate::UserRefreshFromReplicaWorker with a delay to update project authorizations' end end describe '#set_member_namespace_id' do let(:project) { create(:project) } let(:member) { create(:project_member, project: project) } context 'on create' do it 'sets the member_namespace_id' do expect(member.member_namespace_id).to eq project.project_namespace_id end end end end