Devolved nations: Scottish income tax tests + SCP uprating fix (#46)#83
Open
vahid-ahmadi wants to merge 1 commit into
Open
Devolved nations: Scottish income tax tests + SCP uprating fix (#46)#83vahid-ahmadi wants to merge 1 commit into
vahid-ahmadi wants to merge 1 commit into
Conversation
Add worked-example unit tests for the 2025/26 Scottish income tax five/six-band structure (starter/basic/intermediate/higher/advanced/top) at several income levels, plus an explicit Scotland-vs-rUK divergence test at the same income. Correct the Scottish Child Payment weekly amount to £27.15 (2025/26, SSI 2025/100) and £28.20 (2026/27), with projected ~CPI uprating thereafter, fixing the stale £26.70 carried across all years. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Addresses #46
Most of the devolved infrastructure already landed on
main(commit 17324e5 added LBTT/LTT; Scottish income tax bands,is_in_scotlanddispatch, and Scottish Child Payment were already wired and parameterised). This PR closes the two genuine remaining gaps the issue called out: the missing worked-example/divergence test coverage for Scottish income tax, and a documented data bug in the Scottish Child Payment amount.Scope (this PR)
src/variables/income_tax.rspinning HMRC/Revenue-Scotland-style figures for the 2025/26 six-band structure (starter 19% / basic 20% / intermediate 21% / higher 42% / advanced 45% / top 48%):weekly_amountof £26.70 that was carried across every year: now £27.15 for 2025/26 (SSI 2025/100), £28.20 for 2026/27 (April 2026 uprating), and projected ~CPI uprating for 2027/28–2029/30 (£28.75/£29.34/£29.93), matching the repo's convention for other benefits. The existing SCP test reads from params, so it remains valid. UpdatedLEGISLATIVE_REFERENCE.md(the "known issues" and Appendix B.10 notes had flagged exactly this bug).Remaining (out of scope, to document)
Tests
cargo buildandcargo testboth pass — 195 tests, 0 failures, including all new Scottish income tax and divergence tests.🤖 Generated with Claude Code