From 5ed5ed28733fa520bcdcf0a9598db2a23943e8ea Mon Sep 17 00:00:00 2001 From: Eduardo Date: Sat, 30 May 2020 18:04:43 -0300 Subject: [PATCH] fix payment methods table to show proper provider name --- app/views/spree/admin/payment_methods/index.html.haml | 2 +- spec/features/admin/payment_method_spec.rb | 10 ++++++++++ 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/app/views/spree/admin/payment_methods/index.html.haml b/app/views/spree/admin/payment_methods/index.html.haml index ab1333f2c5..adb14d5caa 100644 --- a/app/views/spree/admin/payment_methods/index.html.haml +++ b/app/views/spree/admin/payment_methods/index.html.haml @@ -32,7 +32,7 @@ - method.distributors.each do |distributor| = distributor.name %br/ - %td= method.type + %td= method.class.clean_name %td.align-center= method.environment.to_s.titleize %td.align-center= method.display_on.blank? ? t('.both') : t('.' + method.display_on.to_s) %td.align-center= method.active ? t('.active_yes') : t('.active_no') diff --git a/spec/features/admin/payment_method_spec.rb b/spec/features/admin/payment_method_spec.rb index 19ec0d151d..3bde47e33d 100644 --- a/spec/features/admin/payment_method_spec.rb +++ b/spec/features/admin/payment_method_spec.rb @@ -185,6 +185,16 @@ feature ' expect(page).not_to have_content payment_method3.name end + it "shows only the providers of the existing payment methods" do + payment_method1 + payment_method2 + payment_method3 + + visit spree.admin_payment_methods_path + + expect(page).to have_content "Cash/EFT/etc. (payments for which automatic validation is not required)", count: 2 + end + it "does not show duplicates of payment methods" do payment_method1 payment_method2