-
Notifications
You must be signed in to change notification settings - Fork 0
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
CRIMRE-389 Surface appeal lodged date, change details and maat id in appeal cases #336
Conversation
it 'includes the page title' do | ||
expect(page).to have_content I18n.t('crime_applications.show.page_title') | ||
end | ||
|
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 reordered spec to match order of fields on page
it 'includes button to assign' do | ||
expect(page).to have_content('Assign to your list') | ||
end | ||
|
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 reordered spec to match order of fields on page
it 'does shows the class not determined badge' do | ||
table_body = find(:xpath, | ||
"//table[@class='govuk-table app-dashboard-table govuk-!-margin-bottom-9'] | ||
//tr[contains(td[1], 'Non-listed offence, manually entered')]") |
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 reordered spec to match order of fields on page
it 'does not show the CTAs' do | ||
expect(page).not_to have_content('Mark as completed') | ||
end | ||
|
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 reordered spec to match order of fields on page
|
||
it 'displays calculated overall offence class badgee' do | ||
expect(page).to have_content('Overall offence class Class C') | ||
end |
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 reordered spec to match order of fields on page
expect(row).to have_content('Class C') | ||
it 'displays undetermined overall offence class badge' do | ||
expect(page).to have_content('Overall offence class Undetermined') | ||
end |
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 reordered spec to match order of fields on page
"//table[@class='govuk-table app-dashboard-table govuk-!-margin-bottom-9'] | ||
//tr[contains(td[1], 'Robbery')]") | ||
expect(table_body).to have_content('Class not determined') | ||
end |
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 reordered spec to match order of fields on page
73f1bb2
to
73f5660
Compare
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.
Looks good - perhaps we could think about having a presenter or something to avoid some view logic, but happy for this to be in a further PR etc.
Description of change
In appeal cases surface appeal attributes:
Link to relevant ticket
https://dsdmoj.atlassian.net/browse/CRIMRE-389
Notes for reviewer
Screenshots of changes (if applicable)
Before changes:
After changes:
How to manually test the feature
Put an appeal / appeal with changes case type through Apply, view Case Details section on Review