cucumber / cucumber-js / 29076292206

97%

master: 98%

push

github

Use fake timers instead of real setTimeout waits (#2874)

The stopwatch specs slept on real setTimeout (1200+200+200+200+200ms =
~2s of wall-clock per run) and asserted elapsed time with a closeTo(±50ms)
tolerance, which can flake under CI load when the event loop is delayed
past the tolerance.

Drive the four duration tests with @sinonjs/fake-timers via
FakeTimers.withGlobal(timeMethods) — the same pattern already used in
test_case_runner_spec.ts and assemble_test_cases_spec.ts — advancing the
clock with clock.tick(...). Tests are now synchronous, assert exact
durations, and the file runs in ~12ms instead of ~2s. The 'timestamp
close to now' test is left on the real clock (it is already instant).

Signed-off-by: Nikolaus Schuetz <nikolauspschuetz@gmail.com>

1738 of 2226 branches covered (78.08%)

2684 of 2762 relevant lines covered (97.18%)

789.47 hits per line