From 1382bb3c6bd0c3144ab71b40bd851737502ccfc6 Mon Sep 17 00:00:00 2001 From: Luis Ramos Date: Tue, 3 Mar 2020 15:04:51 +0000 Subject: [PATCH] For some reason in rails 4/spree 2.1 setting the line item as adjustable on the adjustment is not enough to populate line_item.adjustments. Here we make this assignment explicit fixing the spec in rails 4 --- spec/models/spree/line_item_spec.rb | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/spec/models/spree/line_item_spec.rb b/spec/models/spree/line_item_spec.rb index 49edecb14b..0578d90017 100644 --- a/spec/models/spree/line_item_spec.rb +++ b/spec/models/spree/line_item_spec.rb @@ -40,10 +40,13 @@ module Spree describe "finding line items with and without tax" do let(:tax_rate) { create(:tax_rate, calculator: Spree::Calculator::DefaultTax.new) } - let!(:adjustment1) { create(:adjustment, adjustable: li1, originator: tax_rate, label: "TR", amount: 123, included_tax: 10.00) } - let!(:adjustment2) { create(:adjustment, adjustable: li1, originator: tax_rate, label: "TR", amount: 123, included_tax: 10.00) } + let!(:adjustment1) { create(:adjustment, originator: tax_rate, label: "TR", amount: 123, included_tax: 10.00) } - before { li1; li2 } + before do + li1 + li2 + li1.adjustments << adjustment1 + end it "finds line items with tax" do expect(LineItem.with_tax).to eq([li1])