AsyncMonitor.WaitAsync is not picking up the pulse. about asyncex HOT 4 CLOSED

Monitor.WaitAsync is a method that changes the underlying state of the monitor. You especially need to be careful when using Task.WhenAny with synchronization primitives (there is almost never a good reason to do this).

In this case, as far as the monitor knows, it has two waiters: the first WaitAsync and the second WaitAsync. So when the Pulse comes through, it satisfies the first task, as it should.

FYI, the unit test for "pulsing a monitor releases a waiter" is here.

On a side note, unit tests that depend on timings are inherently unstable.

from asyncex.

Thanks for the heads up. I found that I needed to pass a cancellation token to the waiter and let it blow up (catch it)

from asyncex.

I see in your unit tests that you are entering the monitor to pulse. In your docs and my working unit test it doesn't mention that you have to do this. Is this required or just a best practice?

from asyncex.

Yes, it is required. I'll update the docs.

from asyncex.