this spec will correctly fail when reverting the fix

ee/app/controllers/ee/projects/merge_requests/application_controller.rb

@@ -36,7 +36,7 @@ module EE
         def clamp_approvals_before_merge(mr_params)
           return mr_params unless mr_params[:approvals_before_merge]
 
-          # target the MR target project in priority, else it depends wether the project
+          # Target the MR target project in priority, else it depends whether the project
           # is forked.
           target_project = if @merge_request
                              @merge_request.target_project

spec/ee/spec/controllers/projects/merge_requests_controller_spec.rb

@@ -291,19 +291,25 @@ describe Projects::MergeRequestsController do
       let(:upstream) { create(:project, :repository) }
       let(:project) { create(:project, :repository, forked_from_project: upstream) }
 
+      context 'when the MR target upstream' do
+        let(:merge_request) { create(:merge_request, title: 'This is targeting upstream', source_project: project, target_project: upstream) }
+
         before do
-        project.add_developer(user)
           upstream.add_developer(user)
           upstream.update_attributes(approvals_before_merge: 2)
         end
 
-      context 'when the MR target upstream' do
-        let(:merge_request) { create(:merge_request, title: 'This is targeting upstream', source_project: project, target_project: upstream) }
-
         it_behaves_like 'approvals_before_merge param'
       end
 
       context 'when the MR target the fork' do
+        let(:merge_request) { create(:merge_request, title: 'This is targeting the fork', source_project: project, target_project: project) }
+
+        before do
+          project.add_developer(user)
+          project.update_attributes(approvals_before_merge: 0)
+        end
+
         it_behaves_like 'approvals_before_merge param'
       end
     end