From 63cb82717b5a3f8b878189f45fab70256da51375 Mon Sep 17 00:00:00 2001 From: Shen Sat Date: Fri, 2 Jul 2021 08:56:57 +0100 Subject: [PATCH 1/2] Add feature test for shipping categories --- .../admin/shipping_categories/new.html.haml | 2 +- .../configuration/shipping_categories_spec.rb | 26 +++++++++++++++++++ 2 files changed, 27 insertions(+), 1 deletion(-) create mode 100644 spec/features/admin/configuration/shipping_categories_spec.rb diff --git a/app/views/spree/admin/shipping_categories/new.html.haml b/app/views/spree/admin/shipping_categories/new.html.haml index f301434a9e..d2cefa7380 100644 --- a/app/views/spree/admin/shipping_categories/new.html.haml +++ b/app/views/spree/admin/shipping_categories/new.html.haml @@ -5,7 +5,7 @@ - content_for :page_actions do %li - = button_link_to Spree.t(:back_to_shipping_categories_list), spree.admin_shipping_categories_path, :icon => 'icon-arrow-left' + = button_link_to Spree.t(:back_to_shipping_categories), spree.admin_shipping_categories_path, :icon => 'icon-arrow-left' = render :partial => 'spree/shared/error_messages', :locals => { :target => @shipping_category } diff --git a/spec/features/admin/configuration/shipping_categories_spec.rb b/spec/features/admin/configuration/shipping_categories_spec.rb new file mode 100644 index 0000000000..93a04c5a3f --- /dev/null +++ b/spec/features/admin/configuration/shipping_categories_spec.rb @@ -0,0 +1,26 @@ +# frozen_string_literal: true + +require 'spec_helper' + +describe "Tax Categories" do + include AuthenticationHelper + # include WebHelper + + context 'user adds a new shipping category with temperature control' do + it 'user sees new shipping category with temperature control' do + login_as_admin_and_visit spree.admin_shipping_categories_path + click_link "New Shipping Category" + + fill_in "shipping_category_name", with: "freeze" + check "shipping_category_temperature_controlled" + click_button "Create" + + expect(page).to have_content("successfully created!") + expect(page).to have_content("freeze") + row = find('tr', text: 'freeze') + within row do + expect(page).to have_content "Yes" + end + end + end +end From 2f9751f7eaef1b2374b0935d7f895f7bd6d1d87e Mon Sep 17 00:00:00 2001 From: Shen Sat Date: Tue, 13 Jul 2021 08:54:19 +0100 Subject: [PATCH 2/2] Fix typo and remove WebHelper --- spec/features/admin/configuration/shipping_categories_spec.rb | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/spec/features/admin/configuration/shipping_categories_spec.rb b/spec/features/admin/configuration/shipping_categories_spec.rb index 93a04c5a3f..7c461f01e2 100644 --- a/spec/features/admin/configuration/shipping_categories_spec.rb +++ b/spec/features/admin/configuration/shipping_categories_spec.rb @@ -2,9 +2,8 @@ require 'spec_helper' -describe "Tax Categories" do +describe "Shipping Categories" do include AuthenticationHelper - # include WebHelper context 'user adds a new shipping category with temperature control' do it 'user sees new shipping category with temperature control' do