summaryrefslogtreecommitdiff
path: root/spec/services
diff options
context:
space:
mode:
authormanojmj <mmj@gitlab.com>2019-09-23 13:12:26 +0530
committermanojmj <mmj@gitlab.com>2019-10-23 14:56:24 +0530
commitf1320e4f938a5f508628b21a012db1617eab8946 (patch)
treebd3f3b52e0c9f7a7f1710b165e002a162f8cb405 /spec/services
parent1425a56c75beecaa289ad59587d636f8f469509e (diff)
downloadgitlab-ce-f1320e4f938a5f508628b21a012db1617eab8946.tar.gz
Require maintainer permission to transfer projects
Diffstat (limited to 'spec/services')
-rw-r--r--spec/services/projects/transfer_service_spec.rb18
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)