Server test: full_app_setup_session_config_and_backup signs out twice (401 != 204) #80

Closed
opened 2026-09-25 04:00:10 +00:00 by kayg · 1 comment
Owner

Found while running the gates for job/photos-ui on main 1bbd40e.

crates/calternal-server/src/wire.rs test wire::tests::full_app_setup_session_config_and_backup (run through live_apps_run_in_separate_processes) fails at line ~2472:

assertion `left == right` failed
  left: 401
 right: 204

The test signs out twice. After the owner-only delete it signs out, checks /api/v1/auth/me is 401, checks the agent session and /readyz, and then repeats the same "Sign-out revokes only this session and clears the cookie" block. The second DELETE /api/v1/auth/session uses a cookie that is already revoked, so it gets 401. It looks like a merge kept both copies of the block. Removing the second copy should make the test pass.

On older bases (cc6d421, cd53f30) the same child test instead hung (no CPU, over 40 minutes). I reproduced that hang with main's own Photos crate and wire.rs swapped in, so it was not from the Photos UI branch; on 1bbd40e it fails fast instead.

Context for the owning job

  • Repo: kayg/calternal (~/Developer/calternal). Read CLAUDE.md first.
  • Gates: cargo test --workspace must pass; quote the output.
  • Comment on this issue when you start and when finished. Never close it.
Found while running the gates for job/photos-ui on main `1bbd40e`. `crates/calternal-server/src/wire.rs` test `wire::tests::full_app_setup_session_config_and_backup` (run through `live_apps_run_in_separate_processes`) fails at line ~2472: ``` assertion `left == right` failed left: 401 right: 204 ``` The test signs out twice. After the owner-only delete it signs out, checks `/api/v1/auth/me` is 401, checks the agent session and `/readyz`, and then repeats the same "Sign-out revokes only this session and clears the cookie" block. The second `DELETE /api/v1/auth/session` uses a cookie that is already revoked, so it gets 401. It looks like a merge kept both copies of the block. Removing the second copy should make the test pass. On older bases (`cc6d421`, `cd53f30`) the same child test instead hung (no CPU, over 40 minutes). I reproduced that hang with main's own Photos crate and `wire.rs` swapped in, so it was not from the Photos UI branch; on `1bbd40e` it fails fast instead. ## Context for the owning job - Repo: kayg/calternal (~/Developer/calternal). Read CLAUDE.md first. - Gates: `cargo test --workspace` must pass; quote the output. - Comment on this issue when you start and when finished. Never close it.
Author
Owner

Fixed in 9561cc8b6 (origin/dev); wire::tests::live_apps_run_in_separate_processes covers the isolated full-app setup test with one sign-out.

Fixed in 9561cc8b6 (origin/dev); `wire::tests::live_apps_run_in_separate_processes` covers the isolated full-app setup test with one sign-out.
kayg closed this issue 2026-10-03 11:55:04 +00:00
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#80
No description provided.