2013-03-26 03:48:13 -04:00
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
describe NotificationService do
|
|
|
|
let(:notification) { NotificationService.new }
|
2013-08-08 10:42:27 -04:00
|
|
|
|
2013-03-26 03:48:13 -04:00
|
|
|
describe 'Keys' do
|
|
|
|
describe :new_key do
|
2014-04-03 05:05:21 -04:00
|
|
|
let!(:key) { create(:personal_key) }
|
2013-03-26 03:48:13 -04:00
|
|
|
|
|
|
|
it { notification.new_key(key).should be_true }
|
|
|
|
|
|
|
|
it 'should sent email to key owner' do
|
|
|
|
Notify.should_receive(:new_ssh_key_email).with(key.id)
|
|
|
|
notification.new_key(key)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-02-08 22:08:49 -05:00
|
|
|
describe 'Email' do
|
|
|
|
describe :new_email do
|
2014-04-03 05:05:21 -04:00
|
|
|
let!(:email) { create(:email) }
|
2014-02-08 22:08:49 -05:00
|
|
|
|
|
|
|
it { notification.new_email(email).should be_true }
|
|
|
|
|
|
|
|
it 'should send email to email owner' do
|
|
|
|
Notify.should_receive(:new_email_email).with(email.id)
|
|
|
|
notification.new_email(email)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2013-03-28 08:39:59 -04:00
|
|
|
describe 'Notes' do
|
2013-03-28 08:51:25 -04:00
|
|
|
context 'issue note' do
|
|
|
|
let(:issue) { create(:issue, assignee: create(:user)) }
|
2014-02-24 09:31:58 -05:00
|
|
|
let(:mentioned_issue) { create(:issue, assignee: issue.assignee) }
|
2013-05-09 19:37:47 -04:00
|
|
|
let(:note) { create(:note_on_issue, noteable: issue, project_id: issue.project_id, note: '@mention referenced') }
|
2013-03-28 08:39:59 -04:00
|
|
|
|
2013-03-28 08:51:25 -04:00
|
|
|
before do
|
|
|
|
build_team(note.project)
|
2013-03-28 08:39:59 -04:00
|
|
|
end
|
|
|
|
|
2013-03-28 08:51:25 -04:00
|
|
|
describe :new_note do
|
|
|
|
it do
|
|
|
|
should_email(@u_watcher.id)
|
|
|
|
should_email(note.noteable.author_id)
|
|
|
|
should_email(note.noteable.assignee_id)
|
2013-05-09 19:37:47 -04:00
|
|
|
should_email(@u_mentioned.id)
|
2013-03-28 08:51:25 -04:00
|
|
|
should_not_email(note.author_id)
|
|
|
|
should_not_email(@u_participating.id)
|
|
|
|
should_not_email(@u_disabled.id)
|
|
|
|
notification.new_note(note)
|
|
|
|
end
|
|
|
|
|
2014-02-24 09:31:58 -05:00
|
|
|
it 'filters out "mentioned in" notes' do
|
|
|
|
mentioned_note = Note.create_cross_reference_note(mentioned_issue, issue, issue.author, issue.project)
|
|
|
|
|
|
|
|
Notify.should_not_receive(:note_issue_email)
|
|
|
|
notification.new_note(mentioned_note)
|
|
|
|
end
|
2014-03-28 12:15:58 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
describe 'new note on issue in project that belongs to a group' do
|
|
|
|
let(:group) { create(:group) }
|
2014-02-24 09:31:58 -05:00
|
|
|
|
2014-03-28 12:15:58 -04:00
|
|
|
before do
|
|
|
|
note.project.namespace_id = group.id
|
2014-09-14 12:32:51 -04:00
|
|
|
note.project.group.add_user(@u_watcher, GroupMember::MASTER)
|
2014-03-28 12:15:58 -04:00
|
|
|
note.project.save
|
2014-09-14 12:32:51 -04:00
|
|
|
user_project = note.project.project_members.find_by_user_id(@u_watcher.id)
|
2014-03-28 12:15:58 -04:00
|
|
|
user_project.notification_level = Notification::N_PARTICIPATING
|
|
|
|
user_project.save
|
2014-09-14 12:32:51 -04:00
|
|
|
user_group = note.project.group.group_members.find_by_user_id(@u_watcher.id)
|
2014-03-28 12:15:58 -04:00
|
|
|
user_group.notification_level = Notification::N_GLOBAL
|
|
|
|
user_group.save
|
2013-03-28 08:51:25 -04:00
|
|
|
end
|
|
|
|
|
2014-03-28 12:15:58 -04:00
|
|
|
it do
|
|
|
|
should_email(note.noteable.author_id)
|
|
|
|
should_email(note.noteable.assignee_id)
|
|
|
|
should_email(@u_mentioned.id)
|
|
|
|
should_not_email(@u_watcher.id)
|
|
|
|
should_not_email(note.author_id)
|
|
|
|
should_not_email(@u_participating.id)
|
|
|
|
should_not_email(@u_disabled.id)
|
|
|
|
notification.new_note(note)
|
2013-03-28 08:51:25 -04:00
|
|
|
end
|
2013-03-28 08:39:59 -04:00
|
|
|
end
|
2014-03-28 12:15:58 -04:00
|
|
|
|
|
|
|
def should_email(user_id)
|
|
|
|
Notify.should_receive(:note_issue_email).with(user_id, note.id)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_not_email(user_id)
|
|
|
|
Notify.should_not_receive(:note_issue_email).with(user_id, note.id)
|
|
|
|
end
|
2013-03-28 08:51:25 -04:00
|
|
|
end
|
2013-03-28 08:39:59 -04:00
|
|
|
|
2014-06-23 08:02:09 -04:00
|
|
|
context 'issue note mention' do
|
|
|
|
let(:issue) { create(:issue, assignee: create(:user)) }
|
|
|
|
let(:mentioned_issue) { create(:issue, assignee: issue.assignee) }
|
|
|
|
let(:note) { create(:note_on_issue, noteable: issue, project_id: issue.project_id, note: '@all mentioned') }
|
|
|
|
|
|
|
|
before do
|
|
|
|
build_team(note.project)
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :new_note do
|
|
|
|
it do
|
|
|
|
# Notify all team members
|
|
|
|
note.project.team.members.each do |member|
|
|
|
|
# User with disabled notification should not be notified
|
|
|
|
next if member.id == @u_disabled.id
|
|
|
|
should_email(member.id)
|
|
|
|
end
|
|
|
|
should_email(note.noteable.author_id)
|
|
|
|
should_email(note.noteable.assignee_id)
|
|
|
|
|
|
|
|
should_not_email(note.author_id)
|
|
|
|
should_not_email(@u_disabled.id)
|
|
|
|
should_not_email(@u_not_mentioned.id)
|
|
|
|
notification.new_note(note)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'filters out "mentioned in" notes' do
|
|
|
|
mentioned_note = Note.create_cross_reference_note(mentioned_issue, issue, issue.author, issue.project)
|
|
|
|
|
|
|
|
Notify.should_not_receive(:note_issue_email)
|
|
|
|
notification.new_note(mentioned_note)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_email(user_id)
|
|
|
|
Notify.should_receive(:note_issue_email).with(user_id, note.id)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_not_email(user_id)
|
|
|
|
Notify.should_not_receive(:note_issue_email).with(user_id, note.id)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2013-03-28 08:51:25 -04:00
|
|
|
context 'commit note' do
|
2013-07-29 16:34:53 -04:00
|
|
|
let(:note) { create(:note_on_commit) }
|
2013-03-28 08:51:25 -04:00
|
|
|
|
|
|
|
before do
|
|
|
|
build_team(note.project)
|
2013-09-13 09:38:30 -04:00
|
|
|
note.stub(:commit_author => @u_committer)
|
2013-03-28 08:39:59 -04:00
|
|
|
end
|
|
|
|
|
2013-03-28 08:51:25 -04:00
|
|
|
describe :new_note do
|
|
|
|
it do
|
2013-09-13 09:38:30 -04:00
|
|
|
should_email(@u_committer.id, note)
|
2013-07-29 16:34:53 -04:00
|
|
|
should_email(@u_watcher.id, note)
|
|
|
|
should_not_email(@u_mentioned.id, note)
|
|
|
|
should_not_email(note.author_id, note)
|
|
|
|
should_not_email(@u_participating.id, note)
|
|
|
|
should_not_email(@u_disabled.id, note)
|
2013-03-28 08:51:25 -04:00
|
|
|
notification.new_note(note)
|
|
|
|
end
|
|
|
|
|
|
|
|
it do
|
2013-09-13 09:38:30 -04:00
|
|
|
note.update_attribute(:note, '@mention referenced')
|
|
|
|
should_email(@u_committer.id, note)
|
|
|
|
should_email(@u_watcher.id, note)
|
|
|
|
should_email(@u_mentioned.id, note)
|
|
|
|
should_not_email(note.author_id, note)
|
|
|
|
should_not_email(@u_participating.id, note)
|
|
|
|
should_not_email(@u_disabled.id, note)
|
|
|
|
notification.new_note(note)
|
2013-03-28 08:51:25 -04:00
|
|
|
end
|
|
|
|
|
2013-07-29 16:34:53 -04:00
|
|
|
def should_email(user_id, n)
|
|
|
|
Notify.should_receive(:note_commit_email).with(user_id, n.id)
|
2013-03-28 08:51:25 -04:00
|
|
|
end
|
|
|
|
|
2013-07-29 16:34:53 -04:00
|
|
|
def should_not_email(user_id, n)
|
|
|
|
Notify.should_not_receive(:note_commit_email).with(user_id, n.id)
|
2013-03-28 08:51:25 -04:00
|
|
|
end
|
2013-03-28 08:39:59 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2013-03-26 03:48:13 -04:00
|
|
|
describe 'Issues' do
|
|
|
|
let(:issue) { create :issue, assignee: create(:user) }
|
|
|
|
|
2013-03-28 05:59:06 -04:00
|
|
|
before do
|
|
|
|
build_team(issue.project)
|
|
|
|
end
|
|
|
|
|
2013-03-26 03:48:13 -04:00
|
|
|
describe :new_issue do
|
2013-03-28 06:24:04 -04:00
|
|
|
it do
|
|
|
|
should_email(issue.assignee_id)
|
|
|
|
should_email(@u_watcher.id)
|
|
|
|
should_not_email(@u_participating.id)
|
|
|
|
should_not_email(@u_disabled.id)
|
|
|
|
notification.new_issue(issue, @u_disabled)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_email(user_id)
|
|
|
|
Notify.should_receive(:new_issue_email).with(user_id, issue.id)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_not_email(user_id)
|
|
|
|
Notify.should_not_receive(:new_issue_email).with(user_id, issue.id)
|
2013-03-26 03:48:13 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :reassigned_issue do
|
2013-03-28 05:59:06 -04:00
|
|
|
it 'should email new assignee' do
|
|
|
|
should_email(issue.assignee_id)
|
|
|
|
should_email(@u_watcher.id)
|
|
|
|
should_not_email(@u_participating.id)
|
|
|
|
should_not_email(@u_disabled.id)
|
|
|
|
|
|
|
|
notification.reassigned_issue(issue, @u_disabled)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_email(user_id)
|
2014-07-02 07:32:53 -04:00
|
|
|
Notify.should_receive(:reassigned_issue_email).with(user_id, issue.id, nil, @u_disabled.id)
|
2013-03-28 05:59:06 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
def should_not_email(user_id)
|
2014-02-17 12:49:42 -05:00
|
|
|
Notify.should_not_receive(:reassigned_issue_email).with(user_id, issue.id, issue.assignee_id, @u_disabled.id)
|
2013-03-26 03:48:13 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :close_issue do
|
|
|
|
it 'should sent email to issue assignee and issue author' do
|
2013-03-28 05:59:06 -04:00
|
|
|
should_email(issue.assignee_id)
|
|
|
|
should_email(issue.author_id)
|
|
|
|
should_email(@u_watcher.id)
|
|
|
|
should_not_email(@u_participating.id)
|
|
|
|
should_not_email(@u_disabled.id)
|
|
|
|
|
|
|
|
notification.close_issue(issue, @u_disabled)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_email(user_id)
|
2013-03-28 06:24:04 -04:00
|
|
|
Notify.should_receive(:closed_issue_email).with(user_id, issue.id, @u_disabled.id)
|
2013-03-28 05:59:06 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
def should_not_email(user_id)
|
2013-03-28 06:24:04 -04:00
|
|
|
Notify.should_not_receive(:closed_issue_email).with(user_id, issue.id, @u_disabled.id)
|
2013-03-26 03:48:13 -04:00
|
|
|
end
|
|
|
|
end
|
2014-07-03 08:05:07 -04:00
|
|
|
|
|
|
|
describe :reopen_issue do
|
|
|
|
it 'should send email to issue assignee and issue author' do
|
|
|
|
should_email(issue.assignee_id)
|
|
|
|
should_email(issue.author_id)
|
|
|
|
should_email(@u_watcher.id)
|
|
|
|
should_not_email(@u_participating.id)
|
|
|
|
should_not_email(@u_disabled.id)
|
|
|
|
|
|
|
|
notification.reopen_issue(issue, @u_disabled)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_email(user_id)
|
|
|
|
Notify.should_receive(:issue_status_changed_email).with(user_id, issue.id, 'reopened', @u_disabled.id)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_not_email(user_id)
|
|
|
|
Notify.should_not_receive(:issue_status_changed_email).with(user_id, issue.id, 'reopened', @u_disabled.id)
|
|
|
|
end
|
|
|
|
end
|
2013-03-26 03:48:13 -04:00
|
|
|
end
|
2013-03-26 11:16:06 -04:00
|
|
|
|
|
|
|
describe 'Merge Requests' do
|
|
|
|
let(:merge_request) { create :merge_request, assignee: create(:user) }
|
|
|
|
|
2013-03-28 05:59:06 -04:00
|
|
|
before do
|
2013-04-25 10:15:33 -04:00
|
|
|
build_team(merge_request.target_project)
|
2013-03-28 05:59:06 -04:00
|
|
|
end
|
|
|
|
|
2013-03-26 11:16:06 -04:00
|
|
|
describe :new_merge_request do
|
2013-03-28 05:59:06 -04:00
|
|
|
it do
|
|
|
|
should_email(merge_request.assignee_id)
|
|
|
|
should_email(@u_watcher.id)
|
|
|
|
should_not_email(@u_participating.id)
|
|
|
|
should_not_email(@u_disabled.id)
|
|
|
|
notification.new_merge_request(merge_request, @u_disabled)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_email(user_id)
|
2013-03-28 06:24:04 -04:00
|
|
|
Notify.should_receive(:new_merge_request_email).with(user_id, merge_request.id)
|
2013-03-26 11:16:06 -04:00
|
|
|
end
|
|
|
|
|
2013-03-28 05:59:06 -04:00
|
|
|
def should_not_email(user_id)
|
2013-03-28 06:24:04 -04:00
|
|
|
Notify.should_not_receive(:new_merge_request_email).with(user_id, merge_request.id)
|
2013-03-26 11:16:06 -04:00
|
|
|
end
|
|
|
|
end
|
2013-03-28 05:59:06 -04:00
|
|
|
|
|
|
|
describe :reassigned_merge_request do
|
|
|
|
it do
|
|
|
|
should_email(merge_request.assignee_id)
|
|
|
|
should_email(@u_watcher.id)
|
|
|
|
should_not_email(@u_participating.id)
|
|
|
|
should_not_email(@u_disabled.id)
|
|
|
|
notification.reassigned_merge_request(merge_request, merge_request.author)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_email(user_id)
|
2014-07-02 07:32:53 -04:00
|
|
|
Notify.should_receive(:reassigned_merge_request_email).with(user_id, merge_request.id, nil, merge_request.author_id)
|
2013-03-28 05:59:06 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
def should_not_email(user_id)
|
2014-02-17 12:49:42 -05:00
|
|
|
Notify.should_not_receive(:reassigned_merge_request_email).with(user_id, merge_request.id, merge_request.assignee_id, merge_request.author_id)
|
2013-03-28 05:59:06 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :closed_merge_request do
|
|
|
|
it do
|
|
|
|
should_email(merge_request.assignee_id)
|
|
|
|
should_email(@u_watcher.id)
|
|
|
|
should_not_email(@u_participating.id)
|
|
|
|
should_not_email(@u_disabled.id)
|
2013-03-28 06:14:53 -04:00
|
|
|
notification.close_mr(merge_request, @u_disabled)
|
2013-03-28 05:59:06 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
def should_email(user_id)
|
2013-03-28 06:24:04 -04:00
|
|
|
Notify.should_receive(:closed_merge_request_email).with(user_id, merge_request.id, @u_disabled.id)
|
2013-03-28 05:59:06 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
def should_not_email(user_id)
|
2013-03-28 06:24:04 -04:00
|
|
|
Notify.should_not_receive(:closed_merge_request_email).with(user_id, merge_request.id, @u_disabled.id)
|
2013-03-28 05:59:06 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :merged_merge_request do
|
|
|
|
it do
|
|
|
|
should_email(merge_request.assignee_id)
|
|
|
|
should_email(@u_watcher.id)
|
|
|
|
should_not_email(@u_participating.id)
|
|
|
|
should_not_email(@u_disabled.id)
|
2014-03-24 10:11:35 -04:00
|
|
|
notification.merge_mr(merge_request, @u_disabled)
|
2013-03-28 05:59:06 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
def should_email(user_id)
|
2014-03-24 10:11:35 -04:00
|
|
|
Notify.should_receive(:merged_merge_request_email).with(user_id, merge_request.id, @u_disabled.id)
|
2013-03-28 05:59:06 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
def should_not_email(user_id)
|
2014-03-24 10:11:35 -04:00
|
|
|
Notify.should_not_receive(:merged_merge_request_email).with(user_id, merge_request.id, @u_disabled.id)
|
2013-03-28 05:59:06 -04:00
|
|
|
end
|
|
|
|
end
|
2014-07-03 08:05:07 -04:00
|
|
|
|
|
|
|
describe :reopen_merge_request do
|
|
|
|
it do
|
|
|
|
should_email(merge_request.assignee_id)
|
|
|
|
should_email(@u_watcher.id)
|
|
|
|
should_not_email(@u_participating.id)
|
|
|
|
should_not_email(@u_disabled.id)
|
|
|
|
notification.reopen_mr(merge_request, @u_disabled)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_email(user_id)
|
|
|
|
Notify.should_receive(:merge_request_status_email).with(user_id, merge_request.id, 'reopened', @u_disabled.id)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_not_email(user_id)
|
|
|
|
Notify.should_not_receive(:merge_request_status_email).with(user_id, merge_request.id, 'reopened', @u_disabled.id)
|
|
|
|
end
|
|
|
|
end
|
2013-03-28 05:59:06 -04:00
|
|
|
end
|
|
|
|
|
2014-01-15 07:03:52 -05:00
|
|
|
describe 'Projects' do
|
|
|
|
let(:project) { create :project }
|
|
|
|
|
|
|
|
before do
|
|
|
|
build_team(project)
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :project_was_moved do
|
|
|
|
it do
|
|
|
|
should_email(@u_watcher.id)
|
|
|
|
should_email(@u_participating.id)
|
|
|
|
should_not_email(@u_disabled.id)
|
|
|
|
notification.project_was_moved(project)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_email(user_id)
|
|
|
|
Notify.should_receive(:project_was_moved_email).with(project.id, user_id)
|
|
|
|
end
|
|
|
|
|
|
|
|
def should_not_email(user_id)
|
|
|
|
Notify.should_not_receive(:project_was_moved_email).with(project.id, user_id)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2013-03-28 05:59:06 -04:00
|
|
|
def build_team(project)
|
2013-08-08 10:42:27 -04:00
|
|
|
@u_watcher = create(:user, notification_level: Notification::N_WATCH)
|
|
|
|
@u_participating = create(:user, notification_level: Notification::N_PARTICIPATING)
|
|
|
|
@u_disabled = create(:user, notification_level: Notification::N_DISABLED)
|
|
|
|
@u_mentioned = create(:user, username: 'mention', notification_level: Notification::N_PARTICIPATING)
|
2013-09-13 09:38:30 -04:00
|
|
|
@u_committer = create(:user, username: 'committer')
|
2014-06-23 08:02:09 -04:00
|
|
|
@u_not_mentioned = create(:user, username: 'regular', notification_level: Notification::N_PARTICIPATING)
|
2013-03-28 05:59:06 -04:00
|
|
|
|
|
|
|
project.team << [@u_watcher, :master]
|
|
|
|
project.team << [@u_participating, :master]
|
|
|
|
project.team << [@u_disabled, :master]
|
2013-05-09 19:37:47 -04:00
|
|
|
project.team << [@u_mentioned, :master]
|
2013-09-13 09:38:30 -04:00
|
|
|
project.team << [@u_committer, :master]
|
2013-03-26 11:16:06 -04:00
|
|
|
end
|
2013-03-26 03:48:13 -04:00
|
|
|
end
|