corrects the url building

1 files changed, 11 insertions, 2 deletions

diff --git a/spec/services/system_note_service_spec.rb b/spec/services/system_note_service_spec.rb
index a918383ecd2..148f81b6a58 100644
--- a/spec/services/system_note_service_spec.rb
+++ b/spec/services/system_note_service_spec.rb
@@ -690,11 +690,20 @@ describe SystemNoteService do
   end
 
   describe '.new_commit_summary' do
+    let(:merge_request) { create(:merge_request, :simple, target_project: project, source_project: project) }
+
     it 'escapes HTML titles' do
       commit = double(title: '<pre>This is a test</pre>', short_id: '12345678')
-      escaped = '* 12345678 - &lt;pre&gt;This is a test&lt;&#x2F;pre&gt;'
+      escaped = '&lt;pre&gt;This is a test&lt;&#x2F;pre&gt;'
+
+      expect(described_class.new_commit_summary(merge_request, [commit])).to all(match(%r[- #{escaped}]))
+    end
+
+    it 'contains the MR diffs commit url' do
+      commit = merge_request.commits.last
+      url = %r[/merge_requests/#{merge_request.iid}/diffs\?commit_id=#{commit.id}]
 
-      expect(described_class.new_commit_summary([commit])).to eq([escaped])
+      expect(described_class.new_commit_summary(merge_request, [commit])).to all(match(url))
     end
   end