Merge branch 'Add-Issue-Link-to-Open-Message' into 'master'

Add Issue Link to "Issue opened by" Integration Chat Message

See merge request gitlab-org/gitlab!42785

app/models/project_services/chat_message/issue_message.rb

@@ -41,15 +41,11 @@ module ChatMessage
     private
 
     def message
-      if opened_issue?
-        "[#{project_link}] Issue #{state} by #{user_combined_name}"
-      else
       "[#{project_link}] Issue #{issue_link} #{state} by #{user_combined_name}"
     end
-    end
 
     def opened_issue?
-      action == "open"
+      action == 'open'
     end
 
     def description_message
@@ -57,7 +53,7 @@ module ChatMessage
         title: issue_title,
         title_link: issue_url,
         text: format(description),
-        color: "#C95823"
+        color: '#C95823'
       }]
     end
 

changelogs/unreleased/Add-Issue-Link-to-Open-Message.yml

+---
+title: Add Issue Link to "Issue opened by" Integration Chat Message
+merge_request: 42785
+author: Kev @KevSlashNull
+type: changed

spec/models/project_services/chat_message/issue_message_spec.rb

@@ -44,7 +44,7 @@ RSpec.describe ChatMessage::IssueMessage do
     context 'open' do
       it 'returns a message regarding opening of issues' do
         expect(subject.pretext).to eq(
-          '[<http://somewhere.com|project_name>] Issue opened by Test User (test.user)')
+          '[<http://somewhere.com|project_name>] Issue <http://url.com|#100 Issue title> opened by Test User (test.user)')
         expect(subject.attachments).to eq([
           {
             title: "#100 Issue title",
@@ -91,7 +91,7 @@ RSpec.describe ChatMessage::IssueMessage do
     context 'open' do
       it 'returns a message regarding opening of issues' do
         expect(subject.pretext).to eq(
-          '[[project_name](http://somewhere.com)] Issue opened by Test User (test.user)')
+          '[[project_name](http://somewhere.com)] Issue [#100 Issue title](http://url.com) opened by Test User (test.user)')
         expect(subject.attachments).to eq('issue description')
         expect(subject.activity).to eq({
           title: 'Issue opened by Test User (test.user)',