Skip to content
Snippets Groups Projects
Verified Commit 17de7769 authored by Eliot Berriot's avatar Eliot Berriot
Browse files

See #890: ensure no emails are sent to non-mods or inactive users

parent c82e9c11
Branches
No related tags found
No related merge requests found
...@@ -17,6 +17,8 @@ def test_report_created_signal_sends_email_to_mods(factories, mailoutbox, settin ...@@ -17,6 +17,8 @@ def test_report_created_signal_sends_email_to_mods(factories, mailoutbox, settin
mod2 = factories["users.User"](permission_moderation=True) mod2 = factories["users.User"](permission_moderation=True)
# inactive, so no email # inactive, so no email
factories["users.User"](permission_moderation=True, is_active=False) factories["users.User"](permission_moderation=True, is_active=False)
# no moderation permission, so no email
factories["users.User"]()
report = factories["moderation.Report"]() report = factories["moderation.Report"]()
...@@ -28,6 +30,7 @@ def test_report_created_signal_sends_email_to_mods(factories, mailoutbox, settin ...@@ -28,6 +30,7 @@ def test_report_created_signal_sends_email_to_mods(factories, mailoutbox, settin
unresolved_reports_url = federation_utils.full_url( unresolved_reports_url = federation_utils.full_url(
"/manage/moderation/reports?q=resolved:no" "/manage/moderation/reports?q=resolved:no"
) )
assert len(mailoutbox) == 2
for i, mod in enumerate([mod1, mod2]): for i, mod in enumerate([mod1, mod2]):
m = mailoutbox[i] m = mailoutbox[i]
assert m.subject == "[{} moderation - {}] New report from {}".format( assert m.subject == "[{} moderation - {}] New report from {}".format(
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment