From bddfa95eb53e3c71ecb8acb90dec77718210dea4 Mon Sep 17 00:00:00 2001 From: Luis Ramos Date: Tue, 24 Mar 2020 16:12:35 +0000 Subject: [PATCH] Fix broken new property button in properties page --- app/views/spree/admin/properties/new.js.erb | 2 +- spec/features/admin/properties_spec.rb | 27 +++++++++++++++++++++ 2 files changed, 28 insertions(+), 1 deletion(-) create mode 100644 spec/features/admin/properties_spec.rb diff --git a/app/views/spree/admin/properties/new.js.erb b/app/views/spree/admin/properties/new.js.erb index d13137c1aa..c508aba3d0 100644 --- a/app/views/spree/admin/properties/new.js.erb +++ b/app/views/spree/admin/properties/new.js.erb @@ -1,2 +1,2 @@ -$("#new_property").html('<%= escape_javascript(render :template => "spree/admin/properties/new", :formats => [:html], :handlers => [:erb]) %>'); +$("#new_property").html('<%= escape_javascript(render :template => "spree/admin/properties/new", :formats => [:html], :handlers => [:haml]) %>'); $("#new_property_link").parent().hide(); diff --git a/spec/features/admin/properties_spec.rb b/spec/features/admin/properties_spec.rb new file mode 100644 index 0000000000..905810039e --- /dev/null +++ b/spec/features/admin/properties_spec.rb @@ -0,0 +1,27 @@ +# frozen_string_literal: true + +require "spec_helper" + +feature ' + As an admin + I want to manage product properties +' do + include AuthenticationWorkflow + + scenario "creating and editing a property" do + login_to_admin_section + visit spree.admin_properties_path + + click_link 'New Property' + fill_in 'property_name', with: 'New property!' + fill_in 'property_presentation', with: 'New property presentation!' + click_button 'Create' + expect(page).to have_content 'New property!' + + page.find('td.actions a.icon-edit').click + expect(page).to have_field 'property_name', with: 'New property!' + fill_in 'property_name', with: 'New changed property!' + click_button 'Update' + expect(page).to have_content 'New changed property!' + end +end