From 2d8d94a788eb0bf3885ee67bda9638556425fa4b Mon Sep 17 00:00:00 2001
From: Katarzyna Kobierska <kkobierska@gmail.com>
Date: Tue, 30 Aug 2016 13:31:39 +0200
Subject: [PATCH] Change method name

---
 CHANGELOG                         | 8 +-------
 app/models/merge_request.rb       | 6 +++---
 spec/models/merge_request_spec.rb | 8 ++++----
 3 files changed, 8 insertions(+), 14 deletions(-)

diff --git a/CHANGELOG b/CHANGELOG
index c5d035661b170..5332aaa1ab2ea 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -40,6 +40,7 @@ v 8.12.0 (unreleased)
   - Use the default branch for displaying the project icon instead of master !5792 (Hannes Rosenögger)
   - Adds response mime type to transaction metric action when it's not HTML
   - Fix hover leading space bug in pipeline graph !5980
+  - User can edit closed MR with deleted fork (Katarzyna Kobierska Ula Budziszewska) !5496
 
 v 8.11.3 (unreleased)
   - Allow system info page to handle case where info is unavailable
@@ -53,13 +54,6 @@ v 8.11.2
   - Show "Create Merge Request" widget for push events to fork projects on the source project. !5978
   - Use gitlab-workhorse 0.7.11 !5983
   - Does not halt the GitHub import process when an error occurs. !5763
-  - User can edit closed MR with deleted fork (Katarzyna Kobierska Ula Budziszewska) !5496
-
-v 8.11.2 (unreleased)
-  - Show "Create Merge Request" widget for push events to fork projects on the source project
-
-v 8.11.1 (unreleased)
-  - Does not halt the GitHub import process when an error occurs
   - Fix file links on project page when default view is Files !5933
   - Fixed enter key in search input not working !5888
 
diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb
index 27ca5d119d59a..a8dd4a306cfdb 100644
--- a/app/models/merge_request.rb
+++ b/app/models/merge_request.rb
@@ -306,17 +306,17 @@ def validate_branches
   def validate_fork
     return true unless target_project && source_project
     return true if target_project == source_project
-    return true unless fork_missing?
+    return true unless forked_source_project_missing?
 
     errors.add :validate_fork,
                'Source project is not a fork of the target project'
   end
 
   def closed_without_fork?
-    closed? && fork_missing?
+    closed? && forked_source_project_missing?
   end
 
-  def fork_missing?
+  def forked_source_project_missing?
     return false unless for_fork?
     return true unless source_project
 
diff --git a/spec/models/merge_request_spec.rb b/spec/models/merge_request_spec.rb
index 4cbf87ba792fb..901b7bad00747 100644
--- a/spec/models/merge_request_spec.rb
+++ b/spec/models/merge_request_spec.rb
@@ -963,7 +963,7 @@ def create_merge_request(source_branch)
     end
   end
 
-  describe "#fork_missing?" do
+  describe "#forked_source_project_missing?" do
     let(:project)      { create(:project) }
     let(:fork_project) { create(:project, forked_from_project: project) }
     let(:user)         { create(:user) }
@@ -976,13 +976,13 @@ def create_merge_request(source_branch)
           target_project: project)
       end
 
-      it { expect(merge_request.fork_missing?).to be_falsey }
+      it { expect(merge_request.forked_source_project_missing?).to be_falsey }
     end
 
     context "when the source project is the same as the target project" do
       let(:merge_request) { create(:merge_request, source_project: project) }
 
-      it { expect(merge_request.fork_missing?).to be_falsey }
+      it { expect(merge_request.forked_source_project_missing?).to be_falsey }
     end
 
     context "when the fork does not exist" do
@@ -996,7 +996,7 @@ def create_merge_request(source_branch)
         unlink_project.execute
         merge_request.reload
 
-        expect(merge_request.fork_missing?).to be_truthy
+        expect(merge_request.forked_source_project_missing?).to be_truthy
       end
     end
   end
-- 
GitLab