From c5c9e803ab18845a97596483451f74eae229f334 Mon Sep 17 00:00:00 2001 From: filipefurtad0 Date: Sun, 24 Jan 2021 13:32:04 +0000 Subject: [PATCH] Fixes a pending test case and improves the spec run time --- spec/features/admin/shipping_methods_spec.rb | 17 +++++++++++++---- 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/spec/features/admin/shipping_methods_spec.rb b/spec/features/admin/shipping_methods_spec.rb index a2aa27b729..6e4a1c5e44 100644 --- a/spec/features/admin/shipping_methods_spec.rb +++ b/spec/features/admin/shipping_methods_spec.rb @@ -157,26 +157,35 @@ feature 'shipping methods' do expect(page).to have_selector 'td', text: 'Two', count: 1 end - pending "shows me only shipping methods for the enterprise I select" do + it "shows me only shipping methods for the enterprise I select" do shipping_method1 shipping_method2 visit admin_enterprises_path within("#e_#{distributor1.id}") { click_link 'Settings' } + within(".side_menu") do click_link "Shipping Methods" end - expect(page).to have_content shipping_method1.name - expect(page).to have_content shipping_method2.name + + expect(page).to have_content shipping_method1.name + expect(page).to have_content shipping_method2.name + + expect(page).to have_checked_field "enterprise_shipping_method_ids_#{shipping_method2.id}" + expect(page).to have_checked_field "enterprise_shipping_method_ids_#{shipping_method1.id}" click_link 'Enterprises' within("#e_#{distributor2.id}") { click_link 'Settings' } + within(".side_menu") do click_link "Shipping Methods" end - expect(page).not_to have_content shipping_method1.name + expect(page).to have_content shipping_method1.name expect(page).to have_content shipping_method2.name + + expect(page).to have_checked_field "enterprise_shipping_method_ids_#{shipping_method2.id}" + expect(page).to have_unchecked_field "enterprise_shipping_method_ids_#{shipping_method1.id}" end end end