solidus_core 2.10.2 → 2.11.3

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

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

@@ -1,197 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'shared_examples/calculator_shared_examples'
-
-RSpec.describe Spree::Calculator::TieredPercent, type: :model do
-  let(:calculator) { Spree::Calculator::TieredPercent.new }
-
-  it_behaves_like 'a calculator with a description'
-
-  describe "#valid?" do
-    subject { calculator.valid? }
-
-    context "when base percent is less than zero" do
-      before { calculator.preferred_base_percent = -1 }
-      it { is_expected.to be false }
-    end
-
-    context "when base percent is greater than 100" do
-      before { calculator.preferred_base_percent = 110 }
-      it { is_expected.to be false }
-    end
-
-    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 one of the values is not a percent" do
-        before { calculator.preferred_tiers = { 10 => 110 } }
-        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: line_item_count,
-        line_items_price: price
-      )
-    end
-    let(:price) { 10 }
-    let(:preferred_currency) { "USD" }
-
-    before do
-      calculator.preferred_base_percent = 10
-      calculator.preferred_tiers = {
-        20 => 15,
-        30 => 20
-      }
-      calculator.preferred_currency = preferred_currency
-    end
-
-    context "with a line item" do
-      let(:line_item) { order.line_items.first }
-      subject { calculator.compute(line_item) }
-
-      context "for multiple line items" do
-        context "when amount falls within the first tier" do
-          let(:line_item_count) { 1 }
-          it { is_expected.to eq 1.0 }
-        end
-
-        context "when amount falls within the second tier" do
-          let(:line_item_count) { 2 }
-          it { is_expected.to eq 1.5 }
-        end
-
-        context "when amount falls within the third tier" do
-          let(:line_item_count) { 3 }
-          it { is_expected.to eq 2.0 }
-        end
-      end
-
-      context "for a single line item" do
-        let(:line_item_count) { 1 }
-
-        context "when amount falls within the first tier" do
-          let(:price) { 10 }
-          it { is_expected.to eq 1.0 }
-        end
-
-        context "when amount falls within the second tier" do
-          let(:price) { 20 }
-          it { is_expected.to eq 3.0 }
-        end
-
-        context "when amount falls within the third tier" do
-          let(:price) { 30 }
-          it { is_expected.to eq 6.0 }
-        end
-      end
-
-      context "when the order's currency does not match the calculator" do
-        let(:preferred_currency) { "JPY" }
-        let(:line_item_count) { 1 }
-        let(:price) { 15 }
-        it { is_expected.to eq 0 }
-
-        it "rounds based on currency" do
-          allow(order).to receive_messages currency: "JPY"
-          expect(subject).to eq(2)
-        end
-      end
-    end
-
-    context "with an order" do
-      subject { calculator.compute(order) }
-
-      let(:line_item_count) { 1 }
-
-      context "for multiple line items" do
-        context "when amount falls within the first tier" do
-          let(:line_item_count) { 1 }
-          it { is_expected.to eq 1.0 }
-        end
-
-        context "when amount falls within the second tier" do
-          let(:line_item_count) { 2 }
-          it { is_expected.to eq 3.0 }
-        end
-
-        context "when amount falls within the third tier" do
-          let(:line_item_count) { 3 }
-          it { is_expected.to eq 6.0 }
-        end
-      end
-
-      context "for a single line item" do
-        let(:line_item_count) { 1 }
-
-        context "when amount falls within the first tier" do
-          let(:price) { 10 }
-          it { is_expected.to eq 1.0 }
-        end
-
-        context "when amount falls within the second tier" do
-          let(:price) { 20 }
-          it { is_expected.to eq 3.0 }
-        end
-
-        context "when amount falls within the third tier" do
-          let(:price) { 30 }
-          it { is_expected.to eq 6.0 }
-        end
-      end
-
-      context "when the order's currency does not match the calculator" do
-        let(:preferred_currency) { "CAD" }
-        it { is_expected.to eq 0 }
-      end
-    end
-  end
-end

data/spec/models/spree/calculator_spec.rb

@@ -1,43 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::Calculator, type: :model do
-  class SimpleCalculator < Spree::Calculator
-    def compute_simple_computable(_line_item)
-      'computed'
-    end
-  end
-
-  class SimpleComputable
-  end
-
-  describe "#calculators" do
-    subject { Spree::Calculator.calculators }
-
-    it 'returns the (deprecated) calculator step' do
-      Spree::Deprecation.silence do
-        expect(subject).to be_a Spree::Core::Environment::Calculators
-      end
-    end
-  end
-
-  context "with computable" do
-    let(:calculator) { SimpleCalculator.new }
-    let(:computable) { SimpleComputable.new }
-
-    subject { SimpleCalculator.new.compute computable }
-
-    it 'calls compute method of class type' do
-      expect(subject).to eq  'computed'
-    end
-
-    context 'computable does not implement right function name' do
-      let(:computable) { Spree::LineItem.new }
-
-      it 'raises an error' do
-        expect { subject }.to raise_error NotImplementedError, /Please implement \'compute_line_item\(line_item\)\' in your calculator/
-      end
-    end
-  end
-end

data/spec/models/spree/carton_spec.rb

@@ -1,145 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::Carton do
-  let(:carton) { create(:carton) }
-
-  describe 'shipping method' do
-    it 'returns soft deleted shipping method' do
-      carton = create(:carton)
-      carton.shipping_method.discard
-      expect(carton.reload.shipping_method).to be_present
-    end
-  end
-
-  describe "#create" do
-    subject { carton }
-
-    it "raises no errors" do
-      subject
-    end
-  end
-
-  describe "#tracking_url" do
-    subject do
-      carton.tracking_url
-    end
-
-    let(:carton) { create(:carton, shipping_method: shipping_method) }
-    let(:shipping_method) do
-      create(:shipping_method, tracking_url: "https://example.com/:tracking")
-    end
-
-    context "when tracking is not present" do
-      it { is_expected.to be_nil }
-    end
-
-    context "when tracking is present" do
-      let(:carton) do
-        create(:carton, shipping_method: shipping_method, tracking: "1Z12345")
-      end
-
-      it "uses shipping method to determine url" do
-        is_expected.to eq("https://example.com/1Z12345")
-      end
-    end
-  end
-
-  describe "#to_param" do
-    subject do
-      carton.to_param
-    end
-
-    it { is_expected.to eq carton.number }
-  end
-
-  describe "#order_numbers" do
-    subject { carton.order_numbers }
-    let(:order) { carton.orders.first }
-
-    it "returns a list of the order numbers it is associated to" do
-      expect(subject).to eq [order.number]
-    end
-  end
-
-  describe "#shipment_numbers" do
-    subject { carton.shipment_numbers }
-    let(:shipment) { carton.shipments.first }
-
-    it "returns a list of the shipment numbers it is associated to" do
-      expect(subject).to eq [shipment.number]
-    end
-  end
-
-  describe "#order_emails" do
-    subject { carton.order_emails }
-
-    let(:carton) { create(:carton, inventory_units: [first_order.inventory_units, second_order.inventory_units].flatten) }
-    let(:first_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:second_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:email) { 'something@something.com' }
-
-    before do
-      first_order.update!(email: email)
-      second_order.update!(email: email)
-    end
-
-    it "returns a unique list of the order emails it is associated to" do
-      expect(subject).to eq [email]
-    end
-  end
-
-  describe "#manifest" do
-    subject { carton.manifest }
-
-    let(:carton) { create(:carton, inventory_units: [first_order.inventory_units, second_order.inventory_units].flatten) }
-    let(:first_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:first_line_item) { first_order.line_items.first }
-    let(:second_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:second_line_item) { second_order.line_items.first }
-
-    it "contains only the items in both the carton and order" do
-      expect(subject.map(&:line_item)).to match_array([first_line_item, second_line_item])
-    end
-  end
-
-  describe "#manifest_for_order" do
-    subject { carton.manifest_for_order(first_order) }
-
-    let(:carton) { create(:carton, inventory_units: [first_order.inventory_units, second_order.inventory_units].flatten) }
-    let(:first_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:first_line_item) { first_order.line_items.first }
-    let(:second_order) { create(:order_ready_to_ship, line_items_count: 1) }
-
-    it "contains only the items in both the carton and order" do
-      expect(subject.map(&:line_item)).to eq [first_line_item]
-    end
-  end
-
-  describe "#any_exchanges?" do
-    subject { carton.any_exchanges? }
-
-    let(:carton) { create(:carton, inventory_units: [first_order.inventory_units, second_order.inventory_units].flatten) }
-    let(:first_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:second_order) { create(:order_ready_to_ship, line_items_count: 1) }
-
-    context "when any of the inventory has an original return item" do
-      let(:return_item) { create(:return_item) }
-      before do
-        first_order.inventory_units.first.original_return_item = return_item
-        first_order.save
-      end
-
-      it "is true" do
-        expect(subject).to be_truthy
-      end
-    end
-
-    context "when none of the inventory has an original return item" do
-      it "is false" do
-        expect(subject).to be_falsey
-      end
-    end
-  end
-end