raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-04-04 07:09:14 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix errors count when user submit the form

Browse Source 
Since https://github.com/openfoodfoundation/openfoodnetwork/pull/10317 we are displaying the error message also in the flash message. For unknown reason, build didn't fail for that PR, but, as the PR adds some error message, we need to change the spec to reflect that change.

This PR separates error in checkout page itself, and errors in flash message banner.
This commit is contained in:
Jean-Baptiste Bellet
2023-01-26 11:05:55 +01:00
parent 1ea7304549
commit 999440da0a
1 changed files with 13 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
spec/system/consumer/split_checkout_spec.rb
View File

@@ -489,7 +489,12 @@ describe "As a consumer, I want to checkout my order" do
expect(page).to have_field("Address", with: "")
expect(page).to have_field("City", with: "")
expect(page).to have_field("Postcode", with: "")
expect(page).to have_content("can't be blank", count: 3)
within "checkout" do
expect(page).to have_content("can't be blank", count: 3)
end
within ".flash[type='error']" do
expect(page).to have_content("can't be blank", count: 3)
end
end
it "fills in shipping details and redirects the user to the Payment Method step,
@@ -541,7 +546,13 @@ describe "As a consumer, I want to checkout my order" do
expect(page).to have_field("Address", with: "")
expect(page).to have_field("City", with: "")
expect(page).to have_field("Postcode", with: "")
expect(page).to have_content("can't be blank", count: 7)
within "checkout" do
expect(page).to have_content("can't be blank", count: 7)
end
within ".flash[type='error']" do
expect(page).to have_content("can't be blank", count: 13)
expect(page).to have_content("is invalid", count: 1)
end
expect(page).to have_content("Select a shipping method")
end
end