diff options
Diffstat (limited to 'spec/models/state_note_spec.rb')
-rw-r--r-- | spec/models/state_note_spec.rb | 52 |
1 files changed, 48 insertions, 4 deletions
diff --git a/spec/models/state_note_spec.rb b/spec/models/state_note_spec.rb index 5249c1be9ca..bd07af7ceca 100644 --- a/spec/models/state_note_spec.rb +++ b/spec/models/state_note_spec.rb @@ -10,18 +10,62 @@ RSpec.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 |