paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-10 17:07:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

test_classes: Fix misplaced tornado_redirected_to_list comment.

Browse Source
This commit is contained in:
Abhijeet Prasad Bodas
2021-06-03 11:14:56 +05:30
parent b920929182
commit 8b14704d35
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
zerver/lib/test_classes.py
View File

@@ -1272,13 +1272,16 @@ Output:
) -> Iterator[None]: ) -> Iterator[None]:
lst.clear() lst.clear()
real_event_queue_process_notification = django_tornado_api.process_notification real_event_queue_process_notification = django_tornado_api.process_notification
django_tornado_api.process_notification = lambda notice: lst.append(notice)
# process_notification takes a single parameter called 'notice'. # process_notification takes a single parameter called 'notice'.
# lst.append takes a single argument called 'object'. # lst.append takes a single argument called 'object'.
# Some code might call process_notification using keyword arguments, # Some code might call process_notification using keyword arguments,
# so mypy doesn't allow assigning lst.append to process_notification # so mypy doesn't allow assigning lst.append to process_notification
# So explicitly change parameter name to 'notice' to work around this problem # So explicitly change parameter name to 'notice' to work around this problem
django_tornado_api.process_notification = lambda notice: lst.append(notice)
yield yield
django_tornado_api.process_notification = real_event_queue_process_notification django_tornado_api.process_notification = real_event_queue_process_notification
self.assert_length(lst, expected_num_events) self.assert_length(lst, expected_num_events)