Tests: wait for a dropped session's forwards to stop, not just for the tab to go dead #23

Merged

jaap-jan merged 1 commits from claude/elastic-mccarthy-6b4ae7 into main

2026-10-16 12:04:42 +00:00

Author	SHA1	Message	Date
jaap-jan	f43436218a	Tests: wait for a dropped session's forwards to stop, not just for the tab to go dead ci / build and test (pull_request) Successful in 2m42s Details ci / desktop nightly (pull_request) Skipped Details ci / api image (pull_request) Successful in 7s Details ci / deploy ssh.dodotech.cloud (pull_request) Skipped Details ci / android head (pull_request) Successful in 3m42s Details WhenTheSessionEnds_ItsForwardsSayTheyWentWithIt failed once under a full run. Ended clears IsLive and then tells each forward its session is over, in one call. The app's dispatcher never splits that, but the harness's post runs it on the fake shell's thread, so the test's poll on !IsLive could return between the two and find the forwards still listening. The test now also waits until every forward says "Stopped with the session.", the last thing SessionEnded sets. A 300 ms sleep injected between the two steps fails the old test and passes the new one; the full App.Tests suite passed three runs in a row.	2026-10-16 13:17:01 +02:00

Author

SHA1

Message

Date

jaap-jan

f43436218a

Tests: wait for a dropped session's forwards to stop, not just for the tab to go dead

ci / build and test (pull_request) Successful in 2m42s

Details

ci / desktop nightly (pull_request) Skipped

Details

ci / api image (pull_request) Successful in 7s

Details

ci / deploy ssh.dodotech.cloud (pull_request) Skipped

Details

ci / android head (pull_request) Successful in 3m42s

Details

WhenTheSessionEnds_ItsForwardsSayTheyWentWithIt failed once under a full run. Ended clears IsLive
and then tells each forward its session is over, in one call. The app's dispatcher never splits that,
but the harness's post runs it on the fake shell's thread, so the test's poll on !IsLive could return
between the two and find the forwards still listening.

The test now also waits until every forward says "Stopped with the session.", the last thing
SessionEnded sets. A 300 ms sleep injected between the two steps fails the old test and passes the new
one; the full App.Tests suite passed three runs in a row.

2026-10-16 13:17:01 +02:00