solidus_core 2.10.0 → 2.11.0

data/spec/models/spree/calculator/flat_percent_item_total_spec.rb

@@ -1,39 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-RSpec.describe Spree::Calculator::FlatPercentItemTotal, type: :model do
-  let(:calculator) { Spree::Calculator::FlatPercentItemTotal.new }
-  let(:line_item) { create(:line_item) }
-
-  it_behaves_like 'a calculator with a description'
-
-  before { allow(calculator).to receive_messages preferred_flat_percent: 10 }
-
-  context "compute" do
-    it "should round result correctly" do
-      allow(line_item).to receive_messages amount: 31.08
-      expect(calculator.compute(line_item)).to eq 3.11
-
-      allow(line_item).to receive_messages amount: 31.00
-      expect(calculator.compute(line_item)).to eq 3.10
-    end
-
-    it "should round result based on order currency" do
-      line_item.order.currency = 'JPY'
-      allow(line_item).to receive_messages amount: 31.08
-      expect(calculator.compute(line_item)).to eq 3
-
-      allow(line_item).to receive_messages amount: 31.00
-      expect(calculator.compute(line_item)).to eq 3
-    end
-
-    it 'returns object.amount if computed amount is greater' do
-      allow(calculator).to receive_messages preferred_flat_percent: 110
-      allow(line_item).to receive_messages amount: 30.00
-
-      expect(calculator.compute(line_item)).to eq 30.0
-    end
-  end
-end

data/spec/models/spree/calculator/flat_rate_spec.rb

@@ -1,52 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-RSpec.describe Spree::Calculator::FlatRate, type: :model do
-  let(:calculator) { Spree::Calculator::FlatRate.new }
-
-  it_behaves_like 'a calculator with a description'
-
-  let(:order) do
-    mock_model(
-      Spree::Order, quantity: 10, currency: "USD"
-    )
-  end
-
-  context "compute" do
-    it "should compute the amount as the rate when currency matches the order's currency" do
-      calculator.preferred_amount = 25.0
-      calculator.preferred_currency = "GBP"
-      allow(order).to receive_messages currency: "GBP"
-      expect(calculator.compute(order).round(2)).to eq(25.0)
-    end
-
-    it "should compute the amount as 0 when currency does not match the order's currency" do
-      calculator.preferred_amount = 100.0
-      calculator.preferred_currency = "GBP"
-      allow(order).to receive_messages currency: "USD"
-      expect(calculator.compute(order).round(2)).to eq(0.0)
-    end
-
-    it "should compute the amount as 0 when currency is blank" do
-      calculator.preferred_amount = 100.0
-      calculator.preferred_currency = ""
-      allow(order).to receive_messages currency: "GBP"
-      expect(calculator.compute(order).round(2)).to eq(0.0)
-    end
-
-    it "should compute the amount as the rate when the currencies use different casing" do
-      calculator.preferred_amount = 100.0
-      calculator.preferred_currency = "gBp"
-      allow(order).to receive_messages currency: "GBP"
-      expect(calculator.compute(order).round(2)).to eq(100.0)
-    end
-
-    it "should compute the amount as 0 when there is no object" do
-      calculator.preferred_amount = 100.0
-      calculator.preferred_currency = "GBP"
-      expect(calculator.compute.round(2)).to eq(0.0)
-    end
-  end
-end

data/spec/models/spree/calculator/flexi_rate_spec.rb

@@ -1,158 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-RSpec.describe Spree::Calculator::FlexiRate, type: :model do
-  let(:calculator) do
-    Spree::Calculator::FlexiRate.new(
-      preferred_first_item: first_item,
-      preferred_additional_item: additional_item,
-      preferred_max_items: max_items
-    )
-  end
-  let(:first_item) { 0 }
-  let(:additional_item) { 0 }
-  let(:max_items) { 0 }
-
-  it_behaves_like 'a calculator with a description'
-
-  let(:order) do
-    mock_model(
-      Spree::Order, quantity: quantity
-    )
-  end
-
-  context "compute" do
-    subject { calculator.compute(order) }
-    context "with all amounts 0" do
-      context "with quantity 0" do
-        let(:quantity) { 0 }
-        it { should eq 0 }
-      end
-
-      context "with quantity 1" do
-        let(:quantity) { 1 }
-        it { should eq 0 }
-      end
-
-      context "with quantity 2" do
-        let(:quantity) { 2 }
-        it { should eq 0 }
-      end
-
-      context "with quantity 10" do
-        let(:quantity) { 10 }
-        it { should eq 0 }
-      end
-    end
-
-    context "when first_item has a value" do
-      let(:first_item) { 1.23 }
-
-      context "with quantity 0" do
-        let(:quantity) { 0 }
-        it { should eq 0 }
-      end
-
-      context "with quantity 1" do
-        let(:quantity) { 1 }
-        it { should eq 1.23 }
-      end
-
-      context "with quantity 2" do
-        let(:quantity) { 2 }
-        it { should eq 1.23 }
-      end
-
-      context "with quantity 10" do
-        let(:quantity) { 10 }
-        it { should eq 1.23 }
-      end
-    end
-
-    context "when additional_items has a value" do
-      let(:additional_item) { 1.23 }
-
-      context "with quantity 0" do
-        let(:quantity) { 0 }
-        it { should eq 0 }
-      end
-
-      context "with quantity 1" do
-        let(:quantity) { 1 }
-        it { should eq 0 }
-      end
-
-      context "with quantity 2" do
-        let(:quantity) { 2 }
-        it { should eq 1.23 }
-      end
-
-      context "with quantity 10" do
-        let(:quantity) { 10 }
-        it { should eq 11.07 }
-      end
-    end
-
-    context "when first_item and additional_items has a value" do
-      let(:first_item) { 1.13 }
-      let(:additional_item) { 2.11 }
-
-      context "with quantity 0" do
-        let(:quantity) { 0 }
-        it { should eq 0 }
-      end
-
-      context "with quantity 1" do
-        let(:quantity) { 1 }
-        it { should eq 1.13 }
-      end
-
-      context "with quantity 2" do
-        let(:quantity) { 2 }
-        it { should eq 3.24 }
-      end
-
-      context "with quantity 10" do
-        let(:quantity) { 10 }
-        it { should eq 20.12 }
-      end
-
-      context "with max_items 5" do
-        let(:max_items) { 5 }
-
-        context "with quantity 0" do
-          let(:quantity) { 0 }
-          it { should eq 0 }
-        end
-
-        context "with quantity 1" do
-          let(:quantity) { 1 }
-          it { should eq 1.13 }
-        end
-
-        context "with quantity 2" do
-          let(:quantity) { 2 }
-          it { should eq 3.24 }
-        end
-
-        context "with quantity 5" do
-          let(:quantity) { 5 }
-          it { should eq 9.57 }
-        end
-
-        context "with quantity 10" do
-          let(:quantity) { 10 }
-          it { should eq 9.57 }
-        end
-      end
-    end
-  end
-
-  it "should allow creation of new object with all the attributes" do
-    attributes = { preferred_first_item: 1, preferred_additional_item: 1, preferred_max_items: 1 }
-    calculator = Spree::Calculator::FlexiRate.new(attributes)
-    expect(calculator).to have_attributes(attributes)
-  end
-end

data/spec/models/spree/calculator/free_shipping_spec.rb

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-RSpec.describe Spree::Calculator::FreeShipping, type: :model do
-  it_behaves_like 'a calculator with a description'
-end

data/spec/models/spree/calculator/percent_on_line_item_spec.rb

@@ -1,20 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-module Spree
-  RSpec.describe Calculator::PercentOnLineItem, type: :model do
-    context "compute" do
-      let(:line_item) { double("LineItem", amount: 100) }
-
-      before { subject.preferred_percent = 15 }
-
-      it "computes based on item price and quantity" do
-        expect(subject.compute(line_item)).to eq 15
-      end
-    end
-
-    it_behaves_like 'a calculator with a description'
-  end
-end

data/spec/models/spree/calculator/percent_per_item_spec.rb

@@ -1,12 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-require_dependency 'spree/calculator'
-
-module Spree
-  RSpec.describe Calculator::PercentPerItem, type: :model do
-    it_behaves_like 'a calculator with a description'
-  end
-end

data/spec/models/spree/calculator/price_sack_spec.rb

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-RSpec.describe Spree::Calculator::PriceSack, type: :model do
-  let(:calculator) do
-    calculator = Spree::Calculator::PriceSack.new
-    calculator.preferred_minimal_amount = 5
-    calculator.preferred_normal_amount = 10
-    calculator.preferred_discount_amount = 1
-    calculator
-  end
-
-  it_behaves_like 'a calculator with a description'
-
-  let(:order) { stub_model(Spree::Order) }
-  let(:shipment) { stub_model(Spree::Shipment, amount: 10) }
-
-  # Regression test for https://github.com/spree/spree/issues/714 and https://github.com/spree/spree/issues/739
-  it "computes with an order object" do
-    calculator.compute(order)
-  end
-
-  # Regression test for https://github.com/spree/spree/issues/1156
-  it "computes with a shipment object" do
-    calculator.compute(shipment)
-  end
-
-  # Regression test for https://github.com/spree/spree/issues/2055
-  it "computes the correct amount" do
-    expect(calculator.compute(2)).to eq(calculator.preferred_normal_amount)
-    expect(calculator.compute(6)).to eq(calculator.preferred_discount_amount)
-  end
-end

data/spec/models/spree/calculator/refunds/default_refund_amount_spec.rb

@@ -1,70 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-RSpec.describe Spree::Calculator::Returns::DefaultRefundAmount, type: :model do
-  let(:line_item_quantity) { 3 }
-  let(:line_item_price) { 100.0 }
-  let(:line_item) { create(:line_item, price: line_item_price, quantity: line_item_quantity) }
-  let(:shipment) { create(:shipment, order: order) }
-  let(:inventory_unit) { build(:inventory_unit, shipment: shipment, line_item: line_item) }
-  let(:return_item) { build(:return_item, inventory_unit: inventory_unit ) }
-  let(:calculator) { Spree::Calculator::Returns::DefaultRefundAmount.new }
-  let(:order) { line_item.order }
-
-  it_behaves_like 'a calculator with a description'
-
-  subject { calculator.compute(return_item) }
-
-  context "not an exchange" do
-    context "no promotions or taxes" do
-      it { is_expected.to eq line_item_price }
-    end
-
-    context "order adjustments" do
-      let(:adjustment_amount) { -10.0 }
-
-      before do
-        order.adjustments << create(
-          :adjustment,
-          adjustable:  order,
-          order:       order,
-          amount:      adjustment_amount,
-          eligible:    true,
-          label:       'Adjustment',
-          source_type: 'Spree::Order'
-        )
-
-        order.adjustments.first.update(amount: adjustment_amount)
-      end
-
-      it 'will return the line item amount deducted of refund' do
-        # line_item_price    = 100
-        # line_item_quantity = 3
-        # adjustment_amount  = 10
-        # 100 - (10 / 3)     = 96.66666666666666667
-        expect(subject).to eq BigDecimal('96.66666666666666667')
-      end
-    end
-
-    context "shipping adjustments" do
-      let(:adjustment_total) { -50.0 }
-
-      before { order.shipments << Spree::Shipment.new(adjustment_total: adjustment_total) }
-
-      it { is_expected.to eq line_item_price }
-    end
-  end
-
-  context "an exchange" do
-    let(:return_item) { build(:exchange_return_item) }
-
-    it { is_expected.to eq 0.0 }
-  end
-
-  context "line item amount is zero" do
-    let(:line_item_price) { 0 }
-    it { is_expected.to eq 0.0 }
-  end
-end

data/spec/models/spree/calculator/shipping/flat_percent_item_total_spec.rb

@@ -1,44 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-module Spree
-  module Calculator::Shipping
-    RSpec.describe FlatPercentItemTotal, type: :model do
-      it_behaves_like 'a calculator with a description'
-
-      let(:line_item1) { build(:line_item, price: 10.11) }
-      let(:line_item2) { build(:line_item, price: 20.2222) }
-
-      let(:inventory_unit1) { build(:inventory_unit, line_item: line_item1) }
-      let(:inventory_unit2) { build(:inventory_unit, line_item: line_item2) }
-
-      let(:package) do
-        build(
-          :stock_package,
-          contents: [
-            Spree::Stock::ContentItem.new(inventory_unit1),
-            Spree::Stock::ContentItem.new(inventory_unit1),
-            Spree::Stock::ContentItem.new(inventory_unit2),
-          ]
-        )
-      end
-
-      subject { described_class.new(preferred_flat_percent: 10) }
-
-      it "should round result correctly" do
-        expect(subject.compute(package)).to eq(4.04)
-      end
-
-      it "should round result based on order currency" do
-        package.order.currency = 'JPY'
-        expect(subject.compute(package)).to eq(4)
-      end
-
-      it "should return a bigdecimal" do
-        expect(subject.compute(package)).to be_a(BigDecimal)
-      end
-    end
-  end
-end

data/spec/models/spree/calculator/shipping/flat_rate_spec.rb

@@ -1,18 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-module Spree
-  module Calculator::Shipping
-    RSpec.describe FlatRate, type: :model do
-      subject { described_class.new(preferred_amount: 4.00) }
-
-      it_behaves_like 'a calculator with a description'
-
-      it 'always returns the same rate' do
-        expect(subject.compute(build(:stock_package_fulfilled))).to eql 4.00
-      end
-    end
-  end
-end

data/spec/models/spree/calculator/shipping/flexi_rate_spec.rb

@@ -1,56 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-module Spree
-  module Calculator::Shipping
-    RSpec.describe FlexiRate, type: :model do
-      let(:variant1) { build(:variant, price: 10) }
-      let(:variant2) { build(:variant, price: 20) }
-
-      it_behaves_like 'a calculator with a description'
-
-      let(:package) do
-        build(:stock_package, variants_contents: { variant1 => 4, variant2 => 6 })
-      end
-
-      let(:subject) { described_class.new }
-
-      context "compute" do
-        it "should compute amount correctly when all fees are 0" do
-          expect(subject.compute(package).round(2)).to eq(0.0)
-        end
-
-        it "should compute amount correctly when first_item has a value" do
-          subject.preferred_first_item = 1.0
-          expect(subject.compute(package).round(2)).to eq(1.0)
-        end
-
-        it "should compute amount correctly when additional_items has a value" do
-          subject.preferred_additional_item = 1.0
-          expect(subject.compute(package).round(2)).to eq(9.0)
-        end
-
-        it "should compute amount correctly when additional_items and first_item have values" do
-          subject.preferred_first_item = 5.0
-          subject.preferred_additional_item = 1.0
-          expect(subject.compute(package).round(2)).to eq(14.0)
-        end
-
-        it "should compute amount correctly when additional_items and first_item have values AND max items has value" do
-          subject.preferred_first_item = 5.0
-          subject.preferred_additional_item = 1.0
-          subject.preferred_max_items = 3
-          expect(subject.compute(package).round(2)).to eq(26.0)
-        end
-
-        it "should allow creation of new object with all the attributes" do
-          described_class.new(preferred_first_item: 1,
-                        preferred_additional_item: 1,
-                        preferred_max_items: 1)
-        end
-      end
-    end
-  end
-end

data/spec/models/spree/calculator/shipping/per_item_spec.rb

@@ -1,25 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-module Spree
-  module Calculator::Shipping
-    RSpec.describe PerItem, type: :model do
-      let(:variant1) { build(:variant) }
-      let(:variant2) { build(:variant) }
-
-      it_behaves_like 'a calculator with a description'
-
-      let(:package) do
-        build(:stock_package, variants_contents: { variant1 => 5, variant2 => 3 })
-      end
-
-      subject { described_class.new(preferred_amount: 10) }
-
-      it "correctly calculates per item shipping" do
-        expect(subject.compute(package).to_f).to eq(80) # 5 x 10 + 3 x 10
-      end
-    end
-  end
-end

data/spec/models/spree/calculator/shipping/price_sack_spec.rb

@@ -1,44 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-RSpec.describe Spree::Calculator::Shipping::PriceSack, type: :model do
-  let(:calculator) do
-    calculator = described_class.new
-    calculator.preferred_minimal_amount = 5
-    calculator.preferred_normal_amount = 10
-    calculator.preferred_discount_amount = 1
-    calculator
-  end
-
-  it_behaves_like 'a calculator with a description'
-
-  describe '#compute' do
-    subject { calculator.compute(package) }
-    let(:package) { build(:stock_package, variants_contents: { build(:variant) => 1 }) }
-
-    before do
-      # This hack is due to our factories not being so smart to understand
-      # that they should create line items with the price of the associated
-      # variant by default.
-      allow_any_instance_of(Spree::Stock::ContentItem).to receive(:price) { amount }
-    end
-
-    context 'when price < minimal amount' do
-      let(:amount) { 2 }
-
-      it "returns the discounted amount" do
-        expect(subject).to eq(10)
-      end
-    end
-
-    context 'when price > minimal amount' do
-      let(:amount) { 6 }
-
-      it "returns the discounted amount" do
-        expect(subject).to eq(1)
-      end
-    end
-  end
-end

data/spec/models/spree/calculator/tiered_flat_rate_spec.rb

@@ -1,99 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-RSpec.describe Spree::Calculator::TieredFlatRate, type: :model do
-  let(:calculator) { Spree::Calculator::TieredFlatRate.new }
-
-  it_behaves_like 'a calculator with a description'
-
-  describe "#valid?" do
-    subject { calculator.valid? }
-    context "when tiers is a hash" do
-      context "and the key is not a positive number" do
-        before { calculator.preferred_tiers = { "nope" => 20 } }
-        it { is_expected.to be false }
-      end
-
-      context "and the key is an integer" do
-        before { calculator.preferred_tiers = { 20 => 20 } }
-        it "converts successfully" do
-          is_expected.to be true
-          expect(calculator.preferred_tiers).to eq({ BigDecimal('20') => BigDecimal('20') })
-        end
-      end
-
-      context "and the key is a float" do
-        before { calculator.preferred_tiers = { 20.5 => 20.5 } }
-        it "converts successfully" do
-          is_expected.to be true
-          expect(calculator.preferred_tiers).to eq({ BigDecimal('20.5') => BigDecimal('20.5') })
-        end
-      end
-
-      context "and the key is a string number" do
-        before { calculator.preferred_tiers = { "20" => 20 } }
-        it "converts successfully" do
-          is_expected.to be true
-          expect(calculator.preferred_tiers).to eq({ BigDecimal('20') => BigDecimal('20') })
-        end
-      end
-
-      context "and the key is a numeric string with spaces" do
-        before { calculator.preferred_tiers = { "  20 " => 20 } }
-        it "converts successfully" do
-          is_expected.to be true
-          expect(calculator.preferred_tiers).to eq({ BigDecimal('20') => BigDecimal('20') })
-        end
-      end
-
-      context "and the key is a string number with decimals" do
-        before { calculator.preferred_tiers = { "20.5" => "20.5" } }
-        it "converts successfully" do
-          is_expected.to be true
-          expect(calculator.preferred_tiers).to eq({ BigDecimal('20.5') => BigDecimal('20.5') })
-        end
-      end
-    end
-  end
-
-  describe "#compute" do
-    let(:order) do
-      create(
-        :order_with_line_items,
-        line_items_count: 1,
-        line_items_price: amount
-      )
-    end
-    let(:line_item) { order.line_items.first }
-    let(:preferred_currency) { "USD" }
-
-    before do
-      calculator.preferred_base_amount = 10
-      calculator.preferred_tiers = {
-        100 => 15,
-        200 => 20
-      }
-      calculator.preferred_currency = preferred_currency
-    end
-
-    subject { calculator.compute(line_item) }
-
-    context "when amount falls within the first tier" do
-      let(:amount) { 50 }
-      it { is_expected.to eq 10 }
-    end
-
-    context "when amount falls within the second tier" do
-      let(:amount) { 150 }
-      it { is_expected.to eq 15 }
-    end
-
-    context "when the order's currency does not match the calculator" do
-      let(:preferred_currency) { "CAD" }
-      let(:amount) { 50 }
-      it { is_expected.to eq 0 }
-    end
-  end
-end