spree_core 3.2.9 → 3.3.0.rc1

data/spec/models/spree/stock/packer_spec.rb

@@ -0,0 +1,70 @@
+require 'spec_helper'
+
+module Spree
+  module Stock
+    describe Packer, type: :model do
+      let(:inventory_units) { [InventoryUnit.new(variant: create(:variant))] }
+      let(:stock_location) { create(:stock_location) }
+
+      subject { Packer.new(stock_location, inventory_units) }
+
+      context 'packages' do
+        it 'builds an array of packages' do
+          packages = subject.packages
+          expect(packages).to be_a Array
+          expect(packages.first).to be_a Package
+        end
+
+        it 'allows users to set splitters to an empty array' do
+          packer = Packer.new(StockLocation.new, [], [])
+          expect(packer).not_to receive(:build_splitter)
+          packages = packer.packages
+        end
+      end
+
+      context 'default_package' do
+        let!(:inventory_units) { 2.times.map { InventoryUnit.new variant: create(:variant) } }
+
+        it 'contains all the items' do
+          package = subject.default_package
+          expect(package.contents.size).to eq 2
+        end
+
+        it 'variants are added as backordered without enough on_hand' do
+          expect(stock_location).to receive(:fill_status).exactly(2).times.and_return(
+            *(Array.new(1, [1,0]) + Array.new(1, [0,1]))
+          )
+
+          package = subject.default_package
+          expect(package.on_hand.size).to eq 1
+          expect(package.backordered.size).to eq 1
+        end
+
+        context "location doesn't have order items in stock" do
+          let(:stock_location) { create(:stock_location, propagate_all_variants: false) }
+          let(:inventory_units) { [InventoryUnit.new(variant: create(:variant))] }
+          let(:packer) { Packer.new(stock_location, inventory_units) }
+
+          it "builds an empty package" do
+            expect(packer.default_package.contents).to be_empty
+          end
+        end
+
+        context "doesn't track inventory levels" do
+          let(:inventory_units) { 2.times.map { InventoryUnit.new(variant: create(:variant)) } }
+
+          before { Config.track_inventory_levels = false }
+
+          it "doesn't bother stock items status in stock location" do
+            expect(subject.stock_location).not_to receive(:fill_status)
+            subject.default_package
+          end
+
+          it "still creates package with proper quantity" do
+            expect(subject.default_package.quantity).to eql 2
+          end
+        end
+      end
+    end
+  end
+end

data/spec/models/spree/stock/prioritizer_spec.rb

@@ -0,0 +1,125 @@
+require 'spec_helper'
+
+module Spree
+  module Stock
+    describe Prioritizer, type: :model do
+      let(:variant) { build(:variant, weight: 25.0) }
+      let(:stock_location) { build(:stock_location) }
+      let(:order) { build(:order) }
+
+      def inventory_units
+        @inventory_units ||= []
+      end
+
+      def build_inventory_unit
+        build(:inventory_unit, variant: variant).tap do |unit|
+          inventory_units << unit
+        end
+      end
+
+      def pack
+        package = Package.new(stock_location)
+        yield(package) if block_given?
+        package
+      end
+
+      it 'keeps a single package' do
+        package1 = pack do |package|
+          package.add build_inventory_unit
+          package.add build_inventory_unit
+        end
+
+        packages = [package1]
+        prioritizer = Prioritizer.new(packages)
+        packages = prioritizer.prioritized_packages
+        expect(packages.size).to eq 1
+      end
+
+      it 'removes duplicate packages' do
+        package1 = pack do |package|
+          package.add build_inventory_unit
+          package.add build_inventory_unit
+        end
+
+        package2 = pack do |package|
+          package.add inventory_units.first
+          package.add inventory_units.last
+        end
+
+        packages = [package1, package2]
+        prioritizer = Prioritizer.new(packages)
+        packages = prioritizer.prioritized_packages
+        expect(packages.size).to eq 1
+      end
+
+      it 'split over 2 packages' do
+        package1 = pack do |package|
+          package.add build_inventory_unit
+        end
+        package2 = pack do |package|
+          package.add build_inventory_unit
+        end
+
+        packages = [package1, package2]
+        prioritizer = Prioritizer.new(packages)
+        packages = prioritizer.prioritized_packages
+        expect(packages.size).to eq 2
+      end
+
+      it '1st has some, 2nd has remaining' do
+        5.times { build_inventory_unit }
+
+        package1 = pack do |package|
+          2.times { |i| package.add inventory_units[i] }
+        end
+        package2 = pack do |package|
+          5.times { |i| package.add inventory_units[i] }
+        end
+
+        packages = [package1, package2]
+        prioritizer = Prioritizer.new(packages)
+        packages = prioritizer.prioritized_packages
+        expect(packages.count).to eq 2
+        expect(packages[0].quantity).to eq 2
+        expect(packages[1].quantity).to eq 3
+      end
+
+      it '1st has backorder, 2nd has some' do
+        5.times { build_inventory_unit }
+
+        package1 = pack do |package|
+          5.times { |i| package.add inventory_units[i], :backordered }
+        end
+        package2 = pack do |package|
+          2.times { |i| package.add inventory_units[i] }
+        end
+
+        packages = [package1, package2]
+        prioritizer = Prioritizer.new(packages)
+        packages = prioritizer.prioritized_packages
+
+        expect(packages[0].quantity(:backordered)).to eq 3
+        expect(packages[1].quantity(:on_hand)).to eq 2
+      end
+
+      it '1st has backorder, 2nd has all' do
+        5.times { build_inventory_unit }
+
+        package1 = pack do |package|
+          3.times { |i| package.add inventory_units[i], :backordered }
+        end
+        package2 = pack do |package|
+          5.times { |i| package.add inventory_units[i] }
+        end
+
+        packages = [package1, package2]
+        prioritizer = Prioritizer.new(packages)
+        packages = prioritizer.prioritized_packages
+        expect(packages[0]).to eq package2
+        expect(packages[1]).to be_nil
+        expect(packages[0].quantity(:backordered)).to eq 0
+        expect(packages[0].quantity(:on_hand)).to eq 5
+      end
+    end
+  end
+end

data/spec/models/spree/stock/quantifier_spec.rb

@@ -0,0 +1,126 @@
+require 'spec_helper'
+
+shared_examples_for 'unlimited supply' do
+  it 'can_supply? any amount' do
+    expect(subject.can_supply?(1)).to be true
+    expect(subject.can_supply?(101)).to be true
+    expect(subject.can_supply?(100_001)).to be true
+  end
+end
+
+module Spree
+  module Stock
+    describe Quantifier, type: :model do
+      before(:all) { Spree::StockLocation.destroy_all } # FIXME leaky database
+
+      let!(:stock_location) { create :stock_location_with_items  }
+      let!(:stock_item) { stock_location.stock_items.order(:id).first }
+
+      subject { described_class.new(stock_item.variant) }
+
+      specify { expect(subject.stock_items).to eq([stock_item]) }
+      specify { expect(subject.variant).to eq(stock_item.variant) }
+
+      context 'with a single stock location/item' do
+        it 'total_on_hand should match stock_item' do
+          expect(subject.total_on_hand).to eq(stock_item.count_on_hand)
+        end
+
+        context 'when variant is available' do
+          before(:each) do
+            allow(subject.variant).to receive(:available?) { true }
+          end
+
+          context 'when track_inventory_levels is false' do
+            before { configure_spree_preferences { |config| config.track_inventory_levels = false } }
+
+            specify { expect(subject.total_on_hand).to eq(Float::INFINITY) }
+
+            it_should_behave_like 'unlimited supply'
+          end
+
+          context 'when variant inventory tracking is off' do
+            before { stock_item.variant.track_inventory = false }
+
+            specify { expect(subject.total_on_hand).to eq(Float::INFINITY) }
+
+            it_should_behave_like 'unlimited supply'
+          end
+
+          context 'when stock item allows backordering' do
+
+            specify { expect(subject.backorderable?).to be true }
+
+            it_should_behave_like 'unlimited supply'
+          end
+
+          context 'when stock item prevents backordering' do
+            before { stock_item.update_attributes(backorderable: false) }
+
+            specify { expect(subject.backorderable?).to be false }
+
+            it 'can_supply? only upto total_on_hand' do
+              expect(subject.can_supply?(1)).to be true
+              expect(subject.can_supply?(10)).to be true
+              expect(subject.can_supply?(11)).to be false
+            end
+          end
+        end
+
+        context 'when variant is not available' do
+          before(:each) do
+            allow(subject.variant).to receive(:available?) { false }
+          end
+
+          it { expect(subject.can_supply?).to be false }
+        end
+      end
+
+      context 'with multiple stock locations/items' do
+        let!(:stock_location_2) { create :stock_location }
+        let!(:stock_location_3) { create :stock_location, active: false }
+
+        before do
+          stock_location_2.stock_items.where(variant_id: stock_item.variant).update_all(count_on_hand: 5, backorderable: false)
+          stock_location_3.stock_items.where(variant_id: stock_item.variant).update_all(count_on_hand: 5, backorderable: false)
+        end
+
+        it 'total_on_hand should total all active stock_items' do
+          expect(subject.total_on_hand).to eq(15)
+        end
+
+        context 'when variant is available' do
+          before(:each) do
+            allow(subject.variant).to receive(:available?) { true }
+          end
+
+          context 'when any stock item allows backordering' do
+            specify { expect(subject.backorderable?).to be true }
+
+            it_should_behave_like 'unlimited supply'
+          end
+
+          context 'when all stock items prevent backordering' do
+            before { stock_item.update_attributes(backorderable: false) }
+
+            specify { expect(subject.backorderable?).to be false }
+
+            it 'can_supply? upto total_on_hand' do
+              expect(subject.can_supply?(1)).to be true
+              expect(subject.can_supply?(15)).to be true
+              expect(subject.can_supply?(16)).to be false
+            end
+          end
+        end
+
+        context 'when variant is not available' do
+          before(:each) do
+            allow(subject.variant).to receive(:available?) { false }
+          end
+
+          it { expect(subject.can_supply?).to be false }
+        end
+      end
+    end
+  end
+end

data/spec/models/spree/stock/splitter/backordered_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+module Spree
+  module Stock
+    module Splitter
+      describe Backordered, type: :model do
+        let(:variant) { build(:variant) }
+
+        let(:packer) { build(:stock_packer) }
+
+        subject { Backordered.new(packer) }
+
+        it 'splits packages by status' do
+          package = Package.new(packer.stock_location)
+          4.times { package.add build(:inventory_unit, variant: variant) }
+          5.times { package.add build(:inventory_unit, variant: variant), :backordered }
+
+          packages = subject.split([package])
+          expect(packages.count).to eq 2
+          expect(packages.first.quantity).to eq 4
+          expect(packages.first.on_hand.count).to eq 4
+          expect(packages.first.backordered.count).to eq 0
+
+          expect(packages[1].contents.count).to eq 5
+        end
+      end
+    end
+  end
+end

data/spec/models/spree/stock/splitter/base_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+module Spree
+  module Stock
+    module Splitter
+      describe Base, type: :model do
+        let(:packer) { build(:stock_packer) }
+
+        it 'continues to splitter chain' do
+          splitter1 = Base.new(packer)
+          splitter2 = Base.new(packer, splitter1)
+          packages = []
+
+          expect(splitter1).to receive(:split).with(packages)
+          splitter2.split(packages)
+        end
+
+      end
+    end
+  end
+end

data/spec/models/spree/stock/splitter/shipping_category_spec.rb

@@ -0,0 +1,47 @@
+require 'spec_helper'
+
+module Spree
+  module Stock
+    module Splitter
+      describe ShippingCategory, type: :model do
+
+        let(:variant1) { create(:variant) }
+        let(:variant2) { create(:variant) }
+        let(:shipping_category_1) { create(:shipping_category, name: 'A') }
+        let(:shipping_category_2) { create(:shipping_category, name: 'B') }
+
+        def inventory_unit1
+          InventoryUnit.new(variant: variant1).tap do |inventory_unit|
+            inventory_unit.variant.product.shipping_category = shipping_category_1
+          end
+        end
+
+        def inventory_unit2
+          InventoryUnit.new(variant: variant2).tap do |inventory_unit|
+            inventory_unit.variant.product.shipping_category = shipping_category_2
+          end
+        end
+
+        let(:packer) { build(:stock_packer) }
+
+        subject { ShippingCategory.new(packer) }
+
+        it 'splits each package by shipping category' do
+          package1 = Package.new(packer.stock_location)
+          4.times { package1.add inventory_unit1 }
+          8.times { package1.add inventory_unit2 }
+
+          package2 = Package.new(packer.stock_location)
+          6.times { package2.add inventory_unit1 }
+          9.times { package2.add inventory_unit2, :backordered }
+
+          packages = subject.split([package1, package2])
+          expect(packages[0].quantity).to eq 4
+          expect(packages[1].quantity).to eq 8
+          expect(packages[2].quantity).to eq 6
+          expect(packages[3].quantity).to eq 9
+        end
+      end
+    end
+  end
+end

data/spec/models/spree/stock/splitter/weight_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+module Spree
+  module Stock
+    module Splitter
+      describe Weight, type: :model do
+        let(:packer) { build(:stock_packer) }
+        let(:heavy_variant) { build(:base_variant, weight: 100) }
+        let(:variant) { build(:base_variant, weight: 49) }
+
+        subject { Weight.new(packer) }
+
+        it 'splits and keeps splitting until all packages are underweight' do
+          package = Package.new(packer.stock_location)
+          2.times { package.add build(:inventory_unit, variant: heavy_variant) }
+          4.times { package.add build(:inventory_unit, variant: variant) }
+          2.times { package.add build(:inventory_unit, variant: heavy_variant) }
+          packages = subject.split([package])
+          expect(packages.size).to eq 4
+        end
+
+        it 'handles packages that can not be reduced' do
+          package = Package.new(packer.stock_location)
+          allow(variant).to receive_messages(weight: 200)
+          2.times { package.add build(:inventory_unit, variant: variant) }
+          packages = subject.split([package])
+          expect(packages.size).to eq 2
+        end
+      end
+    end
+  end
+end