From 0a214760045b2691e97b493de67d7ed57da26337 Mon Sep 17 00:00:00 2001 From: shanukun Date: Thu, 8 Apr 2021 14:28:47 +0530 Subject: [PATCH] refactor: Make acting_user a mandatory kwarg for do_change_default_all_public_streams. --- zerver/lib/actions.py | 2 +- zerver/tests/test_events.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/zerver/lib/actions.py b/zerver/lib/actions.py index b5cbcb379f..216d6f2164 100644 --- a/zerver/lib/actions.py +++ b/zerver/lib/actions.py @@ -4196,7 +4196,7 @@ def do_change_default_events_register_stream( def do_change_default_all_public_streams( - user_profile: UserProfile, value: bool, acting_user: Optional[UserProfile] = None + user_profile: UserProfile, value: bool, *, acting_user: Optional[UserProfile] ) -> None: old_value = user_profile.default_all_public_streams user_profile.default_all_public_streams = value diff --git a/zerver/tests/test_events.py b/zerver/tests/test_events.py index 2636b6039d..172b51a528 100644 --- a/zerver/tests/test_events.py +++ b/zerver/tests/test_events.py @@ -1454,7 +1454,7 @@ class NormalActionsTest(BaseAction): def test_change_bot_default_all_public_streams(self) -> None: bot = self.create_bot("test") - action = lambda: do_change_default_all_public_streams(bot, True) + action = lambda: do_change_default_all_public_streams(bot, True, acting_user=None) events = self.verify_action(action) check_realm_bot_update("events[0]", events[0], "default_all_public_streams")