ReactiveX / RxPY / 24587602847

94%

push

github

Fix timer resubscription: remove nonlocal duetime mutation (#756)

observable_timer_duetime_and_period used 'nonlocal duetime' inside the
subscribe closure to convert a relative time to an absolute datetime.
This mutated the outer variable shared across all subscriptions, so any
resubscription (e.g. via repeat()) would reuse the stale absolute time
computed by the first subscription and fire immediately instead of
waiting for the configured duetime.

Fix: compute a local 'dt' variable per subscription without touching
the outer 'duetime' parameter.

Adds a regression test covering the timer + take(2) + repeat() pattern
from issue #697.

Closes #697

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

7 of 8 new or added lines in 2 files covered. (87.5%)

25126 of 26887 relevant lines covered (93.45%)

0.93 hits per line