Add failure path cases

This commit is contained in:
Matt Jankowski 2024-09-22 14:42:36 -04:00
parent b7425c257b
commit deb1245244

View file

@ -28,12 +28,22 @@ RSpec.describe 'Filters' do
fill_in_filter_form
expect(page).to have_content(filter_title)
end
it 'Does not save with invalid values' do
navigate_to_filters
click_on I18n.t('filters.new.title')
expect { click_on I18n.t('filters.new.save') }
.to_not change(CustomFilter, :count)
expect(page)
.to have_content("can't be blank")
end
end
describe 'Editing an existing filter' do
let(:new_title) { 'Change title value' }
before { Fabricate :custom_filter, account: user.account, title: filter_title }
let!(:custom_filter) { Fabricate :custom_filter, account: user.account, title: filter_title }
it 'Updates the saved filter' do
navigate_to_filters
@ -45,6 +55,18 @@ RSpec.describe 'Filters' do
expect(page).to have_content(new_title)
end
it 'Does not save with invalid values' do
navigate_to_filters
click_on filter_title
fill_in filter_title_field, with: ''
expect { click_on submit_button }
.to_not(change { custom_filter.reload.updated_at })
expect(page)
.to have_content("can't be blank")
end
end
describe 'Destroying an existing filter' do