2013-09-13 05:46:43 +08:00
|
|
|
# encoding: utf-8
|
|
|
|
require 'spec_helper'
|
|
|
|
require 'composer_messages_finder'
|
|
|
|
|
|
|
|
describe ComposerMessagesFinder do
|
|
|
|
|
|
|
|
context "delegates work" do
|
|
|
|
let(:user) { Fabricate.build(:user) }
|
|
|
|
let(:finder) { ComposerMessagesFinder.new(user, composerAction: 'createTopic') }
|
|
|
|
|
|
|
|
it "calls all the message finders" do
|
|
|
|
finder.expects(:check_education_message).once
|
2013-12-20 02:45:55 +08:00
|
|
|
finder.expects(:check_new_user_many_replies).once
|
2013-09-13 05:46:43 +08:00
|
|
|
finder.expects(:check_avatar_notification).once
|
2013-09-14 01:49:34 +08:00
|
|
|
finder.expects(:check_sequential_replies).once
|
2013-09-18 02:38:39 +08:00
|
|
|
finder.expects(:check_dominating_topic).once
|
2014-03-12 22:44:08 +08:00
|
|
|
finder.expects(:check_reviving_old_topic).once
|
2013-09-13 05:46:43 +08:00
|
|
|
finder.find
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
context '.check_education_message' do
|
|
|
|
let(:user) { Fabricate.build(:user) }
|
|
|
|
|
|
|
|
context 'creating topic' do
|
|
|
|
let(:finder) { ComposerMessagesFinder.new(user, composerAction: 'createTopic') }
|
|
|
|
|
|
|
|
before do
|
|
|
|
SiteSetting.stubs(:educate_until_posts).returns(10)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "returns a message for a user who has not posted any topics" do
|
2013-09-18 00:11:17 +08:00
|
|
|
user.expects(:created_topic_count).returns(9)
|
2013-09-13 05:46:43 +08:00
|
|
|
finder.check_education_message.should be_present
|
|
|
|
end
|
|
|
|
|
|
|
|
it "returns no message when the user has posted enough topics" do
|
2013-09-18 00:11:17 +08:00
|
|
|
user.expects(:created_topic_count).returns(10)
|
2013-09-13 05:46:43 +08:00
|
|
|
finder.check_education_message.should be_blank
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'creating reply' do
|
|
|
|
let(:finder) { ComposerMessagesFinder.new(user, composerAction: 'reply') }
|
|
|
|
|
|
|
|
before do
|
|
|
|
SiteSetting.stubs(:educate_until_posts).returns(10)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "returns a message for a user who has not posted any topics" do
|
2013-09-18 00:11:17 +08:00
|
|
|
user.expects(:post_count).returns(9)
|
2013-09-13 05:46:43 +08:00
|
|
|
finder.check_education_message.should be_present
|
|
|
|
end
|
|
|
|
|
|
|
|
it "returns no message when the user has posted enough topics" do
|
2013-09-18 00:11:17 +08:00
|
|
|
user.expects(:post_count).returns(10)
|
2013-09-13 05:46:43 +08:00
|
|
|
finder.check_education_message.should be_blank
|
|
|
|
end
|
|
|
|
end
|
2013-12-20 02:45:55 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
context '.check_new_user_many_replies' do
|
|
|
|
let(:user) { Fabricate.build(:user) }
|
|
|
|
|
|
|
|
context 'replying' do
|
|
|
|
let(:finder) { ComposerMessagesFinder.new(user, composerAction: 'reply') }
|
|
|
|
|
|
|
|
it "has no message when `posted_too_much_in_topic?` is false" do
|
|
|
|
user.expects(:posted_too_much_in_topic?).returns(false)
|
|
|
|
finder.check_new_user_many_replies.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "has a message when a user has posted too much" do
|
|
|
|
user.expects(:posted_too_much_in_topic?).returns(true)
|
|
|
|
finder.check_new_user_many_replies.should be_present
|
|
|
|
end
|
|
|
|
end
|
2013-09-13 05:46:43 +08:00
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
context '.check_avatar_notification' do
|
|
|
|
let(:finder) { ComposerMessagesFinder.new(user, composerAction: 'createTopic') }
|
|
|
|
let(:user) { Fabricate(:user) }
|
|
|
|
|
|
|
|
context "a user who we haven't checked for an avatar yet" do
|
|
|
|
it "returns no avatar message" do
|
|
|
|
finder.check_avatar_notification.should be_blank
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "a user who has been checked for a custom avatar" do
|
|
|
|
before do
|
|
|
|
UserHistory.create!(action: UserHistory.actions[:checked_for_custom_avatar], target_user_id: user.id )
|
|
|
|
end
|
|
|
|
|
|
|
|
context "success" do
|
|
|
|
let!(:message) { finder.check_avatar_notification }
|
|
|
|
|
|
|
|
it "returns an avatar upgrade message" do
|
|
|
|
message.should be_present
|
|
|
|
end
|
|
|
|
|
|
|
|
it "creates a notified_about_avatar log" do
|
|
|
|
UserHistory.exists_for_user?(user, :notified_about_avatar).should be_true
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it "doesn't return notifications for new users" do
|
2014-09-05 13:20:39 +08:00
|
|
|
user.trust_level = TrustLevel[0]
|
2013-09-13 05:46:43 +08:00
|
|
|
finder.check_avatar_notification.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "doesn't return notifications for users who have custom avatars" do
|
2014-05-22 15:37:02 +08:00
|
|
|
user.uploaded_avatar_id = 1
|
2013-09-13 05:46:43 +08:00
|
|
|
finder.check_avatar_notification.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "doesn't notify users who have been notified already" do
|
|
|
|
UserHistory.create!(action: UserHistory.actions[:notified_about_avatar], target_user_id: user.id )
|
|
|
|
finder.check_avatar_notification.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
2013-09-14 01:49:34 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
context '.check_sequential_replies' do
|
|
|
|
let(:user) { Fabricate(:user) }
|
|
|
|
let(:topic) { Fabricate(:topic) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
SiteSetting.stubs(:educate_until_posts).returns(10)
|
2013-09-14 12:32:18 +08:00
|
|
|
user.stubs(:post_count).returns(11)
|
2013-09-14 01:49:34 +08:00
|
|
|
|
|
|
|
Fabricate(:post, topic: topic, user: user)
|
|
|
|
Fabricate(:post, topic: topic, user: user)
|
|
|
|
|
|
|
|
SiteSetting.stubs(:sequential_replies_threshold).returns(2)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "does not give a message for new topics" do
|
|
|
|
finder = ComposerMessagesFinder.new(user, composerAction: 'createTopic')
|
|
|
|
finder.check_sequential_replies.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "does not give a message without a topic id" do
|
|
|
|
ComposerMessagesFinder.new(user, composerAction: 'reply').check_sequential_replies.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
context "reply" do
|
|
|
|
let(:finder) { ComposerMessagesFinder.new(user, composerAction: 'reply', topic_id: topic.id) }
|
|
|
|
|
|
|
|
|
|
|
|
it "does not give a message to users who are still in the 'education' phase" do
|
2013-09-18 00:11:17 +08:00
|
|
|
user.stubs(:post_count).returns(9)
|
2013-09-14 01:49:34 +08:00
|
|
|
finder.check_sequential_replies.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "doesn't notify a user it has already notified about sequential replies" do
|
2013-09-18 02:38:39 +08:00
|
|
|
UserHistory.create!(action: UserHistory.actions[:notified_about_sequential_replies], target_user_id: user.id, topic_id: topic.id )
|
2013-09-14 01:49:34 +08:00
|
|
|
finder.check_sequential_replies.should be_blank
|
|
|
|
end
|
|
|
|
|
2013-09-18 02:38:39 +08:00
|
|
|
|
|
|
|
it "will notify you if it hasn't in the current topic" do
|
|
|
|
UserHistory.create!(action: UserHistory.actions[:notified_about_sequential_replies], target_user_id: user.id, topic_id: topic.id+1 )
|
|
|
|
finder.check_sequential_replies.should be_present
|
|
|
|
end
|
|
|
|
|
2013-09-14 01:49:34 +08:00
|
|
|
it "doesn't notify a user who has less than the `sequential_replies_threshold` threshold posts" do
|
|
|
|
SiteSetting.stubs(:sequential_replies_threshold).returns(5)
|
|
|
|
finder.check_sequential_replies.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "doesn't notify a user if another user posted" do
|
|
|
|
Fabricate(:post, topic: topic, user: Fabricate(:user))
|
|
|
|
finder.check_sequential_replies.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
context "success" do
|
|
|
|
let!(:message) { finder.check_sequential_replies }
|
|
|
|
|
|
|
|
it "returns a message" do
|
|
|
|
message.should be_present
|
|
|
|
end
|
|
|
|
|
|
|
|
it "creates a notified_about_sequential_replies log" do
|
|
|
|
UserHistory.exists_for_user?(user, :notified_about_sequential_replies).should be_true
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
end
|
2013-09-13 05:46:43 +08:00
|
|
|
|
|
|
|
end
|
|
|
|
|
2013-09-18 02:38:39 +08:00
|
|
|
context '.check_dominating_topic' do
|
|
|
|
let(:user) { Fabricate(:user) }
|
|
|
|
let(:topic) { Fabricate(:topic) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
SiteSetting.stubs(:educate_until_posts).returns(10)
|
|
|
|
user.stubs(:post_count).returns(11)
|
|
|
|
|
2013-11-19 01:48:26 +08:00
|
|
|
SiteSetting.stubs(:summary_posts_required).returns(1)
|
2013-09-18 02:38:39 +08:00
|
|
|
|
|
|
|
Fabricate(:post, topic: topic, user: user)
|
|
|
|
Fabricate(:post, topic: topic, user: user)
|
|
|
|
Fabricate(:post, topic: topic, user: Fabricate(:user))
|
|
|
|
|
|
|
|
SiteSetting.stubs(:sequential_replies_threshold).returns(2)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "does not give a message for new topics" do
|
|
|
|
finder = ComposerMessagesFinder.new(user, composerAction: 'createTopic')
|
|
|
|
finder.check_dominating_topic.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "does not give a message without a topic id" do
|
|
|
|
ComposerMessagesFinder.new(user, composerAction: 'reply').check_dominating_topic.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
context "reply" do
|
|
|
|
let(:finder) { ComposerMessagesFinder.new(user, composerAction: 'reply', topic_id: topic.id) }
|
|
|
|
|
|
|
|
it "does not give a message to users who are still in the 'education' phase" do
|
|
|
|
user.stubs(:post_count).returns(9)
|
|
|
|
finder.check_dominating_topic.should be_blank
|
|
|
|
end
|
|
|
|
|
2013-11-19 01:48:26 +08:00
|
|
|
it "does not notify if the `summary_posts_required` has not been reached" do
|
|
|
|
SiteSetting.stubs(:summary_posts_required).returns(100)
|
2013-09-18 02:38:39 +08:00
|
|
|
finder.check_dominating_topic.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "doesn't notify a user it has already notified in this topic" do
|
2014-02-07 08:19:45 +08:00
|
|
|
UserHistory.create!(action: UserHistory.actions[:notified_about_dominating_topic], topic_id: topic.id, target_user_id: user.id )
|
2013-09-18 02:38:39 +08:00
|
|
|
finder.check_dominating_topic.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "notifies a user if the topic is different" do
|
2014-02-07 08:19:45 +08:00
|
|
|
UserHistory.create!(action: UserHistory.actions[:notified_about_dominating_topic], topic_id: topic.id+1, target_user_id: user.id )
|
2013-09-18 02:38:39 +08:00
|
|
|
finder.check_dominating_topic.should be_present
|
|
|
|
end
|
|
|
|
|
2013-11-19 01:48:26 +08:00
|
|
|
it "doesn't notify a user if the topic has less than `summary_posts_required` posts" do
|
|
|
|
SiteSetting.stubs(:summary_posts_required).returns(5)
|
2013-09-18 02:38:39 +08:00
|
|
|
finder.check_dominating_topic.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "doesn't notify a user if they've posted less than the percentage" do
|
|
|
|
SiteSetting.stubs(:dominating_topic_minimum_percent).returns(100)
|
|
|
|
finder.check_dominating_topic.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "doesn't notify you if it's your own topic" do
|
|
|
|
topic.update_column(:user_id, user.id)
|
|
|
|
finder.check_dominating_topic.should be_blank
|
|
|
|
end
|
|
|
|
|
2014-02-07 08:19:45 +08:00
|
|
|
it "doesn't notify you in a private message" do
|
2014-09-11 15:39:20 +08:00
|
|
|
topic.update_columns(category_id: nil, archetype: Archetype.private_message)
|
2014-02-07 08:19:45 +08:00
|
|
|
finder.check_dominating_topic.should be_blank
|
|
|
|
end
|
|
|
|
|
2013-09-18 02:38:39 +08:00
|
|
|
context "success" do
|
|
|
|
let!(:message) { finder.check_dominating_topic }
|
|
|
|
|
|
|
|
it "returns a message" do
|
|
|
|
message.should be_present
|
|
|
|
end
|
|
|
|
|
2014-02-07 08:19:45 +08:00
|
|
|
it "creates a notified_about_dominating_topic log" do
|
|
|
|
UserHistory.exists_for_user?(user, :notified_about_dominating_topic).should be_true
|
2013-09-18 02:38:39 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
2014-03-12 22:44:08 +08:00
|
|
|
context '.check_reviving_old_topic' do
|
|
|
|
let(:user) { Fabricate(:user) }
|
|
|
|
let(:topic) { Fabricate(:topic) }
|
|
|
|
|
|
|
|
it "does not give a message without a topic id" do
|
|
|
|
described_class.new(user, composerAction: 'createTopic').check_reviving_old_topic.should be_blank
|
|
|
|
described_class.new(user, composerAction: 'reply').check_reviving_old_topic.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
context "a reply" do
|
|
|
|
context "warn_reviving_old_topic_age is 180 days" do
|
|
|
|
before do
|
|
|
|
SiteSetting.stubs(:warn_reviving_old_topic_age).returns(180)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "does not notify if last post is recent" do
|
|
|
|
topic = Fabricate(:topic, last_posted_at: 1.hour.ago)
|
|
|
|
described_class.new(user, composerAction: 'reply', topic_id: topic.id).check_reviving_old_topic.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "notifies if last post is old" do
|
|
|
|
topic = Fabricate(:topic, last_posted_at: 181.days.ago)
|
|
|
|
described_class.new(user, composerAction: 'reply', topic_id: topic.id).check_reviving_old_topic.should_not be_blank
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "warn_reviving_old_topic_age is 0" do
|
|
|
|
before do
|
|
|
|
SiteSetting.stubs(:warn_reviving_old_topic_age).returns(0)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "does not notify if last post is new" do
|
|
|
|
topic = Fabricate(:topic, last_posted_at: 1.hour.ago)
|
|
|
|
described_class.new(user, composerAction: 'reply', topic_id: topic.id).check_reviving_old_topic.should be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "does not notify if last post is old" do
|
|
|
|
topic = Fabricate(:topic, last_posted_at: 365.days.ago)
|
|
|
|
described_class.new(user, composerAction: 'reply', topic_id: topic.id).check_reviving_old_topic.should be_blank
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2013-09-13 05:46:43 +08:00
|
|
|
end
|
|
|
|
|