Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated student finance rates for 2025-2026 academic year #7030

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

antoni-devlin
Copy link
Contributor

@antoni-devlin antoni-devlin commented Jan 9, 2025

Go-live is 3rd Feb 2025; await confirmation from SLC before merging.

@govuk-ci govuk-ci temporarily deployed to smart-answers-pr-7030 January 9, 2025 10:32 Inactive
@govuk-ci govuk-ci temporarily deployed to smart-answers-pr-7030 January 9, 2025 11:08 Inactive
@antoni-devlin antoni-devlin force-pushed the Update-student-finance-rates-for-academic-year-2025-2026 branch from 2b15ede to 3dad2a9 Compare January 9, 2025 13:37
@govuk-ci govuk-ci temporarily deployed to smart-answers-pr-7030 January 9, 2025 13:38 Inactive
@antoni-devlin antoni-devlin force-pushed the Update-student-finance-rates-for-academic-year-2025-2026 branch from 3dad2a9 to c4f6dda Compare January 9, 2025 13:41
@govuk-ci govuk-ci temporarily deployed to smart-answers-pr-7030 January 9, 2025 13:42 Inactive
@antoni-devlin antoni-devlin marked this pull request as ready for review January 9, 2025 13:52
@govuk-ci govuk-ci temporarily deployed to smart-answers-pr-7030 January 9, 2025 14:27 Inactive
@govuk-ci govuk-ci temporarily deployed to smart-answers-pr-7030 January 16, 2025 17:19 Inactive
Now that the 2023-2024 rates have been removed, the tests have been
 updated to use the 2024-2025 rates.

In fixing tests I found a bunch that were using out of date figures. I
 have updated these to reference the constant values in the calculator
 class, rather than those values being copy/pasta'd into the tests,
 which is prone to error.

I also removed some tests that I don't think are necessary, since they
 were testing that a high-income household with no dependants is not
 eligible for grants, when there is already a test that a low-income
 household with no dependants is not eligible.

Some test names also made no sense to me, so I've updated the names to
 be more meaningful.

For the tests that check the maximum amount gets reduced by £1 per every
 £X of income above £25k, I manually regenerated the numbers myself.
@mtaylorgds mtaylorgds force-pushed the Update-student-finance-rates-for-academic-year-2025-2026 branch from 809742c to 6229344 Compare January 17, 2025 14:17
@govuk-ci govuk-ci temporarily deployed to smart-answers-pr-7030 January 17, 2025 14:17 Inactive
Copy link
Contributor

@syed-ali-tw syed-ali-tw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

4 participants