EE Port of Display "commented" only for commit discussions

EE backport of
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/24427

app/assets/javascripts/notes/components/noteable_discussion.vue

@@ -222,6 +222,16 @@ export default {
 
       return this.line;
     },
+    commit() {
+      if (!this.discussion.for_commit) {
+        return null;
+      }
+
+      return {
+        id: this.discussion.commit_id,
+        url: this.discussion.discussion_path,
+      };
+    },
   },
   watch: {
     isReplying() {
@@ -386,6 +396,7 @@ Please check your network connection and try again.`;
                     :is="componentName(initialDiscussion)"
                     :note="componentData(initialDiscussion)"
                     :line="line"
+                    :commit="commit"
                     :help-page-path="helpPagePath"
                     @handleDeleteNote="deleteNoteHandler"
                   >

app/assets/javascripts/notes/components/noteable_note.vue

@@ -2,6 +2,7 @@
 import $ from 'jquery';
 import { mapGetters, mapActions } from 'vuex';
 import { escape } from 'underscore';
+import { truncateSha } from '~/lib/utils/text_utility';
 import TimelineEntryItem from '~/vue_shared/components/notes/timeline_entry_item.vue';
 import Flash from '../../flash';
 import userAvatarLink from '../../vue_shared/components/user_avatar/user_avatar_link.vue';
@@ -37,6 +38,11 @@ export default {
       required: false,
       default: '',
     },
+    commit: {
+      type: Object,
+      required: false,
+      default: () => null,
+    },
   },
   data() {
     return {
@@ -73,6 +79,13 @@ export default {
     isTarget() {
       return this.targetNoteHash === this.noteAnchorId;
     },
+    truncatedHash() {
+      if (!this.commit) {
+        return null;
+      }
+
+      return truncateSha(this.commit.id);
+    },
   },
 
   created() {
@@ -204,14 +217,12 @@ export default {
     </div>
     <div class="timeline-content">
       <div class="note-header">
-        <note-header
-          v-once
-          :author="author"
-          :created-at="note.created_at"
-          :note-id="note.id"
-          action-text="commented"
-        >
-          <slot slot="note-header-info" name="note-header-info"> </slot>
+        <note-header v-once :author="author" :created-at="note.created_at" :note-id="note.id">
+          <span v-if="commit">
+            {{ s__('MergeRequests|commented on commit ')
+            }}<a class="commit-sha monospace" :href="commit.url">{{ truncatedHash }}</a>
+          </span>
+          <span v-else class="d-none d-sm-inline">&middot;</span>
         </note-header>
         <note-actions
           :author-id="author.id"

app/helpers/notes_helper.rb

@@ -85,7 +85,7 @@ module NotesHelper
 
       diffs_project_merge_request_path(discussion.project, discussion.noteable, path_params)
     elsif discussion.for_commit?
-      anchor = discussion.line_code if discussion.diff_discussion?
+      anchor = discussion.diff_discussion? ? discussion.line_code : "note_#{discussion.first_note.id}"
 
       project_commit_path(discussion.project, discussion.noteable, anchor: anchor)
     end

changelogs/unreleased/53950-commit-comments-displayed-on-a-merge-request.yml

+---
+title: Display "commented" only for commit discussions on merge requests
+merge_request: 24427
+author:
+type: changed

locale/gitlab.pot

@@ -5644,6 +5644,9 @@ msgstr ""
 msgid "MergeRequests|View replaced file @ %{commitId}"
 msgstr ""
 
+msgid "MergeRequests|commented on commit "
+msgstr ""
+
 msgid "MergeRequests|started a discussion"
 msgstr ""
 

spec/features/merge_request/user_sees_discussions_spec.rb

@@ -88,5 +88,17 @@ describe 'Merge request > User sees discussions', :js do
         expect(page).to have_content "started a discussion on commit #{note.commit_id[0...7]}"
       end
     end
+
+    context 'a commit non-diff discussion' do
+      let(:note) { create(:discussion_note_on_commit, project: project) }
+
+      it 'displays correct header' do
+        page.within(find("#note_#{note.id}", match: :first)) do
+          refresh # Trigger a refresh of notes.
+          wait_for_requests
+          expect(page).to have_content "commented on commit #{note.commit_id[0...7]}"
+        end
+      end
+    end
   end
 end

spec/helpers/notes_helper_spec.rb

@@ -185,8 +185,8 @@ describe NotesHelper do
       context 'for a non-diff discussion' do
         let(:discussion) { create(:discussion_note_on_commit, project: project).to_discussion }
 
-        it 'returns the commit path' do
-          expect(helper.discussion_path(discussion)).to eq(project_commit_path(project, commit))
+        it 'returns the commit path with the note anchor' do
+          expect(helper.discussion_path(discussion)).to eq(project_commit_path(project, commit, anchor: "note_#{discussion.first_note.id}"))
         end
       end
     end