paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-03 05:23:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor: Make acting_user a mandatory kwarg for do_change_default_event_register_stream.

Browse Source
This commit is contained in:
shanukun
2021-04-08 14:25:24 +05:30
committed by Tim Abbott
parent 8f70386df3
commit 8d3c6a2feb
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
zerver/lib/actions.py
View File

@@ -4155,7 +4155,7 @@ def do_change_default_sending_stream(
def do_change_default_events_register_stream(
user_profile: UserProfile, stream: Optional[Stream], acting_user: Optional[UserProfile] = None
user_profile: UserProfile, stream: Optional[Stream], *, acting_user: Optional[UserProfile]
) -> None:
old_value = user_profile.default_events_register_stream_id
user_profile.default_events_register_stream = stream

4
zerver/tests/test_events.py
View File

@@ -1474,11 +1474,11 @@ class NormalActionsTest(BaseAction):
bot = self.create_bot("test")
stream = get_stream("Rome", bot.realm)
action = lambda: do_change_default_events_register_stream(bot, stream)
action = lambda: do_change_default_events_register_stream(bot, stream, acting_user=None)
events = self.verify_action(action)
check_realm_bot_update("events[0]", events[0], "default_events_register_stream")
action = lambda: do_change_default_events_register_stream(bot, None)
action = lambda: do_change_default_events_register_stream(bot, None, acting_user=None)
events = self.verify_action(action)
check_realm_bot_update("events[0]", events[0], "default_events_register_stream")