Skip to content

Stabilize dashboard token provenance tests - #2422

Closed
Yuxin-Qiao wants to merge 1 commit into
steipete:mainfrom
Yuxin-Qiao:codex/stabilize-dashboard-token-tests
Closed

Stabilize dashboard token provenance tests#2422
Yuxin-Qiao wants to merge 1 commit into
steipete:mainfrom
Yuxin-Qiao:codex/stabilize-dashboard-token-tests

Conversation

@Yuxin-Qiao

Copy link
Copy Markdown
Contributor

Summary

Why

The Dashboard controller otherwise reads process defaults while its request builder reads the fixture settings. That mismatch causes the test to observe empty groups despite a confirmed fixture snapshot.

Validation

  • SpendDashboardTokenProvenanceTests: 13 tests passed
  • make check: passed
  • full make test: not rerun on this standalone branch because it still lacks the separate Controller fixes in test(repo-hygiene): restore macOS test baseline #2418. Together these PRs cover all 13 group-49 failures reproduced from main.

UI proof

Not applicable: test-isolation change only.

Provider / privacy

No production provider behavior, credentials, or account data access changed.

@clawsweeper

clawsweeper Bot commented Jul 23, 2026

Copy link
Copy Markdown

ClawSweeper status: review started.

I am starting a fresh review of this pull request: Stabilize dashboard token provenance tests This is item 1/1 in the current shard. Shard 0/1.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.

@Yuxin-Qiao

Copy link
Copy Markdown
Contributor Author

Superseded by #2418, which now contains this Token Provenance test-isolation fix together with the complete macOS test-baseline restoration and full validation.

@Yuxin-Qiao Yuxin-Qiao closed this Jul 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant