[#78] Exhaustive GAAP accounting audit: presentation + recognition fixes with regression tests #82
No reviewers
Labels
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
Winnie/bankwars!82
Loading…
Reference in a new issue
No description provided.
Delete branch "78-gaap"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
[#78] Exhaustive GAAP accounting audit
Audits every line of the financial reports and the books against real-world presentation (docs/gaap-audit.md: findings, GAAP references, corrections).
Findings fixed:
Report.Accruedfield removed.money()fixed ("-0", negative grouping) and the parenthesized-negatives ruling applied viamoneyParenon every report line: expenses and losses render "(1,941)".Tests: 18 new regression tests (bond_amortization_test.go, writeoff_test.go, gaap_statement_test.go) pinning every correction, including a full seeded AI-game invariant test. All pre-existing tests (acquisition accounting, dead loans, reserve, bond units, award, bidall, iterlog, lisp, hook) pass unchanged.
Verification: go build + go vet + full go test green; spectate smoke (gate config) 0
[aierrors / 0 ACCT DIVERGENCE / 0 CF DIVERGENCE; quick bench 390 games, 0 divergences (empire 114 / mill 106 / vault 63 / adapt 88).