diff options
author | Douglas Barbosa Alexandre <dbalexandre@gmail.com> | 2016-07-26 15:27:42 -0300 |
---|---|---|
committer | Douglas Barbosa Alexandre <dbalexandre@gmail.com> | 2016-07-26 16:31:55 -0300 |
commit | 9865b8e6bdfa33be8206b620670bce2dd6c9bb51 (patch) | |
tree | 86b645c9f036cf57770f47d3cf9680411d2ae3fa /spec/services/projects/update_service_spec.rb | |
parent | e20a92dc5f6d2d066092863d1e97a6c0613183ad (diff) | |
download | gitlab-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.rb | 21 |
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 |