medplum / medplum / 31131675342

92%

main: 92%

push

github

test(agent): stop waitFor timing out on satisfied conditions (#10116)

waitFor exited its poll loop on wall-clock alone, so an event-loop stall past the
deadline threw without ever re-checking the predicate. That surfaced as flakes like
'waitFor: auto-response not satisfied after 1000ms' in bytestream.test.ts, where the
auto-response is written synchronously from the socket's data handler and had in fact
already arrived.

Check the predicate before the deadline test so it is always evaluated once more, and
credit back any poll overrun, since a blocked event loop is not time in which the
predicate could have been observed.

Signed-off-by: Derrick Farris <derrick@medplum.com>

23110 of 26206 branches covered (88.19%)

Branch coverage included in aggregate %.

8 of 8 new or added lines in 1 file covered. (100.0%)

40935 of 43336 relevant lines covered (94.46%)

12161.37 hits per line