2015-10-11 05:41:23 -04:00
|
|
|
require 'rails_helper'
|
2014-03-18 00:22:39 -04:00
|
|
|
|
|
|
|
describe PostAlerter do
|
|
|
|
|
|
|
|
let!(:evil_trout) { Fabricate(:evil_trout) }
|
2016-02-24 09:34:40 -05:00
|
|
|
let(:user) { Fabricate(:user) }
|
2014-03-18 00:22:39 -04:00
|
|
|
|
|
|
|
def create_post_with_alerts(args={})
|
|
|
|
post = Fabricate(:post, args)
|
2014-03-18 21:07:48 -04:00
|
|
|
PostAlerter.post_created(post)
|
2014-03-18 00:22:39 -04:00
|
|
|
end
|
|
|
|
|
2016-03-21 23:28:14 -04:00
|
|
|
context "private message" do
|
|
|
|
it "notifies for pms correctly" do
|
|
|
|
pm = Fabricate(:topic, archetype: 'private_message', category_id: nil)
|
|
|
|
op = Fabricate(:post, user_id: pm.user_id)
|
|
|
|
pm.allowed_users << pm.user
|
|
|
|
PostAlerter.post_created(op)
|
|
|
|
reply = Fabricate(:post, user_id: pm.user_id, topic_id: pm.id, reply_to_post_number: 1)
|
|
|
|
PostAlerter.post_created(reply)
|
|
|
|
|
|
|
|
reply2 = Fabricate(:post, topic_id: pm.id, reply_to_post_number: 1)
|
|
|
|
PostAlerter.post_created(reply2)
|
|
|
|
|
|
|
|
# we get a green notification for a reply
|
|
|
|
expect(Notification.where(user_id: pm.user_id).pluck(:notification_type).first).to eq(Notification.types[:private_message])
|
|
|
|
|
|
|
|
TopicUser.change(pm.user_id, pm.id, notification_level: TopicUser.notification_levels[:tracking])
|
|
|
|
|
|
|
|
Notification.destroy_all
|
|
|
|
|
|
|
|
reply3 = Fabricate(:post, topic_id: pm.id)
|
|
|
|
PostAlerter.post_created(reply3)
|
|
|
|
|
|
|
|
# no notification cause we are tracking
|
|
|
|
expect(Notification.where(user_id: pm.user_id).count).to eq(0)
|
|
|
|
|
|
|
|
Notification.destroy_all
|
|
|
|
|
|
|
|
reply4 = Fabricate(:post, topic_id: pm.id, reply_to_post_number: 1)
|
|
|
|
PostAlerter.post_created(reply4)
|
|
|
|
|
|
|
|
# yes notification cause we were replied to
|
|
|
|
expect(Notification.where(user_id: pm.user_id).count).to eq(1)
|
|
|
|
|
|
|
|
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-10-19 16:31:48 -04:00
|
|
|
context "unread" do
|
|
|
|
it "does not return whispers as unread posts" do
|
|
|
|
op = Fabricate(:post)
|
2015-11-30 01:03:47 -05:00
|
|
|
_whisper = Fabricate(:post, raw: 'this is a whisper post',
|
2015-10-19 16:31:48 -04:00
|
|
|
user: Fabricate(:admin),
|
|
|
|
topic: op.topic,
|
|
|
|
reply_to_post_number: op.post_number,
|
|
|
|
post_type: Post.types[:whisper])
|
|
|
|
|
|
|
|
|
|
|
|
expect(PostAlerter.new.first_unread_post(op.user, op.topic)).to be_blank
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-03-03 00:00:14 -05:00
|
|
|
context 'edits' do
|
|
|
|
it 'notifies correctly on edits' do
|
|
|
|
|
2015-06-18 22:31:36 -04:00
|
|
|
ActiveRecord::Base.observers.enable :all
|
|
|
|
|
|
|
|
post = Fabricate(:post, raw: 'I love waffles')
|
|
|
|
|
|
|
|
admin = Fabricate(:admin)
|
|
|
|
post.revise(admin, {raw: 'I made a revision'})
|
|
|
|
|
2016-03-03 00:00:14 -05:00
|
|
|
# skip this notification cause we already notified on a similar edit
|
|
|
|
Timecop.freeze(2.hours.from_now) do
|
|
|
|
post.revise(admin, {raw: 'I made another revision'})
|
|
|
|
end
|
2015-06-18 22:31:36 -04:00
|
|
|
|
2016-03-03 00:00:14 -05:00
|
|
|
post.revise(Fabricate(:admin), {raw: 'I made a revision'})
|
|
|
|
|
|
|
|
Timecop.freeze(4.hours.from_now) do
|
|
|
|
post.revise(admin, {raw: 'I made another revision'})
|
|
|
|
end
|
|
|
|
|
2016-07-17 23:00:23 -04:00
|
|
|
expect(Notification.where(post_number: 1, topic_id: post.topic_id).count).to eq(3)
|
2016-03-03 00:00:14 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'likes' do
|
2016-03-04 06:55:49 -05:00
|
|
|
|
|
|
|
it 'notifies on likes after an undo' do
|
|
|
|
ActiveRecord::Base.observers.enable :all
|
|
|
|
|
|
|
|
post = Fabricate(:post, raw: 'I love waffles')
|
|
|
|
|
|
|
|
PostAction.act(evil_trout, post, PostActionType.types[:like])
|
|
|
|
PostAction.remove_act(evil_trout, post, PostActionType.types[:like])
|
|
|
|
PostAction.act(evil_trout, post, PostActionType.types[:like])
|
|
|
|
|
2016-07-17 23:00:23 -04:00
|
|
|
expect(Notification.where(post_number: 1, topic_id: post.topic_id).count).to eq(1)
|
2016-03-04 06:55:49 -05:00
|
|
|
end
|
|
|
|
|
2016-03-05 17:12:59 -05:00
|
|
|
it 'notifies on does not notify when never is selected' do
|
|
|
|
ActiveRecord::Base.observers.enable :all
|
|
|
|
|
|
|
|
post = Fabricate(:post, raw: 'I love waffles')
|
|
|
|
|
|
|
|
post.user.user_option.update_columns(like_notification_frequency:
|
|
|
|
UserOption.like_notification_frequency_type[:never])
|
|
|
|
|
|
|
|
PostAction.act(evil_trout, post, PostActionType.types[:like])
|
|
|
|
|
|
|
|
|
2016-07-17 23:00:23 -04:00
|
|
|
expect(Notification.where(post_number: 1, topic_id: post.topic_id).count).to eq(0)
|
2016-03-05 17:12:59 -05:00
|
|
|
end
|
|
|
|
|
2016-03-03 00:00:14 -05:00
|
|
|
it 'notifies on likes correctly' do
|
|
|
|
ActiveRecord::Base.observers.enable :all
|
|
|
|
|
|
|
|
post = Fabricate(:post, raw: 'I love waffles')
|
2016-03-04 06:55:49 -05:00
|
|
|
|
2016-03-03 00:00:14 -05:00
|
|
|
PostAction.act(evil_trout, post, PostActionType.types[:like])
|
2016-03-04 06:55:49 -05:00
|
|
|
admin = Fabricate(:admin)
|
|
|
|
PostAction.act(admin, post, PostActionType.types[:like])
|
2016-03-03 00:00:14 -05:00
|
|
|
|
|
|
|
# one like
|
2016-07-17 23:00:23 -04:00
|
|
|
expect(Notification.where(post_number: 1, topic_id: post.topic_id).count).to eq(1)
|
2016-03-02 07:16:52 -05:00
|
|
|
|
|
|
|
|
|
|
|
post.user.user_option.update_columns(like_notification_frequency:
|
|
|
|
UserOption.like_notification_frequency_type[:always])
|
|
|
|
|
|
|
|
admin2 = Fabricate(:admin)
|
|
|
|
PostAction.act(admin2, post, PostActionType.types[:like])
|
2016-07-17 23:00:23 -04:00
|
|
|
expect(Notification.where(post_number: 1, topic_id: post.topic_id).count).to eq(1)
|
2016-03-04 06:55:49 -05:00
|
|
|
|
|
|
|
# adds info to the notification
|
|
|
|
notification = Notification.find_by(post_number: 1,
|
|
|
|
topic_id: post.topic_id)
|
|
|
|
|
|
|
|
|
|
|
|
expect(notification.data_hash["count"].to_i).to eq(2)
|
|
|
|
expect(notification.data_hash["username2"]).to eq(evil_trout.username)
|
|
|
|
|
|
|
|
# this is a tricky thing ... removing a like should fix up the notifications
|
|
|
|
PostAction.remove_act(evil_trout, post, PostActionType.types[:like])
|
|
|
|
|
|
|
|
# rebuilds the missing notification
|
2016-07-17 23:00:23 -04:00
|
|
|
expect(Notification.where(post_number: 1, topic_id: post.topic_id).count).to eq(1)
|
2016-03-04 06:55:49 -05:00
|
|
|
notification = Notification.find_by(post_number: 1,
|
|
|
|
topic_id: post.topic_id)
|
|
|
|
|
|
|
|
expect(notification.data_hash["count"]).to eq(2)
|
|
|
|
expect(notification.data_hash["username"]).to eq(admin2.username)
|
|
|
|
expect(notification.data_hash["username2"]).to eq(admin.username)
|
|
|
|
|
2016-03-02 07:16:52 -05:00
|
|
|
|
|
|
|
post.user.user_option.update_columns(like_notification_frequency:
|
|
|
|
UserOption.like_notification_frequency_type[:first_time_and_daily])
|
|
|
|
|
|
|
|
# this gets skipped
|
|
|
|
admin3 = Fabricate(:admin)
|
|
|
|
PostAction.act(admin3, post, PostActionType.types[:like])
|
|
|
|
|
|
|
|
Timecop.freeze(2.days.from_now) do
|
|
|
|
admin4 = Fabricate(:admin)
|
|
|
|
PostAction.act(admin4, post, PostActionType.types[:like])
|
|
|
|
end
|
|
|
|
|
|
|
|
# first happend within the same day, no need to notify
|
2016-07-17 23:00:23 -04:00
|
|
|
expect(Notification.where(post_number: 1, topic_id: post.topic_id).count).to eq(2)
|
2016-03-02 07:16:52 -05:00
|
|
|
|
2015-06-18 22:31:36 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-03-18 00:22:39 -04:00
|
|
|
context 'quotes' do
|
|
|
|
|
2015-03-23 20:55:22 -04:00
|
|
|
it 'does not notify for muted users' do
|
|
|
|
post = Fabricate(:post, raw: '[quote="EvilTrout, post:1"]whatup[/quote]')
|
|
|
|
MutedUser.create!(user_id: evil_trout.id, muted_user_id: post.user_id)
|
|
|
|
|
2015-04-25 11:18:35 -04:00
|
|
|
expect {
|
2015-03-23 20:55:22 -04:00
|
|
|
PostAlerter.post_created(post)
|
2015-04-25 11:18:35 -04:00
|
|
|
}.to change(evil_trout.notifications, :count).by(0)
|
2015-03-23 20:55:22 -04:00
|
|
|
end
|
|
|
|
|
2014-03-18 00:22:39 -04:00
|
|
|
it 'notifies a user by username' do
|
2015-04-25 11:18:35 -04:00
|
|
|
expect {
|
2014-03-18 00:22:39 -04:00
|
|
|
create_post_with_alerts(raw: '[quote="EvilTrout, post:1"]whatup[/quote]')
|
2015-04-25 11:18:35 -04:00
|
|
|
}.to change(evil_trout.notifications, :count).by(1)
|
2014-03-18 00:22:39 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
it "won't notify the user a second time on revision" do
|
|
|
|
p1 = create_post_with_alerts(raw: '[quote="Evil Trout, post:1"]whatup[/quote]')
|
2015-04-25 11:18:35 -04:00
|
|
|
expect {
|
2014-10-27 17:06:43 -04:00
|
|
|
p1.revise(p1.user, { raw: '[quote="Evil Trout, post:1"]whatup now?[/quote]' })
|
2015-04-25 11:18:35 -04:00
|
|
|
}.not_to change(evil_trout.notifications, :count)
|
2014-03-18 00:22:39 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
it "doesn't notify the poster" do
|
|
|
|
topic = create_post_with_alerts.topic
|
2015-04-25 11:18:35 -04:00
|
|
|
expect {
|
2014-03-18 00:22:39 -04:00
|
|
|
Fabricate(:post, topic: topic, user: topic.user, raw: '[quote="Bruce Wayne, post:1"]whatup[/quote]')
|
2015-04-25 11:18:35 -04:00
|
|
|
}.not_to change(topic.user.notifications, :count)
|
2014-03-18 00:22:39 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-03-18 21:07:48 -04:00
|
|
|
context 'linked' do
|
|
|
|
it "will notify correctly on linking" do
|
|
|
|
post1 = create_post
|
|
|
|
user = post1.user
|
|
|
|
create_post(raw: "my magic topic\n##{Discourse.base_url}#{post1.url}")
|
|
|
|
|
2015-04-25 11:18:35 -04:00
|
|
|
expect(user.notifications.count).to eq(1)
|
2014-03-18 21:07:48 -04:00
|
|
|
|
2015-12-30 18:51:58 -05:00
|
|
|
topic = Fabricate(:topic)
|
|
|
|
|
|
|
|
watcher = Fabricate(:user)
|
|
|
|
TopicUser.create!(user_id: watcher.id, topic_id: topic.id, notification_level: TopicUser.notification_levels[:watching])
|
|
|
|
|
|
|
|
create_post(topic_id: topic.id, user: user, raw: "my magic topic\n##{Discourse.base_url}#{post1.url}")
|
2014-03-18 21:07:48 -04:00
|
|
|
|
|
|
|
user.reload
|
2016-04-12 14:09:59 -04:00
|
|
|
expect(user.notifications.where(notification_type: Notification.types[:linked]).count).to eq(1)
|
2014-07-29 01:40:05 -04:00
|
|
|
|
2015-12-30 18:51:58 -05:00
|
|
|
expect(watcher.notifications.count).to eq(1)
|
|
|
|
|
2014-07-29 01:40:05 -04:00
|
|
|
# don't notify on reflection
|
|
|
|
post1.reload
|
2015-04-25 11:18:35 -04:00
|
|
|
expect(PostAlerter.new.extract_linked_users(post1).length).to eq(0)
|
2014-07-29 01:40:05 -04:00
|
|
|
|
2014-03-18 21:07:48 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-11-30 01:03:47 -05:00
|
|
|
context '@group mentions' do
|
|
|
|
|
|
|
|
it 'notifies users correctly' do
|
|
|
|
|
|
|
|
group = Fabricate(:group, name: 'group', alias_level: Group::ALIAS_LEVELS[:everyone])
|
|
|
|
group.add(evil_trout)
|
|
|
|
|
|
|
|
expect {
|
|
|
|
create_post_with_alerts(raw: "Hello @group how are you?")
|
|
|
|
}.to change(evil_trout.notifications, :count).by(1)
|
|
|
|
|
2015-12-01 00:52:43 -05:00
|
|
|
expect(GroupMention.count).to eq(1)
|
2015-11-30 01:03:47 -05:00
|
|
|
|
2016-02-23 14:57:54 -05:00
|
|
|
Fabricate(:group, name: 'group-alt', alias_level: Group::ALIAS_LEVELS[:everyone])
|
2015-11-30 01:03:47 -05:00
|
|
|
|
|
|
|
expect {
|
2016-02-23 14:57:54 -05:00
|
|
|
create_post_with_alerts(raw: "Hello, @group-alt should not trigger a notification?")
|
2015-11-30 01:03:47 -05:00
|
|
|
}.to change(evil_trout.notifications, :count).by(0)
|
2015-12-01 00:52:43 -05:00
|
|
|
|
|
|
|
expect(GroupMention.count).to eq(2)
|
2016-02-23 14:57:54 -05:00
|
|
|
|
|
|
|
group.update_columns(alias_level: Group::ALIAS_LEVELS[:members_mods_and_admins])
|
|
|
|
expect {
|
|
|
|
create_post_with_alerts(raw: "Hello @group you are not mentionable")
|
|
|
|
}.to change(evil_trout.notifications, :count).by(0)
|
|
|
|
|
|
|
|
expect(GroupMention.count).to eq(3)
|
2015-11-30 01:03:47 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-03-18 00:22:39 -04:00
|
|
|
context '@mentions' do
|
|
|
|
|
|
|
|
let(:mention_post) { create_post_with_alerts(user: user, raw: 'Hello @eviltrout')}
|
|
|
|
let(:topic) { mention_post.topic }
|
|
|
|
|
|
|
|
it 'notifies a user' do
|
2015-04-25 11:18:35 -04:00
|
|
|
expect {
|
2014-03-18 00:22:39 -04:00
|
|
|
mention_post
|
2015-04-25 11:18:35 -04:00
|
|
|
}.to change(evil_trout.notifications, :count).by(1)
|
2014-03-18 00:22:39 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
it "won't notify the user a second time on revision" do
|
|
|
|
mention_post
|
2015-04-25 11:18:35 -04:00
|
|
|
expect {
|
2014-10-27 17:06:43 -04:00
|
|
|
mention_post.revise(mention_post.user, { raw: "New raw content that still mentions @eviltrout" })
|
2015-04-25 11:18:35 -04:00
|
|
|
}.not_to change(evil_trout.notifications, :count)
|
2014-03-18 00:22:39 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
it "doesn't notify the user who created the topic in regular mode" do
|
|
|
|
topic.notify_regular!(user)
|
|
|
|
mention_post
|
2015-04-25 11:18:35 -04:00
|
|
|
expect {
|
2014-03-18 00:22:39 -04:00
|
|
|
create_post_with_alerts(user: user, raw: 'second post', topic: topic)
|
2015-04-25 11:18:35 -04:00
|
|
|
}.not_to change(user.notifications, :count)
|
2014-03-18 00:22:39 -04:00
|
|
|
end
|
|
|
|
|
2016-03-08 15:26:06 -05:00
|
|
|
it "notification comes from editor is mention is added later" do
|
|
|
|
admin = Fabricate(:admin)
|
|
|
|
post = create_post_with_alerts(user: user, raw: 'No mention here.')
|
|
|
|
expect {
|
|
|
|
post.revise(admin, { raw: "Mention @eviltrout in this edit." })
|
|
|
|
}.to change(evil_trout.notifications, :count)
|
|
|
|
n = evil_trout.notifications.last
|
2016-03-09 12:51:54 -05:00
|
|
|
expect(n.data_hash["original_username"]).to eq(admin.username)
|
2016-03-08 15:26:06 -05:00
|
|
|
end
|
2014-03-18 00:22:39 -04:00
|
|
|
end
|
2016-02-22 19:34:16 -05:00
|
|
|
|
|
|
|
describe ".create_notification" do
|
2016-02-24 09:34:40 -05:00
|
|
|
let(:topic) { Fabricate(:private_message_topic, user: user, created_at: 1.hour.ago) }
|
|
|
|
let(:post) { Fabricate(:post, topic: topic, created_at: 1.hour.ago) }
|
2016-02-22 19:34:16 -05:00
|
|
|
|
2016-02-24 09:34:40 -05:00
|
|
|
it "creates a notification for PMs" do
|
|
|
|
post.revise(user, { raw: 'This is the revised post' }, revised_at: Time.zone.now)
|
|
|
|
|
|
|
|
expect {
|
|
|
|
PostAlerter.new.create_notification(user, Notification.types[:private_message], post)
|
|
|
|
}.to change { user.notifications.count }.by(1)
|
2016-02-22 19:34:16 -05:00
|
|
|
|
2016-02-24 09:34:40 -05:00
|
|
|
expect(user.notifications.last.data_hash["topic_title"]).to eq(topic.title)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "keeps the original title for PMs" do
|
2016-02-22 19:34:16 -05:00
|
|
|
original_title = topic.title
|
|
|
|
|
|
|
|
post.revise(user, { title: "This is the revised title" }, revised_at: Time.now)
|
|
|
|
|
|
|
|
expect {
|
|
|
|
PostAlerter.new.create_notification(user, Notification.types[:private_message], post)
|
2016-02-24 09:34:40 -05:00
|
|
|
}.to change { user.notifications.count }.by(1)
|
2016-02-22 19:34:16 -05:00
|
|
|
|
|
|
|
expect(user.notifications.last.data_hash["topic_title"]).to eq(original_title)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-07-06 15:56:40 -04:00
|
|
|
describe "watching_first_post" do
|
|
|
|
let(:group) { Fabricate(:group) }
|
|
|
|
let(:user) { Fabricate(:user) }
|
|
|
|
let(:category) { Fabricate(:category) }
|
|
|
|
let(:tag) { Fabricate(:tag) }
|
|
|
|
let(:topic) { Fabricate(:topic, category: category, tags: [tag]) }
|
|
|
|
let(:post) { Fabricate(:post, topic: topic) }
|
|
|
|
|
|
|
|
it "doesn't notify people who aren't watching" do
|
|
|
|
PostAlerter.post_created(post)
|
|
|
|
expect(user.notifications.where(notification_type: Notification.types[:watching_first_post]).count).to eq(0)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "notifies the user who is following the first post category" do
|
|
|
|
level = CategoryUser.notification_levels[:watching_first_post]
|
|
|
|
CategoryUser.set_notification_level_for_category(user, level, category.id)
|
2016-07-19 15:57:05 -04:00
|
|
|
PostAlerter.new.after_save_post(post, true)
|
2016-07-06 15:56:40 -04:00
|
|
|
expect(user.notifications.where(notification_type: Notification.types[:watching_first_post]).count).to eq(1)
|
|
|
|
end
|
|
|
|
|
2016-07-19 15:57:05 -04:00
|
|
|
it "doesn't notify when the record is not new" do
|
|
|
|
level = CategoryUser.notification_levels[:watching_first_post]
|
|
|
|
CategoryUser.set_notification_level_for_category(user, level, category.id)
|
|
|
|
PostAlerter.new.after_save_post(post, false)
|
|
|
|
expect(user.notifications.where(notification_type: Notification.types[:watching_first_post]).count).to eq(0)
|
|
|
|
end
|
|
|
|
|
2016-07-06 15:56:40 -04:00
|
|
|
it "notifies the user who is following the first post tag" do
|
|
|
|
level = TagUser.notification_levels[:watching_first_post]
|
|
|
|
TagUser.change(user.id, tag.id, level)
|
|
|
|
PostAlerter.post_created(post)
|
|
|
|
expect(user.notifications.where(notification_type: Notification.types[:watching_first_post]).count).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "notifies the user who is following the first post group" do
|
|
|
|
GroupUser.create(group_id: group.id, user_id: user.id)
|
|
|
|
GroupUser.create(group_id: group.id, user_id: post.user.id)
|
2016-07-21 15:05:10 -04:00
|
|
|
topic.topic_allowed_groups.create(group_id: group.id)
|
2016-07-06 15:56:40 -04:00
|
|
|
|
|
|
|
level = GroupUser.notification_levels[:watching_first_post]
|
|
|
|
GroupUser.where(user_id: user.id, group_id: group.id).update_all(notification_level: level)
|
|
|
|
|
|
|
|
PostAlerter.post_created(post)
|
|
|
|
expect(user.notifications.where(notification_type: Notification.types[:watching_first_post]).count).to eq(1)
|
|
|
|
end
|
|
|
|
end
|
2014-03-18 00:22:39 -04:00
|
|
|
end
|