Flaky: thread.spec "deep thread" on iPhone, a card read by being seen #249

New issue

Closed

opened 2026-10-21 21:57:23 +00:00 by littlefox · 2 comments

littlefox commented

2026-10-21 21:57:23 +00:00

Owner

CI run #400 (PR #248), and earlier in a local full run: thread.spec.ts "a deep thread: indentation stops, cards say whom they answer" fails on iPhone. After the checks of indentation, space shows "Kommentar på djup 2." instead of "… djup 1.".

Cause: the test looks at the thread for several seconds, and the card for depth 1 is on screen long enough to be read by being seen (#180), so the engine skips it. The first test in the file sets window.__neokomSeenOff for this reason; this one doesn't.

Fix: set it here too.

CI run #400 (PR #248), and earlier in a local full run: `thread.spec.ts` "a deep thread: indentation stops, cards say whom they answer" fails on iPhone. After the checks of indentation, space shows "Kommentar på djup 2." instead of "… djup 1.". Cause: the test looks at the thread for several seconds, and the card for depth 1 is on screen long enough to be read by being seen (#180), so the engine skips it. The first test in the file sets `window.__neokomSeenOff` for this reason; this one doesn't. Fix: set it here too.

littlefox added this to the 0.6: Performance milestone

2026-10-21 21:57:23 +00:00

littlefox added the

kind/bug

area/ci

labels

2026-10-21 21:57:23 +00:00

littlefox commented

2026-10-21 21:57:51 +00:00

Author

Owner

Also in run #400: time.spec.ts on Android (passed on retry). The tap on the time didn't switch it to the exact time. The post had just been shown, and it is drawn again when its thread arrives around it, so the tapped label was replaced and its state went with it. The spec now taps until it takes; the re-drawing itself is #229.

Also in run #400: `time.spec.ts` on Android (passed on retry). The tap on the time didn't switch it to the exact time. The post had just been shown, and it is drawn again when its thread arrives around it, so the tapped label was replaced and its state went with it. The spec now taps until it takes; the re-drawing itself is #229.

littlefox referenced this issue

2026-10-21 21:57:52 +00:00

Posts and threads settle or jitter a bit before they stop #229

littlefox referenced this issue from a pull request that will close it,

2026-10-21 21:57:54 +00:00

"Visa fler": count only this thread's opened posts #248

littlefox referenced this issue from a commit

2026-10-21 21:57:54 +00:00

Fix two flaky specs: a card read by being seen, a lost tap

littlefox closed this issue

2026-10-21 22:27:43 +00:00

littlefox referenced this issue

2026-10-21 22:29:51 +00:00

Measure first: a big meeting, a perf spec and a desktop memory run #252

littlefox commented

2026-10-21 23:01:33 +00:00

Author

Owner

Fixed in #248 (main #403 green): thread.spec's deep-thread test turns read-by-seen off; time.spec taps until the label switches. Both passed on all three profiles locally and in CI.