summaryrefslogtreecommitdiff
path: root/spec/services/projects/update_service_spec.rb
diff options
context:
space:
mode:
authorDouglas Barbosa Alexandre <dbalexandre@gmail.com>2016-07-26 15:27:42 -0300
committerDouglas Barbosa Alexandre <dbalexandre@gmail.com>2016-07-26 16:31:55 -0300
commit9865b8e6bdfa33be8206b620670bce2dd6c9bb51 (patch)
tree86b645c9f036cf57770f47d3cf9680411d2ae3fa /spec/services/projects/update_service_spec.rb
parente20a92dc5f6d2d066092863d1e97a6c0613183ad (diff)
downloadgitlab-ce-fix-19538.tar.gz
Fix Project#to_param to keep invalid project suitable for use in URLsfix-19538
Diffstat (limited to 'spec/services/projects/update_service_spec.rb')
-rw-r--r--spec/services/projects/update_service_spec.rb21
1 files changed, 0 insertions, 21 deletions
diff --git a/spec/services/projects/update_service_spec.rb b/spec/services/projects/update_service_spec.rb
index 6337daca9c7..e8b9e6b9238 100644
--- a/spec/services/projects/update_service_spec.rb
+++ b/spec/services/projects/update_service_spec.rb
@@ -139,27 +139,6 @@ describe Projects::UpdateService, services: true do
end
end
- context 'for invalid project path/name' do
- let(:user) { create(:user, admin: true) }
- let(:project) { create(:empty_project, path: 'gitlab', name: 'sample') }
- let(:params) { { path: 'foo&bar', name: 'foo&bar' } }
-
- it 'resets to previous values to keep project in a valid state' do
- update_project(project, user, params)
-
- expect(project.path).to eq 'gitlab'
- expect(project.name).to eq 'sample'
- end
-
- it 'keeps error messages' do
- update_project(project, user, params)
-
- expect(project.errors).not_to be_blank
- expect(project.errors[:name]).to include("can contain only letters, digits, '_', '.', dash and space. It must start with letter, digit or '_'.")
- expect(project.errors[:path]).to include("can contain only letters, digits, '_', '-' and '.'. Cannot start with '-', end in '.git' or end in '.atom'")
- end
- end
-
def update_project(project, user, opts)
Projects::UpdateService.new(project, user, opts).execute
end