2019-04-29 20:27:42 -04:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2015-10-11 05:41:23 -04:00
|
|
|
require "rails_helper"
|
2014-11-24 11:40:21 -05:00
|
|
|
|
|
|
|
describe Jobs::NotifyMailingListSubscribers do
|
|
|
|
|
2019-05-06 23:12:20 -04:00
|
|
|
fab!(:mailing_list_user) { Fabricate(:user) }
|
2016-07-05 06:20:07 -04:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
before { mailing_list_user.user_option.update(mailing_list_mode: true, mailing_list_mode_frequency: 1) }
|
2020-02-19 15:14:42 -05:00
|
|
|
before do
|
|
|
|
SiteSetting.tagging_enabled = true
|
|
|
|
end
|
2016-07-05 06:20:07 -04:00
|
|
|
|
2020-02-19 15:14:42 -05:00
|
|
|
fab!(:tag) { Fabricate(:tag) }
|
|
|
|
fab!(:topic) { Fabricate(:topic, tags: [tag]) }
|
2019-05-06 23:12:20 -04:00
|
|
|
fab!(:user) { Fabricate(:user) }
|
2020-02-19 15:14:42 -05:00
|
|
|
fab!(:post) { Fabricate(:post, topic: topic, user: user) }
|
2016-12-12 09:28:26 -05:00
|
|
|
|
|
|
|
shared_examples "no emails" do
|
|
|
|
it "doesn't send any emails" do
|
|
|
|
UserNotifications.expects(:mailing_list_notify).with(mailing_list_user, post).never
|
|
|
|
Jobs::NotifyMailingListSubscribers.new.execute(post_id: post.id)
|
2016-05-26 22:35:22 -04:00
|
|
|
end
|
2016-12-12 09:28:26 -05:00
|
|
|
end
|
2014-11-24 11:40:21 -05:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
shared_examples "one email" do
|
|
|
|
it "sends the email" do
|
|
|
|
UserNotifications.expects(:mailing_list_notify).with(mailing_list_user, post).once
|
|
|
|
Jobs::NotifyMailingListSubscribers.new.execute(post_id: post.id)
|
|
|
|
end
|
2017-07-19 16:51:32 -04:00
|
|
|
|
|
|
|
it "triggers :notify_mailing_list_subscribers" do
|
|
|
|
events = DiscourseEvent.track_events do
|
|
|
|
Jobs::NotifyMailingListSubscribers.new.execute(post_id: post.id)
|
|
|
|
end
|
2017-07-27 21:20:09 -04:00
|
|
|
expect(events).to include(event_name: :notify_mailing_list_subscribers, params: [[mailing_list_user], post])
|
2017-07-19 16:51:32 -04:00
|
|
|
end
|
2016-12-12 09:28:26 -05:00
|
|
|
end
|
2016-03-23 00:08:34 -04:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "when mailing list mode is globally disabled" do
|
|
|
|
before { SiteSetting.disable_mailing_list_mode = true }
|
|
|
|
include_examples "no emails"
|
|
|
|
end
|
2016-03-23 00:08:34 -04:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "when mailing list mode is globally enabled" do
|
|
|
|
before { SiteSetting.disable_mailing_list_mode = false }
|
2016-03-23 00:08:34 -04:00
|
|
|
|
2017-12-12 23:13:17 -05:00
|
|
|
context "when site requires approval and user is not approved" do
|
|
|
|
before do
|
|
|
|
SiteSetting.login_required = true
|
|
|
|
SiteSetting.must_approve_users = true
|
2019-04-16 14:42:47 -04:00
|
|
|
|
|
|
|
User.update_all(approved: false)
|
2017-12-12 23:13:17 -05:00
|
|
|
end
|
|
|
|
include_examples "no emails"
|
|
|
|
end
|
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "with an invalid post_id" do
|
2017-05-08 15:08:29 -04:00
|
|
|
before { post.update(deleted_at: Time.now) }
|
|
|
|
include_examples "no emails"
|
2016-03-23 00:08:34 -04:00
|
|
|
end
|
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "with a deleted post" do
|
|
|
|
before { post.update(deleted_at: Time.now) }
|
|
|
|
include_examples "no emails"
|
|
|
|
end
|
2016-07-05 06:20:07 -04:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "with a user_deleted post" do
|
|
|
|
before { post.update(user_deleted: true) }
|
|
|
|
include_examples "no emails"
|
2016-07-05 06:20:07 -04:00
|
|
|
end
|
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "with a deleted topic" do
|
|
|
|
before { post.topic.update(deleted_at: Time.now) }
|
|
|
|
include_examples "no emails"
|
|
|
|
end
|
2016-03-23 00:08:34 -04:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "with a valid post from another user" do
|
2016-03-23 00:08:34 -04:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "to an inactive user" do
|
|
|
|
before { mailing_list_user.update(active: false) }
|
|
|
|
include_examples "no emails"
|
2016-03-23 00:08:34 -04:00
|
|
|
end
|
|
|
|
|
2017-11-10 12:18:08 -05:00
|
|
|
context "to a silenced user" do
|
2017-11-13 13:41:36 -05:00
|
|
|
before { mailing_list_user.update(silenced_till: 1.year.from_now) }
|
2016-12-12 09:28:26 -05:00
|
|
|
include_examples "no emails"
|
|
|
|
end
|
2015-08-21 14:39:21 -04:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "to a suspended user" do
|
|
|
|
before { mailing_list_user.update(suspended_till: 1.day.from_now) }
|
|
|
|
include_examples "no emails"
|
2016-10-05 19:28:58 -04:00
|
|
|
end
|
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "to an anonymous user" do
|
2019-05-06 23:12:20 -04:00
|
|
|
fab!(:mailing_list_user) { Fabricate(:anonymous) }
|
2016-12-12 09:28:26 -05:00
|
|
|
include_examples "no emails"
|
2014-11-24 11:40:21 -05:00
|
|
|
end
|
2016-05-21 09:17:54 -04:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "to an user who has disabled mailing list mode" do
|
|
|
|
before { mailing_list_user.user_option.update(mailing_list_mode: false) }
|
|
|
|
include_examples "no emails"
|
2016-10-05 19:28:58 -04:00
|
|
|
end
|
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "to an user who has frequency set to 'always'" do
|
|
|
|
before { mailing_list_user.user_option.update(mailing_list_mode_frequency: 1) }
|
|
|
|
include_examples "one email"
|
2016-10-05 19:28:58 -04:00
|
|
|
end
|
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "to an user who has frequency set to 'no echo'" do
|
|
|
|
before { mailing_list_user.user_option.update(mailing_list_mode_frequency: 2) }
|
|
|
|
include_examples "one email"
|
2016-10-05 19:28:58 -04:00
|
|
|
end
|
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "from a muted user" do
|
|
|
|
before { MutedUser.create(user: mailing_list_user, muted_user: user) }
|
|
|
|
include_examples "no emails"
|
2016-05-21 09:17:54 -04:00
|
|
|
end
|
2014-11-24 11:40:21 -05:00
|
|
|
|
2019-03-21 07:15:34 -04:00
|
|
|
context "from an ignored user" do
|
|
|
|
before { Fabricate(:ignored_user, user: mailing_list_user, ignored_user: user) }
|
|
|
|
include_examples "no emails"
|
|
|
|
end
|
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "from a muted topic" do
|
|
|
|
before { TopicUser.create(user: mailing_list_user, topic: post.topic, notification_level: TopicUser.notification_levels[:muted]) }
|
|
|
|
include_examples "no emails"
|
|
|
|
end
|
2014-11-24 11:40:21 -05:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "from a muted category" do
|
|
|
|
before { CategoryUser.create(user: mailing_list_user, category: post.topic.category, notification_level: CategoryUser.notification_levels[:muted]) }
|
|
|
|
include_examples "no emails"
|
2014-11-24 11:40:21 -05:00
|
|
|
end
|
|
|
|
|
2020-02-19 15:14:42 -05:00
|
|
|
context "from a muted tag" do
|
|
|
|
before { TagUser.create(user: mailing_list_user, tag: tag, notification_level: TagUser.notification_levels[:muted]) }
|
|
|
|
include_examples "no emails"
|
|
|
|
end
|
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "max emails per day was reached" do
|
|
|
|
before { SiteSetting.max_emails_per_day_per_user = 2 }
|
2014-11-24 11:40:21 -05:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
it "doesn't send any emails" do
|
|
|
|
(SiteSetting.max_emails_per_day_per_user + 1).times {
|
|
|
|
mailing_list_user.email_logs.create(email_type: 'foobar', to_address: mailing_list_user.email)
|
|
|
|
}
|
2014-11-24 11:40:21 -05:00
|
|
|
|
2018-08-08 04:25:00 -04:00
|
|
|
expect do
|
|
|
|
UserNotifications.expects(:mailing_list_notify)
|
|
|
|
.with(mailing_list_user, post)
|
|
|
|
.never
|
|
|
|
|
|
|
|
2.times do
|
|
|
|
Jobs::NotifyMailingListSubscribers.new.execute(post_id: post.id)
|
|
|
|
end
|
2018-08-20 22:04:35 -04:00
|
|
|
|
|
|
|
Jobs::NotifyMailingListSubscribers.new.execute(
|
2018-08-20 23:14:43 -04:00
|
|
|
post_id: Fabricate(:post, user: user).id
|
2018-08-20 22:04:35 -04:00
|
|
|
)
|
2018-08-08 04:25:00 -04:00
|
|
|
end.to change { SkippedEmailLog.count }.by(1)
|
2015-08-21 14:39:21 -04:00
|
|
|
|
2018-07-24 00:55:43 -04:00
|
|
|
expect(SkippedEmailLog.exists?(
|
|
|
|
email_type: "mailing_list",
|
|
|
|
user: mailing_list_user,
|
|
|
|
post: post,
|
|
|
|
to_address: mailing_list_user.email,
|
|
|
|
reason_type: SkippedEmailLog.reason_types[:exceeded_emails_limit]
|
|
|
|
)).to eq(true)
|
2018-08-20 22:04:35 -04:00
|
|
|
|
2018-08-20 23:14:43 -04:00
|
|
|
freeze_time(Time.zone.now.tomorrow + 1.second)
|
2018-08-20 22:04:35 -04:00
|
|
|
|
|
|
|
expect do
|
2018-08-20 23:14:43 -04:00
|
|
|
post = Fabricate(:post, user: user)
|
|
|
|
|
|
|
|
UserNotifications.expects(:mailing_list_notify)
|
|
|
|
.with(mailing_list_user, post)
|
|
|
|
.once
|
|
|
|
|
2018-08-20 22:04:35 -04:00
|
|
|
Jobs::NotifyMailingListSubscribers.new.execute(
|
2018-08-20 23:14:43 -04:00
|
|
|
post_id: post.id
|
2018-08-20 22:04:35 -04:00
|
|
|
)
|
2018-08-20 23:14:43 -04:00
|
|
|
end.to change { SkippedEmailLog.count }.by(0)
|
2016-12-12 09:28:26 -05:00
|
|
|
end
|
2014-11-24 11:40:21 -05:00
|
|
|
end
|
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "bounce score was reached" do
|
2015-08-21 14:39:21 -04:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
it "doesn't send any emails" do
|
|
|
|
mailing_list_user.user_stat.update(bounce_score: SiteSetting.bounce_score_threshold + 1)
|
|
|
|
|
|
|
|
Jobs::NotifyMailingListSubscribers.new.execute(post_id: post.id)
|
|
|
|
UserNotifications.expects(:mailing_list_notify).with(mailing_list_user, post).never
|
|
|
|
|
2018-07-24 00:55:43 -04:00
|
|
|
expect(SkippedEmailLog.exists?(
|
|
|
|
email_type: "mailing_list",
|
|
|
|
user: mailing_list_user,
|
|
|
|
post: post,
|
|
|
|
to_address: mailing_list_user.email,
|
|
|
|
reason_type: SkippedEmailLog.reason_types[:exceeded_bounces_limit]
|
|
|
|
)).to eq(true)
|
2016-12-12 09:28:26 -05:00
|
|
|
end
|
2015-08-21 14:39:21 -04:00
|
|
|
|
2014-11-24 11:40:21 -05:00
|
|
|
end
|
2016-12-12 09:28:26 -05:00
|
|
|
|
2014-11-24 11:40:21 -05:00
|
|
|
end
|
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "with a valid post from same user" do
|
2019-05-06 23:12:20 -04:00
|
|
|
fab!(:post) { Fabricate(:post, user: mailing_list_user) }
|
2014-11-24 11:40:21 -05:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "to an user who has frequency set to 'daily'" do
|
|
|
|
before { mailing_list_user.user_option.update(mailing_list_mode_frequency: 0) }
|
|
|
|
include_examples "no emails"
|
|
|
|
end
|
2015-08-21 14:39:21 -04:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "to an user who has frequency set to 'always'" do
|
|
|
|
before { mailing_list_user.user_option.update(mailing_list_mode_frequency: 1) }
|
|
|
|
include_examples "one email"
|
|
|
|
end
|
2014-11-24 11:40:21 -05:00
|
|
|
|
2016-12-12 09:28:26 -05:00
|
|
|
context "to an user who has frequency set to 'no echo'" do
|
|
|
|
before { mailing_list_user.user_option.update(mailing_list_mode_frequency: 2) }
|
|
|
|
include_examples "no emails"
|
|
|
|
end
|
2014-11-24 11:40:21 -05:00
|
|
|
end
|
2016-12-12 09:28:26 -05:00
|
|
|
|
2014-11-24 11:40:21 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
end
|