Money wizard acceptance compares monthly check count to transaction count #1205

Open
opened 2026-10-06 15:30:57 +00:00 by kayg · 0 comments
Owner

Found in merge30j (#1140), production head cc1aae60a. The real Actual fixture reaches Review after the writer and CC fixes, with four Account questions and four selected Card suggestions. The unchanged acceptance assertion then fails:

Synthetic review summary: All 7,320 checks passed
AssertionError: the review summary shows at least 15,000 real checks

The fixture has 54 Accounts, two Categories, 120 months and 15,000 transactions. Its checks are 54 × 120 Account balances, 2 × 120 × 3 Category checks, and 120 Ready-to-assign checks: 7,320. Transaction count is not the check count. This reproduced on both 390 px with restart and 820 px without restart. The expectation totalCheckCount >= 15000 and source fixtures remain unchanged, per the owner rule.

The test also selected the bulk picker as an Account picker after the review-step merge. Narrowing only those three selectors to .account-kind-row-card .select-trigger makes all four existing Card-label assertions pass. No assertion changed. A count-only diagnostic log supplies the evidence above.

The orchestrator must decide whether the threshold should use the actual source-check formula or the fixture should contain more Categories. Do not alter financial checks or inflate the displayed count. This blocks completing the monolithic acceptance script beyond Review; the required web/Rust gates pass and independent format/Undo checks are being run.

Found in merge30j (#1140), production head `cc1aae60a`. The real Actual fixture reaches Review after the writer and CC fixes, with four Account questions and four selected Card suggestions. The unchanged acceptance assertion then fails: ```text Synthetic review summary: All 7,320 checks passed AssertionError: the review summary shows at least 15,000 real checks ``` The fixture has 54 Accounts, two Categories, 120 months and 15,000 transactions. Its checks are 54 × 120 Account balances, 2 × 120 × 3 Category checks, and 120 Ready-to-assign checks: 7,320. Transaction count is not the check count. This reproduced on both 390 px with restart and 820 px without restart. The expectation `totalCheckCount >= 15000` and source fixtures remain unchanged, per the owner rule. The test also selected the bulk picker as an Account picker after the review-step merge. Narrowing only those three selectors to `.account-kind-row-card .select-trigger` makes all four existing Card-label assertions pass. No assertion changed. A count-only diagnostic log supplies the evidence above. The orchestrator must decide whether the threshold should use the actual source-check formula or the fixture should contain more Categories. Do not alter financial checks or inflate the displayed count. This blocks completing the monolithic acceptance script beyond Review; the required web/Rust gates pass and independent format/Undo checks are being run.
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#1205
No description provided.