1 files changed, 119 insertions, 119 deletions

diff --git a/spec/models/route_spec.rb b/spec/models/route_spec.rb
index 48799781b87..9d7c3ca3c9f 100644
--- a/spec/models/route_spec.rb
+++ b/spec/models/route_spec.rb
@@ -1,14 +1,14 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Route do
-  let(:group) { create(:group, path: 'git_lab', name: 'git_lab') }
+  let(:group) { create(:group, path: "git_lab", name: "git_lab") }
   let(:route) { group.route }
 
-  describe 'relationships' do
+  describe "relationships" do
     it { is_expected.to belong_to(:source) }
   end
 
-  describe 'validations' do
+  describe "validations" do
     before do
       expect(route).to be_persisted
     end
@@ -18,28 +18,28 @@ describe Route do
     it { is_expected.to validate_uniqueness_of(:path).case_insensitive }
   end
 
-  describe 'callbacks' do
-    context 'before validation' do
-      it 'calls #delete_conflicting_orphaned_routes' do
+  describe "callbacks" do
+    context "before validation" do
+      it "calls #delete_conflicting_orphaned_routes" do
         expect(route).to receive(:delete_conflicting_orphaned_routes)
         route.valid?
       end
     end
 
-    context 'after update' do
-      it 'calls #create_redirect_for_old_path' do
+    context "after update" do
+      it "calls #create_redirect_for_old_path" do
         expect(route).to receive(:create_redirect_for_old_path)
-        route.update(path: 'foo')
+        route.update(path: "foo")
       end
 
-      it 'calls #delete_conflicting_redirects' do
+      it "calls #delete_conflicting_redirects" do
         expect(route).to receive(:delete_conflicting_redirects)
-        route.update(path: 'foo')
+        route.update(path: "foo")
       end
     end
 
-    context 'after create' do
-      it 'calls #delete_conflicting_redirects' do
+    context "after create" do
+      it "calls #delete_conflicting_redirects" do
         route.destroy
         new_route = described_class.new(source: group, path: group.path)
         expect(new_route).to receive(:delete_conflicting_redirects)
@@ -48,182 +48,182 @@ describe Route do
     end
   end
 
-  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) }
+  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])
+    it "returns correct routes" do
+      expect(described_class.inside_path("git_lab")).to match_array([nested_group.route, deep_nested_group.route])
     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') }
-    let!(:another_group) { create(:group, path: 'gittlab', name: 'gitllab') }
-    let!(:another_group_nested) { create(:group, path: 'git_lab', name: 'git_lab', parent: another_group) }
+  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") }
+    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
+    context "path update" do
+      context "when route name is set" do
         before do
-          route.update(path: 'bar')
+          route.update(path: "bar")
         end
 
-        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
-          expect(described_class.exists?(path: 'gittlab')).to be_truthy
-          expect(described_class.exists?(path: 'gittlab/git_lab')).to be_truthy
+        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
+          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
+      context "when route name is nil" do
         before do
           route.update_column(:name, nil)
         end
 
         it "does not fail" do
-          expect(route.update(path: 'bar')).to be_truthy
+          expect(route.update(path: "bar")).to be_truthy
         end
       end
 
-      context 'when conflicting redirects exist' do
+      context "when conflicting redirects exist" do
         let(:route) { create(:project).route }
-        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') }
+        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(path: 'bar')
+        it "deletes the conflicting redirects" do
+          route.update(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
+          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
-      it 'updates children routes with new path' do
-        route.update(name: 'bar')
+    context "name update" do
+      it "updates children routes with new path" do
+        route.update(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
+        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
+      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(name: 'bar') }
-          .to change { route.name }.from(nil).to('bar')
+        expect { route.update(name: "bar") }
+          .to change { route.name }.from(nil).to("bar")
       end
     end
   end
 
-  describe '#create_redirect_for_old_path' do
-    context 'if the path changed' do
-      it 'creates a RedirectRoute for the old path' do
-        redirect_scope = route.source.redirect_routes.where(path: 'git_lab')
+  describe "#create_redirect_for_old_path" do
+    context "if the path changed" do
+      it "creates a RedirectRoute for the old path" do
+        redirect_scope = route.source.redirect_routes.where(path: "git_lab")
         expect(redirect_scope.exists?).to be_falsey
-        route.path = 'new-path'
+        route.path = "new-path"
         route.save!
         expect(redirect_scope.exists?).to be_truthy
       end
     end
   end
 
-  describe '#create_redirect' do
-    it 'creates a RedirectRoute with the same source' do
-      redirect_route = route.create_redirect('foo')
+  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')
+      expect(redirect_route.path).to eq("foo")
     end
 
-    context 'when the source is a Project' do
-      it 'creates a RedirectRoute' do
+    context "when the source is a Project" do
+      it "creates a RedirectRoute" do
         project = create(:project)
         route = project.route
-        redirect_route = route.create_redirect('foo')
+        redirect_route = route.create_redirect("foo")
         expect(redirect_route).not_to be_nil
       end
     end
 
-    context 'when the source is not a project' do
-      it 'creates a RedirectRoute' do
-        redirect_route = route.create_redirect('foo')
+    context "when the source is not a project" do
+      it "creates a RedirectRoute" do
+        redirect_route = route.create_redirect("foo")
         expect(redirect_route).not_to be_nil
       end
     end
   end
 
-  describe '#delete_conflicting_redirects' do
+  describe "#delete_conflicting_redirects" do
     let(:route) { create(:project).route }
 
-    it 'deletes the redirect' do
+    it "deletes the redirect" do
       route.create_redirect("#{route.path}/foo")
 
-      expect do
+      expect {
         route.delete_conflicting_redirects
-      end.to change { RedirectRoute.count }.by(-1)
+      }.to change { RedirectRoute.count }.by(-1)
     end
 
-    context 'when a redirect route with the same path exists' do
-      context 'when the redirect route has matching case' do
+    context "when a redirect route with the same path exists" do
+      context "when the redirect route has matching case" do
         let(:route) { create(:project).route }
         let!(:redirect1) { route.create_redirect(route.path) }
 
-        it 'deletes the redirect' do
-          expect do
+        it "deletes the redirect" do
+          expect {
             route.delete_conflicting_redirects
-          end.to change { RedirectRoute.count }.by(-1)
+          }.to change { RedirectRoute.count }.by(-1)
         end
 
-        context 'when redirect routes with paths descending from the route path exists' do
+        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
+          it "deletes all redirects with paths that descend from the route path" do
+            expect {
               route.delete_conflicting_redirects
-            end.to change { RedirectRoute.count }.by(-4)
+            }.to change { RedirectRoute.count }.by(-4)
           end
         end
       end
 
-      context 'when the redirect route is differently cased' do
+      context "when the redirect route is differently cased" do
         let(:route) { create(:project).route }
         let!(:redirect1) { route.create_redirect(route.path.upcase) }
 
-        it 'deletes the redirect' do
-          expect do
+        it "deletes the redirect" do
+          expect {
             route.delete_conflicting_redirects
-          end.to change { RedirectRoute.count }.by(-1)
+          }.to change { RedirectRoute.count }.by(-1)
         end
       end
     end
   end
 
-  describe '#conflicting_redirects' do
+  describe "#conflicting_redirects" do
     let(:route) { create(:project).route }
 
-    it 'returns an ActiveRecord::Relation' do
+    it "returns an ActiveRecord::Relation" do
       expect(route.conflicting_redirects).to be_an(ActiveRecord::Relation)
     end
 
-    it 'returns the redirect routes' do
+    it "returns the redirect routes" do
       redirect1 = route.create_redirect("#{route.path}/foo")
       redirect2 = route.create_redirect("#{route.path}/foo/bar")
       redirect3 = route.create_redirect("#{route.path}/baz/quz")
@@ -231,87 +231,87 @@ describe Route do
       expect(route.conflicting_redirects).to match_array([redirect1, redirect2, redirect3])
     end
 
-    context 'when a redirect route with the same path exists' do
+    context "when a redirect route with the same path exists" do
       let(:route) { create(:project).route }
 
-      context 'when the redirect route has matching case' do
+      context "when the redirect route has matching case" do
         let!(:redirect1) { route.create_redirect(route.path) }
 
-        it 'returns the redirect route' do
+        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
+        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
+          it "returns the redirect routes" do
             expect(route.conflicting_redirects).to match_array([redirect1, redirect2, redirect3, redirect4])
           end
         end
       end
 
-      context 'when the redirect route is differently cased' do
+      context "when the redirect route is differently cased" do
         let!(:redirect1) { route.create_redirect(route.path.upcase) }
 
-        it 'returns the redirect route' do
+        it "returns the redirect route" do
           expect(route.conflicting_redirects).to match_array([redirect1])
         end
       end
     end
   end
 
-  describe '#delete_conflicting_orphaned_routes' do
-    context 'when there is a conflicting route' do
-      let!(:conflicting_group) { create(:group, path: 'foo') }
+  describe "#delete_conflicting_orphaned_routes" do
+    context "when there is a conflicting route" do
+      let!(:conflicting_group) { create(:group, path: "foo") }
 
       before do
         route.path = conflicting_group.route.path
       end
 
-      context 'when the route is orphaned' do
+      context "when the route is orphaned" do
         let!(:offending_route) { conflicting_group.route }
 
         before do
           Group.delete(conflicting_group) # Orphan the route
         end
 
-        it 'deletes the orphaned route' do
-          expect do
+        it "deletes the orphaned route" do
+          expect {
             route.valid?
-          end.to change { described_class.count }.from(2).to(1)
+          }.to change { described_class.count }.from(2).to(1)
         end
 
-        it 'passes validation, as usual' do
+        it "passes validation, as usual" do
           expect(route.valid?).to be_truthy
         end
       end
 
-      context 'when the route is not orphaned' do
-        it 'does not delete the conflicting route' do
-          expect do
+      context "when the route is not orphaned" do
+        it "does not delete the conflicting route" do
+          expect {
             route.valid?
-          end.not_to change { described_class.count }
+          }.not_to change { described_class.count }
         end
 
-        it 'fails validation, as usual' do
+        it "fails validation, as usual" do
           expect(route.valid?).to be_falsey
         end
       end
     end
 
-    context 'when there are no conflicting routes' do
-      it 'does not delete any routes' do
+    context "when there are no conflicting routes" do
+      it "does not delete any routes" do
         route
 
-        expect do
+        expect {
           route.valid?
-        end.not_to change { described_class.count }
+        }.not_to change { described_class.count }
       end
 
-      it 'passes validation, as usual' do
+      it "passes validation, as usual" do
         expect(route.valid?).to be_truthy
       end
     end