diff --git a/zerver/lib/send_email.py b/zerver/lib/send_email.py index a62f58eb12..93b1f1c7f4 100644 --- a/zerver/lib/send_email.py +++ b/zerver/lib/send_email.py @@ -434,7 +434,7 @@ def deliver_scheduled_emails(email: ScheduledEmail) -> None: # This state doesn't make sense, so something must be mutating, # or in the process of deleting, the object. We assume it will bring # things to a correct state, and we just do nothing except logging this event. - logger.warning("ScheduledEmail id %s has empty users and address attributes.", email.id) + logger.error("ScheduledEmail id %s has empty users and address attributes.", email.id) return if user_ids: diff --git a/zerver/tests/test_users.py b/zerver/tests/test_users.py index 81ae02d489..3330c01614 100644 --- a/zerver/tests/test_users.py +++ b/zerver/tests/test_users.py @@ -1468,7 +1468,7 @@ class ActivateTest(ZulipTestCase): self.assertEqual( info_log.output, [ - f"WARNING:zulip.send_email:ScheduledEmail id {email.id} has empty users and address attributes." + f"ERROR:zulip.send_email:ScheduledEmail id {email.id} has empty users and address attributes." ], )