From d4750b9f26e1a9d94c0f27ec6cf5abfb990ecde4 Mon Sep 17 00:00:00 2001 From: Matt-Yorkley <9029026+Matt-Yorkley@users.noreply.github.com> Date: Wed, 3 Feb 2021 14:00:14 +0000 Subject: [PATCH] Improve clarity of orders controller test --- spec/controllers/spree/orders_controller_spec.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/spec/controllers/spree/orders_controller_spec.rb b/spec/controllers/spree/orders_controller_spec.rb index cf8ba1cfef..55cf94470f 100644 --- a/spec/controllers/spree/orders_controller_spec.rb +++ b/spec/controllers/spree/orders_controller_spec.rb @@ -242,15 +242,15 @@ describe Spree::OrdersController, type: :controller do allow(subject).to receive(:order_to_update) { order } end - it "updates the fees" do + it "updates the shipping and payment fees" do spree_post :update, order: { line_items_attributes: { "0" => { id: line_item1.id, quantity: 1 }, "1" => { id: line_item2.id, quantity: 0 } } } - expect(order.line_items.count).to eq 1 - expect(order.adjustment_total).to eq((item_num - 1) * (shipping_fee + payment_fee)) + expect(order.reload.line_items.count).to eq 1 + expect(order.adjustment_total).to eq(1 * (shipping_fee + payment_fee)) expect(order.shipment.adjustment.included_tax).to eq 0.6 end end