raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-02-27 01:43:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

Make controller req. params a bit more obvious

Browse Source
This commit is contained in:
Pau Perez
2019-01-21 12:32:48 +01:00
parent f589d4125d
commit 9d1d8513e7
1 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
spec/controllers/spree/orders_controller_spec.rb
View File

@@ -176,10 +176,6 @@ describe Spree::OrdersController, type: :controller do
let(:payment_fee) { 5 }
let(:item_num) { order.line_items.length }
let(:expected_fees) { item_num * (shipping_fee + payment_fee) }
let(:params) { { order: { line_items_attributes: {
"0" => {id: line_item1.id, quantity: 1},
"1" => {id: line_item2.id, quantity: 0}
} } } }
before do
allow(Spree::Config).to receive(:shipment_inc_vat) { true }
@@ -196,10 +192,13 @@ describe Spree::OrdersController, type: :controller do
end
it "updates the fees" do
# Setting quantity of an item to zero
spree_post :update, params
spree_post :update, {
order: { line_items_attributes: {
"0" => { id: line_item1.id, quantity: 1 },
"1" => { id: line_item2.id, quantity: 0 }
} }
}
# Check if fees got updated
order.reload
expect(order.line_items.count).to eq 1
expect(order.adjustment_total).to eq expected_fees - shipping_fee - payment_fee