diff options
author | manojmj <mmj@gitlab.com> | 2019-09-23 13:12:26 +0530 |
---|---|---|
committer | manojmj <mmj@gitlab.com> | 2019-10-23 14:56:24 +0530 |
commit | f1320e4f938a5f508628b21a012db1617eab8946 (patch) | |
tree | bd3f3b52e0c9f7a7f1710b165e002a162f8cb405 /spec/services | |
parent | 1425a56c75beecaa289ad59587d636f8f469509e (diff) | |
download | gitlab-ce-f1320e4f938a5f508628b21a012db1617eab8946.tar.gz |
Require maintainer permission to transfer projects
Diffstat (limited to 'spec/services')
-rw-r--r-- | spec/services/projects/transfer_service_spec.rb | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/spec/services/projects/transfer_service_spec.rb b/spec/services/projects/transfer_service_spec.rb index 26d8ac9b479..298867f483b 100644 --- a/spec/services/projects/transfer_service_spec.rb +++ b/spec/services/projects/transfer_service_spec.rb @@ -222,6 +222,24 @@ describe Projects::TransferService do it { expect(project.errors[:new_namespace]).to include('Project with same name or path in target namespace already exists') } end + context 'target namespace allows developers to create projects' do + let(:group) { create(:group, project_creation_level: ::Gitlab::Access::DEVELOPER_MAINTAINER_PROJECT_ACCESS) } + + context 'the user is a member of the target namespace with developer permissions' do + subject(:transfer_project_result) { transfer_project(project, user, group) } + + before do + group.add_developer(user) + end + + it 'does not allow project transfer to the target namespace' do + expect(transfer_project_result).to eq false + expect(project.namespace).to eq(user.namespace) + expect(project.errors[:new_namespace]).to include('Transfer failed, please contact an admin.') + end + end + end + def transfer_project(project, user, new_namespace) service = Projects::TransferService.new(project, user) |