FIX: system user edits should not generate notifications

This commit is contained in:
Régis Hanol 2018-01-30 22:21:07 +01:00
parent aab2d102ff
commit b2f18fc98f
2 changed files with 19 additions and 0 deletions

View File

@ -483,6 +483,7 @@ class PostRevisor
end
def alert_users
return if @editor.id == Discourse::SYSTEM_USER_ID
PostAlerter.new.after_save_post(@post)
end

View File

@ -451,6 +451,24 @@ describe PostRevisor do
end
end
context "alerts" do
let(:mentioned_user) { Fabricate(:user) }
it "generates a notification for a mention" do
expect {
subject.revise!(Fabricate(:user), raw: "Random user is mentioning @#{mentioned_user.username_lower}")
}.to change { Notification.where(notification_type: Notification.types[:mentioned]).count }
end
it "never generates a notification for a mention when the System user revise a post" do
expect {
subject.revise!(Discourse.system_user, raw: "System user is mentioning @#{mentioned_user.username_lower}")
}.not_to change { Notification.where(notification_type: Notification.types[:mentioned]).count }
end
end
context "tagging" do
context "tagging disabled" do
before do