2013-03-20 02:15:08 +08:00
|
|
|
require 'spec_helper'
|
|
|
|
require 'post_destroyer'
|
|
|
|
|
|
|
|
describe PostDestroyer do
|
|
|
|
|
2013-05-14 09:59:55 +08:00
|
|
|
before do
|
|
|
|
ActiveRecord::Base.observers.enable :all
|
|
|
|
end
|
|
|
|
|
2013-03-20 02:15:08 +08:00
|
|
|
let(:moderator) { Fabricate(:moderator) }
|
|
|
|
let(:post) { Fabricate(:post) }
|
|
|
|
|
|
|
|
describe 'basic destroying' do
|
|
|
|
|
|
|
|
let(:moderator) { Fabricate(:moderator) }
|
2013-05-03 15:56:23 +08:00
|
|
|
let(:admin) { Fabricate(:admin) }
|
2013-03-20 02:15:08 +08:00
|
|
|
|
|
|
|
context "as the creator of the post" do
|
|
|
|
before do
|
|
|
|
PostDestroyer.new(post.user, post).destroy
|
|
|
|
post.reload
|
|
|
|
end
|
|
|
|
|
|
|
|
it "doesn't delete the post" do
|
|
|
|
post.deleted_at.should be_blank
|
|
|
|
post.raw.should == I18n.t('js.post.deleted_by_author')
|
|
|
|
post.version.should == 2
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "as a moderator" do
|
|
|
|
before do
|
|
|
|
PostDestroyer.new(moderator, post).destroy
|
|
|
|
end
|
|
|
|
|
|
|
|
it "deletes the post" do
|
|
|
|
post.deleted_at.should be_present
|
|
|
|
end
|
|
|
|
end
|
2013-05-14 09:59:55 +08:00
|
|
|
|
2013-05-03 15:56:23 +08:00
|
|
|
context "as an admin" do
|
|
|
|
before do
|
|
|
|
PostDestroyer.new(admin, post).destroy
|
|
|
|
end
|
|
|
|
|
|
|
|
it "deletes the post" do
|
|
|
|
post.deleted_at.should be_present
|
|
|
|
end
|
|
|
|
end
|
2013-03-20 02:15:08 +08:00
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'deleting the second post in a topic' do
|
|
|
|
|
|
|
|
let(:user) { Fabricate(:user) }
|
|
|
|
let!(:post) { Fabricate(:post, user: user) }
|
|
|
|
let(:topic) { post.topic }
|
|
|
|
let(:second_user) { Fabricate(:coding_horror) }
|
|
|
|
let!(:second_post) { Fabricate(:post, topic: topic, user: second_user) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
PostDestroyer.new(moderator, second_post).destroy
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'resets the last_poster_id back to the OP' do
|
|
|
|
topic.last_post_user_id.should == user.id
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'resets the last_posted_at back to the OP' do
|
|
|
|
topic.last_posted_at.to_i.should == post.created_at.to_i
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'topic_user' do
|
|
|
|
|
|
|
|
let(:topic_user) { second_user.topic_users.where(topic_id: topic.id).first }
|
|
|
|
|
|
|
|
it 'clears the posted flag for the second user' do
|
|
|
|
topic_user.posted?.should be_false
|
|
|
|
end
|
|
|
|
|
|
|
|
it "sets the second user's last_read_post_number back to 1" do
|
|
|
|
topic_user.last_read_post_number.should == 1
|
|
|
|
end
|
|
|
|
|
|
|
|
it "sets the second user's last_read_post_number back to 1" do
|
|
|
|
topic_user.seen_post_count.should == 1
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'after delete' do
|
|
|
|
|
|
|
|
let!(:coding_horror) { Fabricate(:coding_horror) }
|
|
|
|
let!(:post) { Fabricate(:post, raw: "Hello @CodingHorror") }
|
|
|
|
|
|
|
|
it "should feature the users again (in case they've changed)" do
|
|
|
|
Jobs.expects(:enqueue).with(:feature_topic_users, has_entries(topic_id: post.topic_id, except_post_id: post.id))
|
|
|
|
PostDestroyer.new(moderator, post).destroy
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'with a reply' do
|
|
|
|
|
|
|
|
let!(:reply) { Fabricate(:basic_reply, user: coding_horror, topic: post.topic) }
|
|
|
|
let!(:post_reply) { PostReply.create(post_id: post.id, reply_id: reply.id) }
|
|
|
|
|
|
|
|
it 'changes the post count of the topic' do
|
|
|
|
post.reload
|
|
|
|
lambda {
|
|
|
|
PostDestroyer.new(moderator, reply).destroy
|
|
|
|
post.topic.reload
|
|
|
|
}.should change(post.topic, :posts_count).by(-1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'lowers the reply_count when the reply is deleted' do
|
|
|
|
lambda {
|
|
|
|
PostDestroyer.new(moderator, reply).destroy
|
|
|
|
}.should change(post.post_replies, :count).by(-1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'should increase the post_number when there are deletion gaps' do
|
|
|
|
PostDestroyer.new(moderator, reply).destroy
|
|
|
|
p = Fabricate(:post, user: post.user, topic: post.topic)
|
|
|
|
p.post_number.should == 3
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
context '@mentions' do
|
|
|
|
let!(:evil_trout) { Fabricate(:evil_trout) }
|
|
|
|
let!(:mention_post) { Fabricate(:post, raw: 'Hello @eviltrout')}
|
|
|
|
|
|
|
|
it 'removes notifications when deleted' do
|
|
|
|
lambda {
|
|
|
|
PostDestroyer.new(Fabricate(:moderator), mention_post).destroy
|
|
|
|
}.should change(evil_trout.notifications, :count).by(-1)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe "flag counts" do
|
|
|
|
let(:codinghorror) { Fabricate(:coding_horror) }
|
|
|
|
let(:bookmark) { PostAction.new(user_id: post.user_id, post_action_type_id: PostActionType.types[:bookmark] , post_id: post.id) }
|
|
|
|
|
|
|
|
it "should reset counts when a post is deleted" do
|
|
|
|
second_post = Fabricate(:post, topic_id: post.topic_id)
|
|
|
|
PostAction.act(codinghorror, second_post, PostActionType.types[:off_topic])
|
|
|
|
-> { PostDestroyer.new(moderator, second_post).destroy }.should change(PostAction, :flagged_posts_count).by(-1)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|