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

Refactor, use instance_double in variant spec

Browse Source
This commit is contained in:
Gaetan Craig-Riou
2024-06-30 14:54:14 +10:00
parent 1ad7123a9d
commit e2c762f06b
1 changed files with 7 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
spec/models/spree/variant_spec.rb
View File

@@ -696,10 +696,9 @@ RSpec.describe Spree::Variant do
end
describe "calculating the price with enterprise fees" do
# TODO use instance double
it "returns the price plus the fees" do
distributor = double(:distributor)
order_cycle = double(:order_cycle)
distributor = instance_double(Enterprise)
order_cycle = instance_double(OrderCycle)
variant = Spree::Variant.new price: 100
expect(variant).to receive(:fees_for).with(distributor, order_cycle) { 23 }
@@ -709,8 +708,8 @@ RSpec.describe Spree::Variant do
describe "calculating the fees" do
it "delegates to EnterpriseFeeCalculator" do
distributor = double(:distributor)
order_cycle = double(:order_cycle)
distributor = instance_double(Enterprise)
order_cycle = instance_double(OrderCycle)
variant = Spree::Variant.new
expect_any_instance_of(OpenFoodNetwork::EnterpriseFeeCalculator)
@@ -722,10 +721,10 @@ RSpec.describe Spree::Variant do
describe "calculating fees broken down by fee type" do
it "delegates to EnterpriseFeeCalculator" do
distributor = double(:distributor)
order_cycle = double(:order_cycle)
distributor = instance_double(Enterprise)
order_cycle = instance_double(OrderCycle)
variant = Spree::Variant.new
fees = double(:fees)
fees = instance_double(EnterpriseFee)
expect_any_instance_of(OpenFoodNetwork::EnterpriseFeeCalculator)
.to receive(:fees_by_type_for).with(variant) { fees }