Bryan Helmkamp c6cafceae0 test: de-flake pr_list and full_http_lifecycle_cancel ... Two CLI/server tests racing against peer state on the shared fabro server session, surfaced by running the default nextest profile 20 times. pr_list_missing_github_credentials_errors depended on an empty shared store; if pr_view_reads_pull_request_from_store_without_pull_request_json ran first it left a PR record behind and this test hit the credentials-required branch instead of "No pull requests found." The snapshot captured the empty path, but the test name promises the error path. Seed a PullRequestCreated event against the test's own run so the store is guaranteed non-empty and the credentials-required error fires deterministically. full_http_lifecycle_cancel asserted that the cancel response body's pending_control == "cancel", but that field is re-read from the store projection after the worker has been signaled. The worker is sitting at a human gate; on hot CI it can emit a clearing event before the handler re-reads the projection, yielding a legitimate null. Relax the assertion to accept "cancel" or null; durable convergence to failed/cancelled is still asserted below. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>		2026-04-20 15:04:08 -04:00
..
crates	test: de-flake pr_list and full_http_lifecycle_cancel	2026-04-20 15:04:08 -04:00
packages/fabro-api-client	Merge remote-tracking branch 'origin/main'	2026-04-20 09:47:47 -04:00