From 250f7be8a0692916000fc0ae5eb07167dd9de26c Mon Sep 17 00:00:00 2001 From: David Cook Date: Fri, 23 Feb 2024 12:36:42 +1100 Subject: [PATCH] Re-organise spec --- spec/models/spree/product_spec.rb | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/spec/models/spree/product_spec.rb b/spec/models/spree/product_spec.rb index 59f1c12907..d9bed878ce 100644 --- a/spec/models/spree/product_spec.rb +++ b/spec/models/spree/product_spec.rb @@ -42,7 +42,11 @@ module Spree let(:product) { create(:product) } it "should properly assign properties" do - product.set_property('the_prop', 'value1') + expect { + product.set_property('the_prop', 'value1') + product.save + product.reload + }.to change { product.properties.length }.by(1) expect(product.property('the_prop')).to eq 'value1' product.set_property('the_prop', 'value2') @@ -58,13 +62,6 @@ module Spree product.save product.reload }.not_to change { product.properties.length } - - expect { - product.set_property('the_prop_new', 'value') - product.save - product.reload - expect(product.property('the_prop_new')).to eq 'value' - }.to change { product.properties.length }.by(1) end # Regression test for #2455