# frozen_string_literal: true require 'spec_helper' RSpec.describe Issuable::CommonSystemNotesService do let_it_be(:project) { create(:project) } let_it_be(:user) { create(:user) } let(:issuable) { create(:issue, project: project) } context 'on issuable update' do it_behaves_like 'system note creation', { title: 'New title' }, 'changed title' it_behaves_like 'system note creation', { description: 'New description' }, 'changed the description' it_behaves_like 'system note creation', { discussion_locked: true }, 'locked this issue' it_behaves_like 'system note creation', { time_estimate: 5 }, 'changed time estimate' context 'when new label is added' do let(:label) { create(:label, project: project) } before do issuable.labels << label issuable.save end it 'creates a resource label event' do described_class.new(project, user).execute(issuable, old_labels: []) event = issuable.reload.resource_label_events.last expect(event).not_to be_nil expect(event.label_id).to eq label.id expect(event.user_id).to eq user.id end end context 'with merge requests Draft note' do context 'adding Draft note' do let(:issuable) { create(:merge_request, title: "merge request") } it_behaves_like 'system note creation', { title: "Draft: merge request" }, 'marked as a **Work In Progress**' context 'and changing title' do before do issuable.update_attribute(:title, "Draft: changed title") end it_behaves_like 'draft notes creation', 'marked' end end context 'removing Draft note' do let(:issuable) { create(:merge_request, title: "Draft: merge request") } it_behaves_like 'system note creation', { title: "merge request" }, 'unmarked as a **Work In Progress**' context 'and changing title' do before do issuable.update_attribute(:title, "changed title") end it_behaves_like 'draft notes creation', 'unmarked' end end end end context 'on issuable create' do let(:issuable) { build(:issue, project: project) } subject { described_class.new(project, user).execute(issuable, old_labels: [], is_update: false) } it 'does not create system note for title and description' do issuable.save expect { subject }.not_to change { issuable.notes.count } end it 'creates a resource label event for labels added' do label = create(:label, project: project) issuable.labels << label issuable.save expect { subject }.to change { issuable.resource_label_events.count }.from(0).to(1) event = issuable.reload.resource_label_events.last expect(event).not_to be_nil expect(event.label_id).to eq label.id expect(event.user_id).to eq user.id end context 'when changing milestones' do let_it_be(:milestone) { create(:milestone, project: project) } let_it_be(:issuable) { create(:issue, project: project, milestone: milestone) } it 'does not create a system note for milestone set' do expect { subject }.not_to change { issuable.notes.count } end it 'creates a milestone change event' do expect { subject }.to change { ResourceMilestoneEvent.count }.from(0).to(1) end end it 'creates a system note for due_date set' do issuable.due_date = Date.today issuable.save expect { subject }.to change { issuable.notes.count }.from(0).to(1) expect(issuable.notes.last.note).to match('changed due date') end end end