2019-04-29 20:27:42 -04:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2019-01-03 12:03:01 -05:00
|
|
|
require 'rails_helper'
|
|
|
|
|
|
|
|
RSpec.describe ReviewableFlaggedPost, type: :model do
|
|
|
|
|
|
|
|
def pending_count
|
|
|
|
ReviewableFlaggedPost.default_visible.pending.count
|
|
|
|
end
|
|
|
|
|
2019-05-06 23:12:20 -04:00
|
|
|
fab!(:user) { Fabricate(:user) }
|
|
|
|
fab!(:post) { Fabricate(:post) }
|
|
|
|
fab!(:moderator) { Fabricate(:moderator) }
|
2019-01-03 12:03:01 -05:00
|
|
|
|
|
|
|
it "sets `potential_spam` when a spam flag is added" do
|
|
|
|
reviewable = PostActionCreator.off_topic(user, post).reviewable
|
|
|
|
expect(reviewable.potential_spam?).to eq(false)
|
|
|
|
PostActionCreator.spam(Fabricate(:user), post)
|
|
|
|
expect(reviewable.reload.potential_spam?).to eq(true)
|
|
|
|
end
|
|
|
|
|
|
|
|
describe "actions" do
|
|
|
|
let!(:result) { PostActionCreator.spam(user, post) }
|
|
|
|
let(:reviewable) { result.reviewable }
|
|
|
|
let(:score) { result.reviewable_score }
|
|
|
|
let(:guardian) { Guardian.new(moderator) }
|
|
|
|
|
|
|
|
describe "actions_for" do
|
2019-05-08 10:20:51 -04:00
|
|
|
|
2019-01-03 12:03:01 -05:00
|
|
|
it "returns appropriate defaults" do
|
|
|
|
actions = reviewable.actions_for(guardian)
|
|
|
|
expect(actions.has?(:agree_and_hide)).to eq(true)
|
|
|
|
expect(actions.has?(:agree_and_keep)).to eq(true)
|
2019-08-01 11:23:23 -04:00
|
|
|
expect(actions.has?(:agree_and_keep_hidden)).to eq(false)
|
2019-01-03 12:03:01 -05:00
|
|
|
expect(actions.has?(:agree_and_silence)).to eq(true)
|
|
|
|
expect(actions.has?(:agree_and_suspend)).to eq(true)
|
|
|
|
expect(actions.has?(:delete_spammer)).to eq(true)
|
|
|
|
expect(actions.has?(:disagree)).to eq(true)
|
|
|
|
expect(actions.has?(:ignore)).to eq(true)
|
|
|
|
expect(actions.has?(:delete_and_ignore)).to eq(true)
|
2019-04-04 15:51:36 -04:00
|
|
|
expect(actions.has?(:delete_and_ignore_replies)).to eq(false)
|
2019-01-03 12:03:01 -05:00
|
|
|
expect(actions.has?(:delete_and_agree)).to eq(true)
|
2019-04-04 15:51:36 -04:00
|
|
|
expect(actions.has?(:delete_and_replies)).to eq(false)
|
2019-01-03 12:03:01 -05:00
|
|
|
|
|
|
|
expect(actions.has?(:disagree_and_restore)).to eq(false)
|
|
|
|
end
|
|
|
|
|
2019-05-30 16:42:59 -04:00
|
|
|
it "doesn't include deletes for category topics" do
|
2019-08-06 06:26:54 -04:00
|
|
|
c = Fabricate(:category_with_definition)
|
2019-05-30 16:42:59 -04:00
|
|
|
flag = PostActionCreator.spam(user, c.topic.posts.first).reviewable
|
|
|
|
actions = flag.actions_for(guardian)
|
|
|
|
expect(actions.has?(:delete_and_ignore)).to eq(false)
|
|
|
|
expect(actions.has?(:delete_and_ignore_replies)).to eq(false)
|
|
|
|
expect(actions.has?(:delete_and_agree)).to eq(false)
|
|
|
|
expect(actions.has?(:delete_and_replies)).to eq(false)
|
|
|
|
end
|
|
|
|
|
2019-08-01 11:23:23 -04:00
|
|
|
it "changes `agree_and_keep` to `agree_and_keep_hidden` if it's been hidden" do
|
|
|
|
post.hidden = true
|
|
|
|
actions = reviewable.actions_for(guardian)
|
|
|
|
expect(actions.has?(:agree_and_keep)).to eq(false)
|
|
|
|
expect(actions.has?(:agree_and_keep_hidden)).to eq(true)
|
|
|
|
end
|
|
|
|
|
2019-01-03 12:03:01 -05:00
|
|
|
it "returns `agree_and_restore` if the post is user deleted" do
|
|
|
|
post.update(user_deleted: true)
|
|
|
|
expect(reviewable.actions_for(guardian).has?(:agree_and_restore)).to eq(true)
|
|
|
|
end
|
|
|
|
|
2019-04-04 15:51:36 -04:00
|
|
|
it "returns delete replies options if there are replies" do
|
|
|
|
post.update(reply_count: 3)
|
|
|
|
expect(reviewable.actions_for(guardian).has?(:delete_and_agree_replies)).to eq(true)
|
|
|
|
expect(reviewable.actions_for(guardian).has?(:delete_and_ignore_replies)).to eq(true)
|
|
|
|
end
|
|
|
|
|
2019-01-03 12:03:01 -05:00
|
|
|
it "returns appropriate actions for a hidden post" do
|
|
|
|
post.update(hidden: true, hidden_at: Time.now)
|
|
|
|
expect(reviewable.actions_for(guardian).has?(:agree_and_hide)).to eq(false)
|
|
|
|
expect(reviewable.actions_for(guardian).has?(:disagree_and_restore)).to eq(true)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "won't return the penalty options if the user is not regular" do
|
|
|
|
post.user.update(moderator: true)
|
|
|
|
expect(reviewable.actions_for(guardian).has?(:agree_and_silence)).to eq(false)
|
|
|
|
expect(reviewable.actions_for(guardian).has?(:agree_and_suspend)).to eq(false)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it "agree_and_keep agrees with the flags and keeps the post" do
|
|
|
|
reviewable.perform(moderator, :agree_and_keep)
|
|
|
|
expect(reviewable).to be_approved
|
|
|
|
expect(score.reload).to be_agreed
|
|
|
|
expect(post).not_to be_hidden
|
|
|
|
end
|
|
|
|
|
|
|
|
it "agree_and_suspend agrees with the flags and keeps the post" do
|
|
|
|
reviewable.perform(moderator, :agree_and_suspend)
|
|
|
|
expect(reviewable).to be_approved
|
|
|
|
expect(score.reload).to be_agreed
|
|
|
|
expect(post).not_to be_hidden
|
|
|
|
end
|
|
|
|
|
|
|
|
it "agree_and_silence agrees with the flags and keeps the post" do
|
|
|
|
reviewable.perform(moderator, :agree_and_silence)
|
|
|
|
expect(reviewable).to be_approved
|
|
|
|
expect(score.reload).to be_agreed
|
|
|
|
expect(post).not_to be_hidden
|
|
|
|
end
|
|
|
|
|
|
|
|
it "agree_and_hide agrees with the flags and hides the post" do
|
|
|
|
reviewable.perform(moderator, :agree_and_hide)
|
|
|
|
expect(reviewable).to be_approved
|
|
|
|
expect(score.reload).to be_agreed
|
|
|
|
expect(post).to be_hidden
|
|
|
|
end
|
|
|
|
|
|
|
|
it "agree_and_restore agrees with the flags and restores the post" do
|
|
|
|
post.update(user_deleted: true)
|
|
|
|
reviewable.perform(moderator, :agree_and_restore)
|
|
|
|
expect(reviewable).to be_approved
|
|
|
|
expect(score.reload).to be_agreed
|
|
|
|
expect(post.user_deleted?).to eq(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "supports deleting a spammer" do
|
|
|
|
reviewable.perform(moderator, :delete_spammer)
|
|
|
|
expect(reviewable).to be_approved
|
|
|
|
expect(score.reload).to be_agreed
|
|
|
|
expect(post.reload.deleted_at).to be_present
|
|
|
|
expect(User.find_by(id: reviewable.target_created_by_id)).to be_blank
|
|
|
|
end
|
|
|
|
|
|
|
|
it "ignores the flags" do
|
|
|
|
reviewable.perform(moderator, :ignore)
|
|
|
|
expect(reviewable).to be_ignored
|
|
|
|
expect(score.reload).to be_ignored
|
|
|
|
end
|
|
|
|
|
2019-04-04 15:51:36 -04:00
|
|
|
it "delete_and_ignore ignores the flags and deletes post" do
|
2019-01-03 12:03:01 -05:00
|
|
|
reviewable.perform(moderator, :delete_and_ignore)
|
|
|
|
expect(reviewable).to be_ignored
|
|
|
|
expect(score.reload).to be_ignored
|
|
|
|
expect(post.reload.deleted_at).to be_present
|
|
|
|
end
|
|
|
|
|
2019-04-04 15:51:36 -04:00
|
|
|
it "delete_and_ignore_replies ignores the flags and deletes post + replies" do
|
|
|
|
reply = PostCreator.create(
|
|
|
|
Fabricate(:user),
|
|
|
|
raw: 'this is the reply text',
|
|
|
|
reply_to_post_number: post.post_number,
|
|
|
|
topic_id: post.topic_id
|
|
|
|
)
|
2019-04-30 12:54:53 -04:00
|
|
|
nested_reply = PostCreator.create(
|
|
|
|
Fabricate(:user),
|
|
|
|
raw: 'this is the reply text2',
|
|
|
|
reply_to_post_number: reply.post_number,
|
|
|
|
topic_id: post.topic_id
|
|
|
|
)
|
2019-04-04 15:51:36 -04:00
|
|
|
post.reload
|
|
|
|
|
|
|
|
reviewable.perform(moderator, :delete_and_ignore_replies)
|
|
|
|
expect(reviewable).to be_ignored
|
|
|
|
expect(score.reload).to be_ignored
|
|
|
|
expect(post.reload.deleted_at).to be_present
|
|
|
|
expect(reply.reload.deleted_at).to be_present
|
2019-04-30 12:54:53 -04:00
|
|
|
expect(nested_reply.reload.deleted_at).to be_present
|
2019-04-04 15:51:36 -04:00
|
|
|
end
|
|
|
|
|
2019-01-03 12:03:01 -05:00
|
|
|
it "delete_and_agree agrees with the flags and deletes post" do
|
|
|
|
reviewable.perform(moderator, :delete_and_agree)
|
|
|
|
expect(reviewable).to be_approved
|
|
|
|
expect(score.reload).to be_agreed
|
|
|
|
expect(post.reload.deleted_at).to be_present
|
|
|
|
end
|
|
|
|
|
2019-04-04 15:51:36 -04:00
|
|
|
it "delete_and_agree_replies agrees w/ the flags and deletes post + replies" do
|
|
|
|
reply = PostCreator.create(
|
|
|
|
Fabricate(:user),
|
|
|
|
raw: 'this is the reply text',
|
|
|
|
reply_to_post_number: post.post_number,
|
|
|
|
topic_id: post.topic_id
|
|
|
|
)
|
2019-04-30 12:54:53 -04:00
|
|
|
nested_reply = PostCreator.create(
|
|
|
|
Fabricate(:user),
|
|
|
|
raw: 'this is the reply text2',
|
|
|
|
reply_to_post_number: reply.post_number,
|
|
|
|
topic_id: post.topic_id
|
|
|
|
)
|
2019-04-04 15:51:36 -04:00
|
|
|
post.reload
|
|
|
|
|
|
|
|
reviewable.perform(moderator, :delete_and_agree_replies)
|
|
|
|
expect(reviewable).to be_approved
|
|
|
|
expect(score.reload).to be_agreed
|
|
|
|
expect(post.reload.deleted_at).to be_present
|
|
|
|
expect(reply.reload.deleted_at).to be_present
|
2019-04-30 12:54:53 -04:00
|
|
|
expect(nested_reply.reload.deleted_at).to be_present
|
2019-04-04 15:51:36 -04:00
|
|
|
end
|
|
|
|
|
2019-01-03 12:03:01 -05:00
|
|
|
it "disagrees with the flags" do
|
|
|
|
reviewable.perform(moderator, :disagree)
|
|
|
|
expect(reviewable).to be_rejected
|
|
|
|
expect(score.reload).to be_disagreed
|
|
|
|
end
|
|
|
|
|
|
|
|
it "disagrees with the flags and restores the post" do
|
|
|
|
post.update(hidden: true, hidden_at: Time.now)
|
|
|
|
reviewable.perform(moderator, :disagree_and_restore)
|
|
|
|
expect(reviewable).to be_rejected
|
|
|
|
expect(score.reload).to be_disagreed
|
|
|
|
expect(post.user_deleted?).to eq(false)
|
2019-04-11 16:21:00 -04:00
|
|
|
expect(post.hidden?).to eq(false)
|
2019-01-03 12:03:01 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
describe "pending count" do
|
|
|
|
it "increments the numbers correctly" do
|
|
|
|
expect(pending_count).to eq(0)
|
|
|
|
|
|
|
|
result = PostActionCreator.off_topic(user, post)
|
|
|
|
expect(pending_count).to eq(1)
|
|
|
|
|
|
|
|
result.reviewable.perform(Discourse.system_user, :disagree)
|
|
|
|
expect(pending_count).to eq(0)
|
|
|
|
end
|
|
|
|
|
2019-05-07 13:25:11 -04:00
|
|
|
it "respects `reviewable_default_visibility`" do
|
|
|
|
Reviewable.set_priorities(high: 7.5)
|
|
|
|
SiteSetting.reviewable_default_visibility = 'high'
|
2019-01-03 12:03:01 -05:00
|
|
|
expect(pending_count).to eq(0)
|
|
|
|
|
|
|
|
PostActionCreator.off_topic(user, post)
|
|
|
|
expect(pending_count).to eq(0)
|
|
|
|
|
|
|
|
PostActionCreator.spam(moderator, post)
|
|
|
|
expect(pending_count).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "should reset counts when a topic is deleted" do
|
|
|
|
PostActionCreator.off_topic(user, post)
|
|
|
|
expect(pending_count).to eq(1)
|
|
|
|
|
|
|
|
PostDestroyer.new(moderator, post).destroy
|
|
|
|
expect(pending_count).to eq(0)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "should not review non-human users" do
|
|
|
|
post = create_post(user: Discourse.system_user)
|
|
|
|
reviewable = PostActionCreator.off_topic(user, post).reviewable
|
|
|
|
expect(reviewable).to be_blank
|
|
|
|
expect(pending_count).to eq(0)
|
|
|
|
end
|
|
|
|
|
|
|
|
it "should ignore handled flags" do
|
|
|
|
post = create_post
|
|
|
|
reviewable = PostActionCreator.off_topic(user, post).reviewable
|
|
|
|
expect(post.hidden).to eq(false)
|
|
|
|
expect(post.hidden_at).to be_blank
|
|
|
|
|
|
|
|
reviewable.perform(moderator, :ignore)
|
|
|
|
expect(pending_count).to eq(0)
|
|
|
|
|
|
|
|
post.reload
|
|
|
|
expect(post.hidden).to eq(false)
|
|
|
|
expect(post.hidden_at).to be_blank
|
|
|
|
|
|
|
|
post.hide!(PostActionType.types[:off_topic])
|
|
|
|
|
|
|
|
post.reload
|
|
|
|
expect(post.hidden).to eq(true)
|
|
|
|
expect(post.hidden_at).to be_present
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|