Skip to content

Commit

Permalink
Fix (partially) test "tests/test_concurrent.py::test_message_order_in…
Browse files Browse the repository at this point in the history
…_subscribe_unsubscribe_all_loop[False-async]".

There is still a race: all calls to `unsubscribe` may finish before the subscription is registered.
  • Loading branch information
Andrey Zelenchuk committed Mar 10, 2022
1 parent 28632fc commit 05bc254
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/test_concurrent.py
Original file line number Diff line number Diff line change
Expand Up @@ -732,9 +732,9 @@ async def test_message_order_in_subscribe_unsubscribe_all_loop(
'complete' message.
"""

NUMBER_OF_UNSUBSCRIBE_CALLS = 50 # pylint: disable=invalid-name
NUMBER_OF_UNSUBSCRIBE_CALLS = 100 # pylint: disable=invalid-name
# Delay in seconds.
DELAY_BETWEEN_UNSUBSCRIBE_CALLS = 0.01 # pylint: disable=invalid-name
DELAY_BETWEEN_UNSUBSCRIBE_CALLS = 0.02 # pylint: disable=invalid-name
# Gradually stop the test if time is up.
TIME_BORDER = 20 # pylint: disable=invalid-name

Expand Down

0 comments on commit 05bc254

Please sign in to comment.