Files e2e: Recent type-to-select focuses another d-prefixed file #964

Open
opened 2026-10-03 00:35:33 +00:00 by kayg · 0 comments
Owner

Found by job/fix-7a-product (branch from job/merge-round-7a, head 4acefcbac), full bun apps/web/e2e/files.mjs against a rebuilt server and production SPA. All steps up to and including PASS keyboard copy, paste and move passed (including the #961 slow-click rename). The next step failed:

AssertionError: type-to-select moves focus to the renamed Recent file
false !== true
    at main (apps/web/e2e/files.mjs:1799:10)

Likely cause (not verified): the Recent list also holds Inbox/Child/deep.txt, which also starts with d. A rename does not change the file mtime, and gamma.md (renamed to delta.md) was uploaded before deep.txt. If Recent orders by modification time, type-to-select d focuses deep.txt first. The assertion also reads the focus class right after the key press, without a wait.

To do: check the Recent order on a real build, then fix either the fixture expectation (type a longer prefix such as de + l) or the Recent ordering, whichever matches DESIGN. Keep the type-to-select assertion. Not a merge blocker class (test or ordering behavior).

Found by job/fix-7a-product (branch from job/merge-round-7a, head 4acefcbac), full `bun apps/web/e2e/files.mjs` against a rebuilt server and production SPA. All steps up to and including `PASS keyboard copy, paste and move` passed (including the #961 slow-click rename). The next step failed: ``` AssertionError: type-to-select moves focus to the renamed Recent file false !== true at main (apps/web/e2e/files.mjs:1799:10) ``` Likely cause (not verified): the Recent list also holds `Inbox/Child/deep.txt`, which also starts with `d`. A rename does not change the file mtime, and `gamma.md` (renamed to `delta.md`) was uploaded before `deep.txt`. If Recent orders by modification time, type-to-select `d` focuses `deep.txt` first. The assertion also reads the focus class right after the key press, without a wait. To do: check the Recent order on a real build, then fix either the fixture expectation (type a longer prefix such as `de` + `l`) or the Recent ordering, whichever matches DESIGN. Keep the type-to-select assertion. Not a merge blocker class (test or ordering behavior).
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
kayg/calternal#964
No description provided.