Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WASM: Add pumping of timer queue to ThreadlessXunitTestRunner #290

Merged
merged 1 commit into from
Aug 3, 2020

Conversation

akoeplinger
Copy link
Member

@premun premun merged commit 4b70279 into dotnet:master Aug 3, 2020
@akoeplinger akoeplinger deleted the pump-timers branch August 3, 2020 16:12
akoeplinger added a commit to akoeplinger/runtime that referenced this pull request Aug 3, 2020
We only pumped the threadpool but we didn't pump the timer queue inside the xharness runner, so in the case of Task.Delay() it'll schedule a callback on the timer queue and that will never happen which leads to the infinite loop waiting for the test to finish.
To fix that we call an internal method to pump the timer queue like we do for the threadpool.

Requires an xharness bump to include dotnet/xharness#290

Fixes dotnet#38931
akoeplinger added a commit to dotnet/runtime that referenced this pull request Aug 4, 2020
* WASM: Pump timer queue in xharness when running unit tests

We only pumped the threadpool but we didn't pump the timer queue inside the xharness runner, so in the case of Task.Delay() it'll schedule a callback on the timer queue and that will never happen which leads to the infinite loop waiting for the test to finish.
To fix that we call an internal method to pump the timer queue like we do for the threadpool.

Requires an xharness bump to include dotnet/xharness#290

Fixes #38931

* Don't nullref if no timers were started
Jacksondr5 pushed a commit to Jacksondr5/runtime that referenced this pull request Aug 10, 2020
)

* WASM: Pump timer queue in xharness when running unit tests

We only pumped the threadpool but we didn't pump the timer queue inside the xharness runner, so in the case of Task.Delay() it'll schedule a callback on the timer queue and that will never happen which leads to the infinite loop waiting for the test to finish.
To fix that we call an internal method to pump the timer queue like we do for the threadpool.

Requires an xharness bump to include dotnet/xharness#290

Fixes dotnet#38931

* Don't nullref if no timers were started
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants