What changed?
I have merged the timeout logic for the tests in internal_workers_test, this raises the timeout to 10s for all tests, and treats timing out as a test failure.
Why?
The tests were flaky, sometimes timing out sometimes not, also it was hard to find the reason they failed as timing out was not treated as a failure.
How did you test it?
I ran all unit tests, this only changes the structure of the tests.
Potential risks
This only changes the structure of the tests so it should not pose any risks.
What changed? I have merged the timeout logic for the tests in internal_workers_test, this raises the timeout to 10s for all tests, and treats timing out as a test failure.
Why? The tests were flaky, sometimes timing out sometimes not, also it was hard to find the reason they failed as timing out was not treated as a failure.
How did you test it? I ran all unit tests, this only changes the structure of the tests.
Potential risks This only changes the structure of the tests so it should not pose any risks.