From f8ef7d56b9c329bda6afa6cd22bbf32218dd0db3 Mon Sep 17 00:00:00 2001 From: shanukun Date: Mon, 8 Mar 2021 09:40:24 +0530 Subject: [PATCH] refactor: Make acting_user a mandatory kwarg for do_set_realm_message_editing. --- zerver/lib/actions.py | 3 ++- zerver/tests/test_events.py | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/zerver/lib/actions.py b/zerver/lib/actions.py index 0e8b6b0cc1..c1803905ce 100644 --- a/zerver/lib/actions.py +++ b/zerver/lib/actions.py @@ -859,7 +859,8 @@ def do_set_realm_message_editing( allow_message_editing: bool, message_content_edit_limit_seconds: int, allow_community_topic_editing: bool, - acting_user: Optional[UserProfile] = None, + *, + acting_user: Optional[UserProfile], ) -> None: old_values = dict( allow_message_editing=realm.allow_message_editing, diff --git a/zerver/tests/test_events.py b/zerver/tests/test_events.py index a47b1b07e6..d0bad29054 100644 --- a/zerver/tests/test_events.py +++ b/zerver/tests/test_events.py @@ -1140,6 +1140,7 @@ class NormalActionsTest(BaseAction): allow_message_editing, message_content_edit_limit_seconds, False, + acting_user=None, ) ) check_realm_update_dict("events[0]", events[0])