raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-01-24 20:36:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

Corrects test case by adding expect

Browse Source 
has_select? only returns true or false, it requires an assertion to assure the test is evaluated

Improves syntax
This commit is contained in:
filipefurtad0
2024-06-12 18:53:29 -06:00
parent 10dbe77f71
commit 9e5c21f7cd
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
spec/system/admin/products_v3/products_spec.rb
View File

@@ -1419,9 +1419,8 @@ RSpec.describe 'As an enterprise user, I can manage my products', feature: :admi
it "shows only suppliers that I manage or have permission to" do
visit spree.admin_products_path
within row_containing_name(product_supplied.name) do
page.has_select?(
expect(page).to have_select(
'_products_0_supplier_id',
options: [
supplier_managed1.name, supplier_managed2.name, supplier_permitted.name
@@ -1430,11 +1429,11 @@ RSpec.describe 'As an enterprise user, I can manage my products', feature: :admi
end
within row_containing_name(product_supplied_permitted.name) do
page.has_select?(
'_products_0_supplier_id',
expect(page).to have_select(
'_products_1_supplier_id',
options: [
supplier_managed1.name, supplier_managed2.name, supplier_permitted.name
], selected: supplier_managed1.name
], selected: supplier_permitted.name
)
end
end