Add a SystemNoteService class

There's a lot of code in the Note model that only deals with creating
system notes, so we're going to split that into its own class.

1 files changed, 0 insertions, 256 deletions

diff --git a/spec/models/note_spec.rb b/spec/models/note_spec.rb
index 43bfd25a48f..4a769f36771 100644
--- a/spec/models/note_spec.rb
+++ b/spec/models/note_spec.rb
@@ -150,178 +150,6 @@ describe Note do
     end
   end
 
-  describe '#create_status_change_note' do
-    let(:project) { create(:project) }
-    let(:thing) { create(:issue, project: project) }
-    let(:author) { create(:user) }
-    let(:status) { 'new_status' }
-
-    subject { Note.create_status_change_note(thing, project, author, status, nil) }
-
-    it 'creates and saves a Note' do
-      is_expected.to be_a Note
-      expect(subject.id).not_to be_nil
-    end
-
-    describe '#noteable' do
-      subject { super().noteable }
-      it { is_expected.to eq(thing) }
-    end
-
-    describe '#project' do
-      subject { super().project }
-      it { is_expected.to eq(thing.project) }
-    end
-
-    describe '#author' do
-      subject { super().author }
-      it { is_expected.to eq(author) }
-    end
-
-    describe '#note' do
-      subject { super().note }
-      it { is_expected.to eq("Status changed to #{status}") }
-    end
-
-    it 'appends a back-reference if a closing mentionable is supplied' do
-      commit = double('commit', gfm_reference: 'commit 123456')
-      n = Note.create_status_change_note(thing, project, author, status, commit)
-
-      expect(n.note).to eq("Status changed to #{status} by commit 123456")
-    end
-  end
-
-  describe '#create_assignee_change_note' do
-    let(:project) { create(:project) }
-    let(:thing) { create(:issue, project: project) }
-    let(:author) { create(:user) }
-    let(:assignee) { create(:user, username: "assigned_user") }
-
-    subject { Note.create_assignee_change_note(thing, project, author, assignee) }
-
-    context 'creates and saves a Note' do
-      it { is_expected.to be_a Note }
-
-      describe '#id' do
-        subject { super().id }
-        it { is_expected.not_to be_nil }
-      end
-    end
-
-    describe '#noteable' do
-      subject { super().noteable }
-      it { is_expected.to eq(thing) }
-    end
-
-    describe '#project' do
-      subject { super().project }
-      it { is_expected.to eq(thing.project) }
-    end
-
-    describe '#author' do
-      subject { super().author }
-      it { is_expected.to eq(author) }
-    end
-
-    describe '#note' do
-      subject { super().note }
-      it { is_expected.to eq('Reassigned to @assigned_user') }
-    end
-
-    context 'assignee is removed' do
-      let(:assignee) { nil }
-
-      describe '#note' do
-        subject { super().note }
-        it { is_expected.to eq('Assignee removed') }
-      end
-    end
-  end
-
-  describe '#create_labels_change_note' do
-    let(:project) { create(:project) }
-    let(:thing) { create(:issue, project: project) }
-    let(:author) { create(:user) }
-    let(:label1) { create(:label) }
-    let(:label2) { create(:label) }
-    let(:added_labels) { [label1, label2] }
-    let(:removed_labels) { [] }
-
-    subject { Note.create_labels_change_note(thing, project, author, added_labels, removed_labels) }
-
-    context 'creates and saves a Note' do
-      it { is_expected.to be_a Note }
-
-      describe '#id' do
-        subject { super().id }
-        it { is_expected.not_to be_nil }
-      end
-    end
-
-    describe '#noteable' do
-      subject { super().noteable }
-      it { is_expected.to eq(thing) }
-    end
-
-    describe '#project' do
-      subject { super().project }
-      it { is_expected.to eq(thing.project) }
-    end
-
-    describe '#author' do
-      subject { super().author }
-      it { is_expected.to eq(author) }
-    end
-
-    describe '#note' do
-      subject { super().note }
-      it { is_expected.to eq("Added ~#{label1.id} ~#{label2.id} labels") }
-    end
-
-    context 'label is removed' do
-      let(:added_labels) { [label1]  }
-      let(:removed_labels) { [label2] }
-
-      describe '#note' do
-        subject { super().note }
-        it { is_expected.to eq("Added ~#{label1.id} and removed ~#{label2.id} labels") }
-      end
-    end
-  end
-
-  describe '#create_milestone_change_note' do
-    let(:project) { create(:project) }
-    let(:thing) { create(:issue, project: project) }
-    let(:milestone) { create(:milestone, project: project, title: "first_milestone") }
-    let(:author) { create(:user) }
-
-    subject { Note.create_milestone_change_note(thing, project, author, milestone) }
-
-    context 'creates and saves a Note' do
-      it { is_expected.to be_a Note }
-
-      describe '#id' do
-        subject { super().id }
-        it { is_expected.not_to be_nil }
-      end
-    end
-
-    describe '#project' do
-      subject { super().project }
-      it { is_expected.to eq(thing.project) }
-    end
-
-    describe '#author' do
-      subject { super().author }
-      it { is_expected.to eq(author) }
-    end
-
-    describe '#note' do
-      subject { super().note }
-      it { is_expected.to eq("Milestone changed to first_milestone") }
-    end
-  end
-
   describe '#create_cross_reference_note' do
     let(:project)    { create(:project) }
     let(:author)     { create(:user) }
@@ -476,70 +304,6 @@ describe Note do
     end
   end
 
-  describe '#cross_reference_exists?' do
-    let(:project) { create :project }
-    let(:author) { create :user }
-    let(:issue) { create :issue }
-    let(:commit0) { project.commit }
-    let(:commit1) { project.commit('HEAD~2') }
-
-    before do
-      Note.create_cross_reference_note(issue, commit0, author)
-    end
-
-    it 'detects if a mentionable has already been mentioned' do
-      expect(Note.cross_reference_exists?(issue, commit0)).to be_truthy
-    end
-
-    it 'detects if a mentionable has not already been mentioned' do
-      expect(Note.cross_reference_exists?(issue, commit1)).to be_falsey
-    end
-
-    context 'commit on commit' do
-      before do
-        Note.create_cross_reference_note(commit0, commit1, author)
-      end
-
-      it { expect(Note.cross_reference_exists?(commit0, commit1)).to be_truthy }
-      it { expect(Note.cross_reference_exists?(commit1, commit0)).to be_falsey }
-    end
-
-    context 'legacy note with Markdown emphasis' do
-      let(:issue2) { create :issue, project: project }
-      let!(:note) do
-        create :note, system: true, noteable_id: issue2.id,
-                      noteable_type: "Issue", note: "_mentioned in issue " \
-                      "#{issue.project.path_with_namespace}##{issue.iid}_"
-      end
-
-      it 'detects if a mentionable with emphasis has been mentioned' do
-        expect(Note.cross_reference_exists?(issue2, issue)).to be_truthy
-      end
-    end
-  end
-
-  describe '#cross_references_with_underscores?' do
-    let(:project) { create :project, path: "first_project" }
-    let(:second_project) { create :project, path: "second_project" }
-
-    let(:author) { create :user }
-    let(:issue0) { create :issue, project: project }
-    let(:issue1) { create :issue, project: second_project }
-    let!(:note) { Note.create_cross_reference_note(issue0, issue1, author) }
-
-    it 'detects if a mentionable has already been mentioned' do
-      expect(Note.cross_reference_exists?(issue0, issue1)).to be_truthy
-    end
-
-    it 'detects if a mentionable has not already been mentioned' do
-      expect(Note.cross_reference_exists?(issue1, issue0)).to be_falsey
-    end
-
-    it 'detects that text has underscores' do
-      expect(note.note).to eq("mentioned in issue #{second_project.path_with_namespace}##{issue1.iid}")
-    end
-  end
-
   describe '#system?' do
     let(:project) { create(:project) }
     let(:issue)   { create(:issue, project: project) }
@@ -554,30 +318,10 @@ describe Note do
       expect(@note).not_to be_system
     end
 
-    it 'should identify status-change notes as system notes' do
-      @note = Note.create_status_change_note(issue, project, author, 'closed', nil)
-      expect(@note).to be_system
-    end
-
     it 'should identify cross-reference notes as system notes' do
       @note = Note.create_cross_reference_note(issue, other, author)
       expect(@note).to be_system
     end
-
-    it 'should identify assignee-change notes as system notes' do
-      @note = Note.create_assignee_change_note(issue, project, author, assignee)
-      expect(@note).to be_system
-    end
-
-    it 'should identify label-change notes as system notes' do
-      @note = Note.create_labels_change_note(issue, project, author, [label], [])
-      expect(@note).to be_system
-    end
-
-    it 'should identify milestone-change notes as system notes' do
-      @note = Note.create_milestone_change_note(issue, project, author, milestone)
-      expect(@note).to be_system
-    end
   end
 
   describe :authorization do