1 files changed, 49 insertions, 5 deletions
diff --git a/spec/models/state_note_spec.rb b/spec/models/state_note_spec.rb
index d3409315e41..bd07af7ceca 100644
--- a/spec/models/state_note_spec.rb
+++ b/spec/models/state_note_spec.rb
@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-describe StateNote do
+RSpec.describe StateNote do
   describe '.from_event' do
     let_it_be(:author) { create(:user) }
     let_it_be(:project) { create(:project, :repository) }
@@ -10,18 +10,62 @@ describe StateNote do
 
     ResourceStateEvent.states.each do |state, _value|
       context "with event state #{state}" do
-        let_it_be(:event) { create(:resource_state_event, issue: noteable, state: state, created_at: '2020-02-05') }
+        let(:event) { create(:resource_state_event, issue: noteable, state: state, created_at: '2020-02-05') }
 
         subject { described_class.from_event(event, resource: noteable, resource_parent: project) }
 
-        it_behaves_like 'a system note', exclude_project: true do
-          let(:action) { state.to_s }
+        it_behaves_like 'a synthetic note', state == 'reopened' ? 'opened' : state
+
+        it 'contains the expected values' do
+          expect(subject.author).to eq(author)
+          expect(subject.created_at).to eq(event.created_at)
+          expect(subject.note).to eq(state)
+        end
+      end
+    end
+
+    context 'with a mentionable source' do
+      subject { described_class.from_event(event, resource: noteable, resource_parent: project) }
+
+      context 'with a commit' do
+        let(:commit) { create(:commit, project: project) }
+        let(:event) { create(:resource_state_event, issue: noteable, state: :closed, created_at: '2020-02-05', source_commit: commit.id) }
+
+        it 'contains the expected values' do
+          expect(subject.author).to eq(author)
+          expect(subject.created_at).to eq(subject.created_at)
+          expect(subject.note).to eq("closed via commit #{commit.id}")
+        end
+      end
+
+      context 'with a merge request' do
+        let(:merge_request) { create(:merge_request, source_project: project) }
+        let(:event) { create(:resource_state_event, issue: noteable, state: :closed, created_at: '2020-02-05', source_merge_request: merge_request) }
+
+        it 'contains the expected values' do
+          expect(subject.author).to eq(author)
+          expect(subject.created_at).to eq(event.created_at)
+          expect(subject.note).to eq("closed via merge request !#{merge_request.iid}")
+        end
+      end
+
+      context 'when closed by error tracking' do
+        let(:event) { create(:resource_state_event, issue: noteable, state: :closed, created_at: '2020-02-05', close_after_error_tracking_resolve: true) }
+
+        it 'contains the expected values' do
+          expect(subject.author).to eq(author)
+          expect(subject.created_at).to eq(event.created_at)
+          expect(subject.note).to eq('resolved the corresponding error and closed the issue.')
         end
+      end
+
+      context 'when closed by promotheus alert' do
+        let(:event) { create(:resource_state_event, issue: noteable, state: :closed, created_at: '2020-02-05', close_auto_resolve_prometheus_alert: true) }
 
         it 'contains the expected values' do
           expect(subject.author).to eq(author)
           expect(subject.created_at).to eq(event.created_at)
-          expect(subject.note_html).to eq("<p dir=\"auto\">#{state}</p>")
+          expect(subject.note).to eq('automatically closed this issue because the alert resolved.')
         end
       end
     end