2019-07-07 11:18:12 +05:30
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
require 'spec_helper'
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
describe SentNotification do
|
2020-03-09 13:42:32 +05:30
|
|
|
let_it_be(:user) { create(:user) }
|
|
|
|
let_it_be(:project) { create(:project) }
|
2018-03-17 18:26:18 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
describe 'validation' do
|
|
|
|
describe 'note validity' do
|
|
|
|
context "when the project doesn't match the noteable's project" do
|
|
|
|
subject { build(:sent_notification, noteable: create(:issue)) }
|
|
|
|
|
|
|
|
it "is invalid" do
|
|
|
|
expect(subject).not_to be_valid
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "when the project doesn't match the discussion project" do
|
|
|
|
let(:discussion_id) { create(:note).discussion_id }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { build(:sent_notification, in_reply_to_discussion_id: discussion_id) }
|
|
|
|
|
|
|
|
it "is invalid" do
|
|
|
|
expect(subject).not_to be_valid
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "when the noteable project and discussion project match" do
|
2017-09-10 17:25:29 +05:30
|
|
|
let(:project) { create(:project, :repository) }
|
2017-08-17 22:00:37 +05:30
|
|
|
let(:issue) { create(:issue, project: project) }
|
|
|
|
let(:discussion_id) { create(:note, project: project, noteable: issue).discussion_id }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { build(:sent_notification, project: project, noteable: issue, in_reply_to_discussion_id: discussion_id) }
|
|
|
|
|
|
|
|
it "is valid" do
|
|
|
|
expect(subject).to be_valid
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2019-03-02 22:35:43 +05:30
|
|
|
shared_examples 'a successful sent notification' do
|
|
|
|
it 'creates a new SentNotification' do
|
|
|
|
expect { subject }.to change { described_class.count }.by(1)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
describe '.record' do
|
|
|
|
let(:issue) { create(:issue) }
|
|
|
|
|
2019-03-02 22:35:43 +05:30
|
|
|
subject { described_class.record(issue, user.id) }
|
|
|
|
|
|
|
|
it_behaves_like 'a successful sent notification'
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
describe '.record_note' do
|
2019-03-02 22:35:43 +05:30
|
|
|
subject { described_class.record_note(note, note.author.id) }
|
2017-08-17 22:00:37 +05:30
|
|
|
|
2019-03-02 22:35:43 +05:30
|
|
|
context 'for a discussion note' do
|
|
|
|
let(:note) { create(:diff_note_on_merge_request) }
|
|
|
|
|
|
|
|
it_behaves_like 'a successful sent notification'
|
|
|
|
|
|
|
|
it 'sets in_reply_to_discussion_id' do
|
|
|
|
expect(subject.in_reply_to_discussion_id).to eq(note.discussion_id)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for an individual note' do
|
|
|
|
let(:note) { create(:note_on_merge_request) }
|
|
|
|
|
|
|
|
it_behaves_like 'a successful sent notification'
|
|
|
|
|
|
|
|
it 'does not set in_reply_to_discussion_id' do
|
|
|
|
expect(subject.in_reply_to_discussion_id).to be_nil
|
|
|
|
end
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2018-03-17 18:26:18 +05:30
|
|
|
describe '#unsubscribable?' do
|
|
|
|
shared_examples 'an unsubscribable notification' do |noteable_type|
|
|
|
|
subject { described_class.record(noteable, user.id) }
|
|
|
|
|
|
|
|
context "for #{noteable_type}" do
|
|
|
|
it { expect(subject).to be_unsubscribable }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
shared_examples 'a non-unsubscribable notification' do |noteable_type|
|
|
|
|
subject { described_class.record(noteable, user.id) }
|
|
|
|
|
|
|
|
context "for a #{noteable_type}" do
|
|
|
|
it { expect(subject).not_to be_unsubscribable }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'an unsubscribable notification', 'issue' do
|
|
|
|
let(:noteable) { create(:issue, project: project) }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'an unsubscribable notification', 'merge request' do
|
|
|
|
let(:noteable) { create(:merge_request, source_project: project) }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a non-unsubscribable notification', 'commit' do
|
|
|
|
let(:project) { create(:project, :repository) }
|
|
|
|
let(:noteable) { project.commit }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a non-unsubscribable notification', 'personal snippet' do
|
|
|
|
let(:noteable) { create(:personal_snippet, project: project) }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a non-unsubscribable notification', 'project snippet' do
|
|
|
|
let(:noteable) { create(:project_snippet, project: project) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#for_commit?' do
|
|
|
|
shared_examples 'a commit notification' do |noteable_type|
|
|
|
|
subject { described_class.record(noteable, user.id) }
|
|
|
|
|
|
|
|
context "for #{noteable_type}" do
|
|
|
|
it { expect(subject).to be_for_commit }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
shared_examples 'a non-commit notification' do |noteable_type|
|
|
|
|
subject { described_class.record(noteable, user.id) }
|
|
|
|
|
|
|
|
context "for a #{noteable_type}" do
|
|
|
|
it { expect(subject).not_to be_for_commit }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a non-commit notification', 'issue' do
|
|
|
|
let(:noteable) { create(:issue, project: project) }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a non-commit notification', 'merge request' do
|
|
|
|
let(:noteable) { create(:merge_request, source_project: project) }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a commit notification', 'commit' do
|
|
|
|
let(:project) { create(:project, :repository) }
|
|
|
|
let(:noteable) { project.commit }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a non-commit notification', 'personal snippet' do
|
|
|
|
let(:noteable) { create(:personal_snippet, project: project) }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a non-commit notification', 'project snippet' do
|
|
|
|
let(:noteable) { create(:project_snippet, project: project) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#for_snippet?' do
|
|
|
|
shared_examples 'a snippet notification' do |noteable_type|
|
|
|
|
subject { described_class.record(noteable, user.id) }
|
|
|
|
|
|
|
|
context "for #{noteable_type}" do
|
|
|
|
it { expect(subject).to be_for_snippet }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
shared_examples 'a non-snippet notification' do |noteable_type|
|
|
|
|
subject { described_class.record(noteable, user.id) }
|
|
|
|
|
|
|
|
context "for a #{noteable_type}" do
|
|
|
|
it { expect(subject).not_to be_for_snippet }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a non-snippet notification', 'issue' do
|
|
|
|
let(:noteable) { create(:issue, project: project) }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a non-snippet notification', 'merge request' do
|
|
|
|
let(:noteable) { create(:merge_request, source_project: project) }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a non-snippet notification', 'commit' do
|
|
|
|
let(:project) { create(:project, :repository) }
|
|
|
|
let(:noteable) { project.commit }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a snippet notification', 'personal snippet' do
|
|
|
|
let(:noteable) { create(:personal_snippet, project: project) }
|
|
|
|
end
|
|
|
|
|
|
|
|
it_behaves_like 'a snippet notification', 'project snippet' do
|
|
|
|
let(:noteable) { create(:project_snippet, project: project) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
describe '#create_reply' do
|
|
|
|
context 'for issue' do
|
|
|
|
let(:issue) { create(:issue) }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { described_class.record(issue, issue.author.id) }
|
|
|
|
|
|
|
|
it 'creates a comment on the issue' do
|
|
|
|
note = subject.create_reply('Test')
|
|
|
|
expect(note.in_reply_to?(issue)).to be_truthy
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for issue comment' do
|
|
|
|
let(:note) { create(:note_on_issue) }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { described_class.record_note(note, note.author.id) }
|
|
|
|
|
|
|
|
it 'creates a comment on the issue' do
|
|
|
|
new_note = subject.create_reply('Test')
|
|
|
|
expect(new_note.in_reply_to?(note)).to be_truthy
|
|
|
|
expect(new_note.discussion_id).not_to eq(note.discussion_id)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for issue discussion' do
|
|
|
|
let(:note) { create(:discussion_note_on_issue) }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { described_class.record_note(note, note.author.id) }
|
|
|
|
|
|
|
|
it 'creates a reply on the discussion' do
|
|
|
|
new_note = subject.create_reply('Test')
|
|
|
|
expect(new_note.in_reply_to?(note)).to be_truthy
|
|
|
|
expect(new_note.discussion_id).to eq(note.discussion_id)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for merge request' do
|
|
|
|
let(:merge_request) { create(:merge_request) }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { described_class.record(merge_request, merge_request.author.id) }
|
|
|
|
|
|
|
|
it 'creates a comment on the merge_request' do
|
|
|
|
note = subject.create_reply('Test')
|
|
|
|
expect(note.in_reply_to?(merge_request)).to be_truthy
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for merge request comment' do
|
|
|
|
let(:note) { create(:note_on_merge_request) }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { described_class.record_note(note, note.author.id) }
|
|
|
|
|
|
|
|
it 'creates a comment on the merge request' do
|
|
|
|
new_note = subject.create_reply('Test')
|
|
|
|
expect(new_note.in_reply_to?(note)).to be_truthy
|
|
|
|
expect(new_note.discussion_id).not_to eq(note.discussion_id)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for merge request diff discussion' do
|
|
|
|
let(:note) { create(:diff_note_on_merge_request) }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { described_class.record_note(note, note.author.id) }
|
|
|
|
|
|
|
|
it 'creates a reply on the discussion' do
|
|
|
|
new_note = subject.create_reply('Test')
|
|
|
|
expect(new_note.in_reply_to?(note)).to be_truthy
|
|
|
|
expect(new_note.discussion_id).to eq(note.discussion_id)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for merge request non-diff discussion' do
|
|
|
|
let(:note) { create(:discussion_note_on_merge_request) }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { described_class.record_note(note, note.author.id) }
|
|
|
|
|
|
|
|
it 'creates a reply on the discussion' do
|
|
|
|
new_note = subject.create_reply('Test')
|
|
|
|
expect(new_note.in_reply_to?(note)).to be_truthy
|
|
|
|
expect(new_note.discussion_id).to eq(note.discussion_id)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for commit' do
|
2017-09-10 17:25:29 +05:30
|
|
|
let(:project) { create(:project, :repository) }
|
2017-08-17 22:00:37 +05:30
|
|
|
let(:commit) { project.commit }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { described_class.record(commit, project.creator.id) }
|
|
|
|
|
|
|
|
it 'creates a comment on the commit' do
|
|
|
|
note = subject.create_reply('Test')
|
|
|
|
expect(note.in_reply_to?(commit)).to be_truthy
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for commit comment' do
|
|
|
|
let(:note) { create(:note_on_commit) }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { described_class.record_note(note, note.author.id) }
|
|
|
|
|
|
|
|
it 'creates a comment on the commit' do
|
|
|
|
new_note = subject.create_reply('Test')
|
|
|
|
expect(new_note.in_reply_to?(note)).to be_truthy
|
|
|
|
expect(new_note.discussion_id).not_to eq(note.discussion_id)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for commit diff discussion' do
|
|
|
|
let(:note) { create(:diff_note_on_commit) }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { described_class.record_note(note, note.author.id) }
|
|
|
|
|
|
|
|
it 'creates a reply on the discussion' do
|
|
|
|
new_note = subject.create_reply('Test')
|
|
|
|
expect(new_note.in_reply_to?(note)).to be_truthy
|
|
|
|
expect(new_note.discussion_id).to eq(note.discussion_id)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'for commit non-diff discussion' do
|
|
|
|
let(:note) { create(:discussion_note_on_commit) }
|
2020-03-09 13:42:32 +05:30
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
subject { described_class.record_note(note, note.author.id) }
|
|
|
|
|
|
|
|
it 'creates a reply on the discussion' do
|
|
|
|
new_note = subject.create_reply('Test')
|
|
|
|
expect(new_note.in_reply_to?(note)).to be_truthy
|
|
|
|
expect(new_note.discussion_id).to eq(note.discussion_id)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|