The result of unit test 'test_timer_leak_nocancel' and 'test_timer_leak_nocancel' is nondeterministic.

achimnol / aiotools

Idiomatic asyncio utilties

https://aiotools.readthedocs.io

MIT License

153 stars 11 forks source link

The result of unit test 'test_timer_leak_nocancel' and 'test_timer_leak_nocancel' is nondeterministic. #8

Closed Suckzoo closed 3 years ago

Suckzoo commented 7 years ago

It seems there is an issue related with common concurrency problem (e.g. deadlock?) Mostly it succeeds, but it fails sometimes on local development environment in two ways:

On test_timer_leak_nocancel, it returns wrong result in this line
```
   assert spawn_count == done_count # e.g. 9 == 8
```
On test_timer_cancel, it does not halt sometimes.

achimnol commented 3 years ago

Now it's resolved using v1.0's timer.VirtualClock applied to the test suite.