Satisfy Douwe Maan

3 files changed, 8 insertions, 10 deletions

diff --git a/CHANGELOG b/CHANGELOG
index 6c2fa75899c..163a0ea54e8 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,7 +1,7 @@
 Please view this file on the master branch, on stable branches it's out of date.
 
 v 8.3.0 (unreleased)
-- Merge when build succeeds (Zeger-Jan van de Weg)
+  - Merge when build succeeds (Zeger-Jan van de Weg)
   - Fix: Assignee selector is empty when 'Unassigned' is selected (Jose Corcuera)
   - Fix 500 error when update group member permission
   - Trim leading and trailing whitespace of milestone and issueable titles (Jose Corcuera)
diff --git a/app/services/system_note_service.rb b/app/services/system_note_service.rb
index 7cb9417b13e..6975b2ee55b 100644
--- a/app/services/system_note_service.rb
+++ b/app/services/system_note_service.rb
@@ -132,7 +132,7 @@ class SystemNoteService
 
   # Called when 'merge when build succeeds' is executed
   def self.merge_when_build_succeeds(noteable, project, author, last_commit)
-    body = "Enabled an automatic merge when the build for #{last_commit.to_reference} succeeds"
+    body = "Enabled an automatic merge when the build for #{last_commit.to_reference(project)} succeeds"
 
     create_note(noteable: noteable, project: project, author: author, note: body)
   end
diff --git a/spec/features/merge_requests/merge_when_build_succeeds_spec.rb b/spec/features/merge_requests/merge_when_build_succeeds_spec.rb
index 3a8f0a344e8..ee2fb25e9e5 100644
--- a/spec/features/merge_requests/merge_when_build_succeeds_spec.rb
+++ b/spec/features/merge_requests/merge_when_build_succeeds_spec.rb
@@ -1,6 +1,4 @@
 require 'spec_helper'
-# rubocop:disable Lint/UselessAssignment
-# As rubocop doesn't see a need for both `ci_commit` and `ci_build`
 
 feature 'Merge When Build Succeeds', feature: true, js: true do
   let(:user) { create(:user) }
@@ -14,10 +12,10 @@ feature 'Merge When Build Succeeds', feature: true, js: true do
   end
 
   context "Active build for Merge Request" do
-    before do
-      ci_commit = create(:ci_commit, gl_project: project, sha: merge_request.last_commit.id, ref: merge_request.source_branch)
-      ci_build = create(:ci_build, commit: ci_commit)
+    let!(:ci_commit) { create(:ci_commit, gl_project: project, sha: merge_request.last_commit.id, ref: merge_request.source_branch) }
+    let!(:ci_build) { create(:ci_build, commit: ci_commit) }
 
+    before do
       login_as user
       visit_merge_request(merge_request)
     end
@@ -49,14 +47,15 @@ feature 'Merge When Build Succeeds', feature: true, js: true do
                                         merge_user: user, title: "MepMep", merge_when_build_succeeds: true)
     end
 
+    let!(:ci_commit) { create(:ci_commit, gl_project: project, sha: merge_request.last_commit.id, ref: merge_request.source_branch) }
+    let!(:ci_build) { create(:ci_build, commit: ci_commit) }
+
     before do
       merge_request.source_project.team << [user, :master]
       merge_request.source_branch = "feature"
       merge_request.target_branch = "master"
       merge_request.save!
 
-      ci_commit = create(:ci_commit, gl_project: project, sha: merge_request.last_commit.id, ref: merge_request.source_branch)
-      ci_build = create(:ci_build, commit: ci_commit)
 
       login_as user
       visit_merge_request(merge_request)
@@ -90,4 +89,3 @@ feature 'Merge When Build Succeeds', feature: true, js: true do
     visit namespace_project_merge_request_path(merge_request.project.namespace, merge_request.project, merge_request)
   end
 end
-# rubocop:enable Lint/UselessAssignment