-
Notifications
You must be signed in to change notification settings - Fork 1
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
EL-1745 Check Answers display #1550
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just the one query.
Also, do we need to put in a bunch of flow tests?
@MazOneTwoOne re this comment I am not keen on the idea of adding a bunch of tests to check what displays given that UATing it will confirm that for us without the ongoing overhead of running 3 or 4 tests to cover the variations needed for this bug. If the bug was more serious then tests may be required. |
8f56e9a
to
e05e2e4
Compare
Maybe not a bunch of tests, but at least we should update this spec It may stop us adding in a similar regression. |
5192105
to
33336dd
Compare
8eed2d9
to
c36fc23
Compare
0d161e4
to
1e4d633
Compare
add expectation for the contents of EE check answers add nocov to fix coverage issue, there are tests covering both sides of the branch
1e4d633
to
4bb5230
Compare
Jira ticket
What changed and why
Added a check to see if we are responding to an EE gross income result
If we are we display only relevant sections on Check ANswers
Guidance to review
Checklist
Before you ask people to review this PR: