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
HOTT-4244 Error message when the corresponding year exchange rates do not exist #1799
base: main
Are you sure you want to change the base?
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.
LGTM - minor suggestion but perfectly happy without it as well
context 'when there are no exchange rates' do | ||
let(:exchange_rate_collection) { build(:exchange_rate_collection, exchange_rates: []) } | ||
|
||
it { is_expected.to have_css 'p', text: "There are no #{exchange_rate_collection.type} exchange rates for the year specified." } |
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.
you could test that the table is not included?
it { is_expected.not_to have_css 'table' }
…hub.com/trade-tariff/trade-tariff-frontend into HOTT-4244-Exchange-Rate-Missing-Message
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.
LGTM 🙌
Jira link
https://transformuk.atlassian.net/browse/HOTT-4244
What?
I have altered:
Why?
I am doing this because:
Have you? (optional)