gitlab-org--gitlab-foss/spec/models/route_spec.rb

322 lines
11 KiB
Ruby
Raw Normal View History

require 'spec_helper'
describe Route do
2017-05-03 13:14:30 -04:00
let(:group) { create(:group, path: 'git_lab', name: 'git_lab') }
let(:route) { group.route }
describe 'relationships' do
it { is_expected.to belong_to(:source) }
end
describe 'validations' do
before do
expect(route).to be_persisted
end
it { is_expected.to validate_presence_of(:source) }
it { is_expected.to validate_presence_of(:path) }
it { is_expected.to validate_uniqueness_of(:path).case_insensitive }
end
2017-05-03 13:14:30 -04:00
describe 'callbacks' do
context 'after update' do
it 'calls #create_redirect_for_old_path' do
expect(route).to receive(:create_redirect_for_old_path)
route.update_attributes(path: 'foo')
end
it 'calls #delete_conflicting_redirects' do
expect(route).to receive(:delete_conflicting_redirects)
route.update_attributes(path: 'foo')
end
end
context 'after create' do
it 'calls #delete_conflicting_redirects' do
route.destroy
new_route = described_class.new(source: group, path: group.path)
2017-05-03 13:14:30 -04:00
expect(new_route).to receive(:delete_conflicting_redirects)
new_route.save!
end
end
end
2017-03-21 12:04:12 -04:00
describe '.inside_path' do
let!(:nested_group) { create(:group, path: 'test', name: 'test', parent: group) }
let!(:deep_nested_group) { create(:group, path: 'foo', name: 'foo', parent: nested_group) }
let!(:another_group) { create(:group, path: 'other') }
let!(:similar_group) { create(:group, path: 'gitllab') }
let!(:another_group_nested) { create(:group, path: 'another', name: 'another', parent: similar_group) }
it 'returns correct routes' do
expect(described_class.inside_path('git_lab')).to match_array([nested_group.route, deep_nested_group.route])
2017-03-21 12:04:12 -04:00
end
end
describe '#rename_descendants' do
let!(:nested_group) { create(:group, path: 'test', name: 'test', parent: group) }
let!(:deep_nested_group) { create(:group, path: 'foo', name: 'foo', parent: nested_group) }
let!(:similar_group) { create(:group, path: 'gitlab-org', name: 'gitlab-org') }
2017-03-21 12:04:12 -04:00
let!(:another_group) { create(:group, path: 'gittlab', name: 'gitllab') }
let!(:another_group_nested) { create(:group, path: 'git_lab', name: 'git_lab', parent: another_group) }
context 'path update' do
context 'when route name is set' do
before do
route.update_attributes(path: 'bar')
end
2017-05-03 13:14:30 -04:00
it 'updates children routes with new path' do
expect(described_class.exists?(path: 'bar')).to be_truthy
expect(described_class.exists?(path: 'bar/test')).to be_truthy
expect(described_class.exists?(path: 'bar/test/foo')).to be_truthy
expect(described_class.exists?(path: 'gitlab-org')).to be_truthy
2017-03-21 12:04:12 -04:00
expect(described_class.exists?(path: 'gittlab')).to be_truthy
expect(described_class.exists?(path: 'gittlab/git_lab')).to be_truthy
end
end
context 'when route name is nil' do
before do
route.update_column(:name, nil)
end
it "does not fail" do
expect(route.update_attributes(path: 'bar')).to be_truthy
end
end
2017-05-03 13:14:30 -04:00
context 'when conflicting redirects exist' do
let(:route) { create(:project).route }
2017-05-03 13:14:30 -04:00
let!(:conflicting_redirect1) { route.create_redirect('bar/test') }
let!(:conflicting_redirect2) { route.create_redirect('bar/test/foo') }
let!(:conflicting_redirect3) { route.create_redirect('gitlab-org') }
it 'deletes the conflicting redirects' do
route.update_attributes(path: 'bar')
expect(RedirectRoute.exists?(path: 'bar/test')).to be_falsey
expect(RedirectRoute.exists?(path: 'bar/test/foo')).to be_falsey
expect(RedirectRoute.exists?(path: 'gitlab-org')).to be_truthy
end
end
end
context 'name update' do
2017-05-03 13:14:30 -04:00
it 'updates children routes with new path' do
2017-03-21 12:04:12 -04:00
route.update_attributes(name: 'bar')
expect(described_class.exists?(name: 'bar')).to be_truthy
expect(described_class.exists?(name: 'bar / test')).to be_truthy
expect(described_class.exists?(name: 'bar / test / foo')).to be_truthy
expect(described_class.exists?(name: 'gitlab-org')).to be_truthy
end
it 'handles a rename from nil' do
# Note: using `update_columns` to skip all validation and callbacks
route.update_columns(name: nil)
expect { route.update_attributes(name: 'bar') }
.to change { route.name }.from(nil).to('bar')
end
end
end
2017-05-01 19:48:05 -04:00
2017-05-03 13:14:30 -04:00
describe '#create_redirect_for_old_path' do
2017-05-01 19:48:05 -04:00
context 'if the path changed' do
it 'creates a RedirectRoute for the old path' do
2017-05-03 13:14:30 -04:00
redirect_scope = route.source.redirect_routes.where(path: 'git_lab')
expect(redirect_scope.exists?).to be_falsey
2017-05-01 19:48:05 -04:00
route.path = 'new-path'
route.save!
2017-05-03 13:14:30 -04:00
expect(redirect_scope.exists?).to be_truthy
2017-05-01 19:48:05 -04:00
end
end
2017-05-03 13:14:30 -04:00
end
2017-05-01 19:48:05 -04:00
2017-05-03 13:14:30 -04:00
describe '#create_redirect' do
it 'creates a RedirectRoute with the same source' do
redirect_route = route.create_redirect('foo')
expect(redirect_route).to be_a(RedirectRoute)
expect(redirect_route).to be_persisted
expect(redirect_route.source).to eq(route.source)
expect(redirect_route.path).to eq('foo')
end
context 'when the source is a Project' do
it 'creates a temporal RedirectRoute' do
project = create(:project)
route = project.route
redirect_route = route.create_redirect('foo')
expect(redirect_route.permanent?).to be_falsy
end
end
context 'when the source is not a project' do
it 'creates a permanent RedirectRoute' do
redirect_route = route.create_redirect('foo', permanent: true)
expect(redirect_route.permanent?).to be_truthy
end
end
2017-05-03 13:14:30 -04:00
end
describe '#delete_conflicting_redirects' do
context 'with permanent redirect' do
it 'does not delete the redirect' do
route.create_redirect("#{route.path}/foo", permanent: true)
expect do
route.delete_conflicting_redirects
end.not_to change { RedirectRoute.count }
end
end
context 'with temporal redirect' do
let(:route) { create(:project).route }
it 'deletes the redirect' do
route.create_redirect("#{route.path}/foo")
expect do
route.delete_conflicting_redirects
end.to change { RedirectRoute.count }.by(-1)
end
end
2017-05-03 13:14:30 -04:00
context 'when a redirect route with the same path exists' do
2017-08-07 19:41:39 -04:00
context 'when the redirect route has matching case' do
let(:route) { create(:project).route }
2017-08-07 19:41:39 -04:00
let!(:redirect1) { route.create_redirect(route.path) }
2017-05-03 13:14:30 -04:00
2017-08-07 19:41:39 -04:00
it 'deletes the redirect' do
expect do
route.delete_conflicting_redirects
2017-08-09 11:11:08 -04:00
end.to change { RedirectRoute.count }.by(-1)
2017-08-07 19:41:39 -04:00
end
context 'when redirect routes with paths descending from the route path exists' do
let!(:redirect2) { route.create_redirect("#{route.path}/foo") }
let!(:redirect3) { route.create_redirect("#{route.path}/foo/bar") }
let!(:redirect4) { route.create_redirect("#{route.path}/baz/quz") }
let!(:other_redirect) { route.create_redirect("other") }
it 'deletes all redirects with paths that descend from the route path' do
expect do
route.delete_conflicting_redirects
2017-08-09 11:11:08 -04:00
end.to change { RedirectRoute.count }.by(-4)
2017-08-07 19:41:39 -04:00
end
end
2017-05-03 13:14:30 -04:00
end
2017-08-07 19:41:39 -04:00
context 'when the redirect route is differently cased' do
let(:route) { create(:project).route }
2017-08-07 19:41:39 -04:00
let!(:redirect1) { route.create_redirect(route.path.upcase) }
2017-05-03 13:14:30 -04:00
2017-08-07 19:41:39 -04:00
it 'deletes the redirect' do
expect do
route.delete_conflicting_redirects
2017-08-09 11:11:08 -04:00
end.to change { RedirectRoute.count }.by(-1)
2017-05-03 13:14:30 -04:00
end
end
end
end
describe '#conflicting_redirects' do
2017-08-07 19:41:39 -04:00
it 'returns an ActiveRecord::Relation' do
expect(route.conflicting_redirects).to be_an(ActiveRecord::Relation)
end
context 'with permanent redirects' do
it 'does not return anything' do
route.create_redirect("#{route.path}/foo", permanent: true)
route.create_redirect("#{route.path}/foo/bar", permanent: true)
route.create_redirect("#{route.path}/baz/quz", permanent: true)
expect(route.conflicting_redirects).to be_empty
end
end
context 'with temporal redirects' do
let(:route) { create(:project).route }
it 'returns the redirect routes' do
route = create(:project).route
redirect1 = route.create_redirect("#{route.path}/foo")
redirect2 = route.create_redirect("#{route.path}/foo/bar")
redirect3 = route.create_redirect("#{route.path}/baz/quz")
expect(route.conflicting_redirects).to match_array([redirect1, redirect2, redirect3])
end
end
2017-05-03 13:14:30 -04:00
context 'when a redirect route with the same path exists' do
let(:route) { create(:project).route }
2017-08-07 19:41:39 -04:00
context 'when the redirect route has matching case' do
let!(:redirect1) { route.create_redirect(route.path) }
2017-05-03 13:14:30 -04:00
2017-08-07 19:41:39 -04:00
it 'returns the redirect route' do
expect(route.conflicting_redirects).to match_array([redirect1])
end
context 'when redirect routes with paths descending from the route path exists' do
let!(:redirect2) { route.create_redirect("#{route.path}/foo") }
let!(:redirect3) { route.create_redirect("#{route.path}/foo/bar") }
let!(:redirect4) { route.create_redirect("#{route.path}/baz/quz") }
let!(:other_redirect) { route.create_redirect("other") }
it 'returns the redirect routes' do
expect(route.conflicting_redirects).to match_array([redirect1, redirect2, redirect3, redirect4])
end
end
2017-05-03 13:14:30 -04:00
end
2017-08-07 19:41:39 -04:00
context 'when the redirect route is differently cased' do
let!(:redirect1) { route.create_redirect(route.path.upcase) }
2017-05-03 13:14:30 -04:00
2017-08-07 19:41:39 -04:00
it 'returns the redirect route' do
expect(route.conflicting_redirects).to match_array([redirect1])
2017-05-03 13:14:30 -04:00
end
2017-05-01 19:48:05 -04:00
end
end
end
describe "#conflicting_redirect_exists?" do
context 'when a conflicting redirect exists' do
let(:group1) { create(:group, path: 'foo') }
let(:group2) { create(:group, path: 'baz') }
it 'should not be saved' do
group1.path = 'bar'
group1.save
group2.path = 'foo'
expect(group2.save).to be_falsy
end
it 'should return an error on path' do
group1.path = 'bar'
group1.save
group2.path = 'foo'
group2.valid?
expect(group2.errors["route.path"].first).to eq('foo has been taken before. Please use another one')
end
end
context 'when a conflicting redirect does not exist' do
let(:project1) { create(:project, path: 'foo') }
let(:project2) { create(:project, path: 'baz') }
it 'should be saved' do
project1.path = 'bar'
project1.save
project2.path = 'foo'
expect(project2.save).to be_truthy
end
end
end
end