Skip to content

Commit

Permalink
Fix student loan partial
Browse files Browse the repository at this point in the history
  • Loading branch information
RoseSAK committed Jan 20, 2025
1 parent 83d166a commit 6eabf0c
Show file tree
Hide file tree
Showing 6 changed files with 27 additions and 25 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -25,14 +25,16 @@

<!-- STUDENT FINANCE -->

<%= render(
"shared/check_answers/student_finance",
individual: "Client",
individual_with_determiner: t("generic.client"),
model: @legal_aid_application.applicant,
url: providers_legal_aid_application_means_student_finance_path(@legal_aid_application),
read_only: false,
) %>
<% unless @legal_aid_application.applicant_receives_benefit? %>
<%= render(
"shared/check_answers/student_finance",
individual: "Client",
individual_with_determiner: t("generic.client"),
model: @legal_aid_application.applicant,
url: providers_legal_aid_application_means_student_finance_path(@legal_aid_application),
read_only: false,
) %>
<% end %>

<!-- OUTGOINGS -->

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -59,13 +59,15 @@

<!--STUDENT FINANCE -->

<%= render(
"shared/check_answers/student_finance",
individual: "Client",
individual_with_determiner: t("generic.client"),
model: @legal_aid_application.applicant,
read_only: true,
) %>
<% unless @legal_aid_application.applicant_receives_benefit? %>
<%= render(
"shared/check_answers/student_finance",
individual: "Client",
individual_with_determiner: t("generic.client"),
model: @legal_aid_application.applicant,
read_only: true,
) %>
<% end %>

<!--OUTGOINGS -->

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,15 +47,14 @@ Feature: Bank statement upload check your answers

And the "Payments your client pays" section's questions and answers should match:
| question | answer |
| Housing payments | £1,600.00\ntotal in last 3 months |
| Housing payments | £1,600.00 total in last 3 months |
| Childcare payments | None |
| Maintenance payments to a former partner | None |
| Payments towards legal aid in a criminal case | None |

And the "Housing Benefit" section's questions and answers should match:
| question | answer |
| Does your client get Housing Benefit? | Yes |
| Amount | £1,200.00\ntotal in last 3 months |
| Amount of Housing benefit | £1,200.00\ntotal in last 3 months |

When I click Check Your Answers Change link for "bank statements client"
And I upload an evidence file named "hello_world.pdf"
Expand Down Expand Up @@ -94,8 +93,7 @@ Feature: Bank statement upload check your answers

When I click "Save and continue"
Then I should be on the "check_income_answers" page showing "Check your answers"
And the answer for "applicant student finance question" should be "Yes"
And the answer for "student finance annual amount" should be "£5,000"
And the answer for "applicant student finance annual amount" should be "£5,000"

When I click Check Your Answers Change link for "Payments your client pays"
Then I should be on the "regular_outgoings" page showing "Which of these payments does your client pay?"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ Feature: Bank statement upload journey happy path
And I should not see 'What not to include'

Then I check "Pension"
Then I should see "monthly"
Then I should see "Monthly"
And I fill "Pension amount" with "100"
And I choose "providers-means-regular-income-form-pension-frequency-monthly-field"

Expand Down Expand Up @@ -104,7 +104,7 @@ Feature: Bank statement upload journey happy path
And I should not see 'What not to include'

Then I check "Pension"
Then I should see "monthly"
Then I should see "Monthly"
And I fill "Pension amount" with "100"
And I choose "providers-means-regular-income-form-pension-frequency-monthly-field"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ Feature: Bank statement upload check your answers

And the "Payments the partner pays" section's questions and answers should match:
| question | answer |
| Housing payments | £1,600.00\ntotal in last 3 months |
| Housing payments | £1,600.00 total in last 3 months |
| Childcare payments | None |
| Maintenance payments to a former partner | None |
| Payments towards legal aid in a criminal case | None |
Expand Down
4 changes: 2 additions & 2 deletions features/providers/review_and_print.feature
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ Feature: Review and print your application
Given I have completed a bank statement upload application with merits
And the feature flag for linked_applications is enabled
When I view the review and print your application page

Then the following sections should exist:
| tag | section |
| h2 | Client details |
Expand Down Expand Up @@ -58,6 +59,7 @@ Feature: Review and print your application
| h2 | Income, regular payments and assets |
| h3 | Income |
| h3 | Regular payments |
| h3 | Student finance |
| h3 | Property |
| h3 | Vehicles |
| h2 | Bank accounts |
Expand All @@ -77,7 +79,6 @@ Feature: Review and print your application
| h3 | Employment income |
| h3 | Payments your client gets |
| h3 | Payments your client gets in cash |
| h3 | Student finance |
| h2 | Your client's outgoings |
| h3 | Payments your client pays |
| h3 | Payments your client pays in cash|
Expand All @@ -90,7 +91,6 @@ Feature: Review and print your application
| Maintenance payments from a former partner total |
| Income from a property or lodger total |
| Pension total |
| Student loan |

And I should not see "Housing benefit total"

Expand Down

0 comments on commit 6eabf0c

Please sign in to comment.