From edd042071d1b5e6f494881dc2d9c39b46e42f74b Mon Sep 17 00:00:00 2001
From: GitLab Bot <gitlab-bot@gitlab.com>
Date: Mon, 16 Sep 2019 06:06:02 +0000
Subject: [PATCH] Add latest changes from gitlab-org/gitlab@master

---
 config/routes/project.rb                      |  3 +-
 .../merge_requests_controller_spec.rb         | 34 +++++++++++++++++++
 2 files changed, 36 insertions(+), 1 deletion(-)

diff --git a/config/routes/project.rb b/config/routes/project.rb
index 14caae35a83..953ddfc4165 100644
--- a/config/routes/project.rb
+++ b/config/routes/project.rb
@@ -239,8 +239,9 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do
         end
       end
 
-      resources :merge_requests, concerns: :awardable, except: [:new, :create], constraints: { id: /\d+/ } do
+      resources :merge_requests, concerns: :awardable, except: [:new, :create, :show], constraints: { id: /\d+/ } do
         member do
+          get :show # Insert this first to ensure redirections using merge_requests#show match this route
           get :commit_change_content
           post :merge
           post :cancel_auto_merge
diff --git a/spec/controllers/projects/merge_requests_controller_spec.rb b/spec/controllers/projects/merge_requests_controller_spec.rb
index d0370dfaeee..e1f67054d0a 100644
--- a/spec/controllers/projects/merge_requests_controller_spec.rb
+++ b/spec/controllers/projects/merge_requests_controller_spec.rb
@@ -78,6 +78,40 @@ describe Projects::MergeRequestsController do
           expect(response).to be_successful
         end
       end
+
+      context 'when project has moved' do
+        let(:new_project) { create(:project) }
+
+        before do
+          project.route.destroy
+          new_project.redirect_routes.create!(path: project.full_path)
+          new_project.add_developer(user)
+        end
+
+        it 'redirects from an old merge request correctly' do
+          get :show,
+              params: {
+                namespace_id: project.namespace,
+                project_id: project,
+                id: merge_request
+              }
+
+          expect(response).to redirect_to(project_merge_request_path(new_project, merge_request))
+          expect(response).to have_gitlab_http_status(302)
+        end
+
+        it 'redirects from an old merge request commits correctly' do
+          get :commits,
+              params: {
+                namespace_id: project.namespace,
+                project_id: project,
+                id: merge_request
+              }
+
+          expect(response).to redirect_to(commits_project_merge_request_path(new_project, merge_request))
+          expect(response).to have_gitlab_http_status(302)
+        end
+      end
     end
 
     context 'when user is setting notes filters' do