solidus_core 1.1.0 → 1.1.1

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

@@ -0,0 +1,91 @@
+require 'spec_helper'
+
+module Spree
+  module Stock
+    describe Packer, :type => :model do
+      let!(:inventory_units) { 5.times.map { build(:inventory_unit) } }
+      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.size).to eq 1
+          expect(packages.first.contents.size).to eq 5
+        end
+
+        it 'allows users to set splitters to an empty array' do
+          packages = Packer.new(stock_location, inventory_units, []).packages
+          expect(packages.size).to eq 1
+        end
+      end
+
+      context 'default_package' do
+        it 'contains all the items' do
+          package = subject.default_package
+          expect(package.contents.size).to eq 5
+        end
+
+        it 'variants are added as backordered without enough on_hand' do
+          expect(stock_location).to receive(:fill_status).exactly(5).times.and_return(
+            *(Array.new(3, [1,0]) + Array.new(2, [0,1]))
+          )
+
+          package = subject.default_package
+          expect(package.on_hand.size).to eq 3
+          expect(package.backordered.size).to eq 2
+        end
+
+        context "location doesn't have order items in stock" do
+          let(:stock_location) { create(:stock_location, propagate_all_variants: false) }
+          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 "not enough on hand and not backorderable" do
+          let(:packer) { Packer.new(stock_location, inventory_units) }
+
+          before do
+            stock_location.stock_items.update_all(backorderable: false)
+            stock_location.stock_items.each { |si| si.set_count_on_hand(0) }
+          end
+
+          it "raises an error" do
+            expect { packer.default_package }.to raise_error Spree::Order::InsufficientStock
+          end
+        end
+
+        context "doesn't track inventory levels" do
+          let(:variant) { build(:variant) }
+          let(:order) { build(:order_with_line_items, line_items_count: 1) }
+          let(:line_item) { order.line_items.first }
+          let(:inventory_units) {
+            30.times.map do
+              build(
+                :inventory_unit,
+                order: order,
+                line_item: line_item,
+                variant: variant,
+              )
+            end
+          }
+
+          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 30
+          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(:order) { mock_model(Order) }
+      let(:stock_location) { build(:stock_location) }
+      let(:variant) { build(:variant) }
+
+      def inventory_units
+        @inventory_units ||= []
+      end
+
+      def build_inventory_unit
+        mock_model(InventoryUnit, variant: variant).tap do |unit|
+          inventory_units << unit
+        end
+      end
+
+      def pack
+        package = Package.new(order)
+        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(inventory_units, 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(inventory_units, 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(inventory_units, 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(inventory_units, 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(inventory_units, 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(inventory_units, 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,115 @@
+require 'spec_helper'
+
+shared_examples_for 'unlimited supply' do
+  it 'can_supply? any amount' do
+    expect(subject.can_supply?(1)).to eq true
+    expect(subject.can_supply?(101)).to eq true
+    expect(subject.can_supply?(100_001)).to eq true
+  end
+end
+
+module Spree
+  module Stock
+    describe Quantifier, :type => :model do
+
+      let(:target_stock_location) { nil }
+      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, target_stock_location) }
+
+      specify { expect(subject.stock_items).to eq([stock_item]) }
+
+      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 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 '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 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 'with a specific stock location' do
+        let!(:stock_location_2)     { create :stock_location }
+        let!(:stock_location_3)     { create :stock_location, active: false }
+        let(:target_stock_location) { stock_location_3 }
+
+        before do
+          Spree::StockItem.update_all(count_on_hand: 0, backorderable: false)
+          stock_location_3.stock_items.where(variant_id: stock_item.variant).update_all(count_on_hand: 5, backorderable: false)
+        end
+
+        it 'can_supply? only upto total_on_hand' do
+          expect(subject.can_supply?(5)).to eq true
+          expect(subject.can_supply?(6)).to eq 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,50 @@
+require 'spec_helper'
+
+module Spree
+  module Stock
+    module Splitter
+      describe ShippingCategory, :type => :model do
+
+        let(:order) { create(:order_with_line_items, line_items_count: 1) }
+        let(:line_item) { order.line_items.first }
+        let(:variant1) { build(:variant) }
+        let(:variant2) { build(:variant) }
+        let(:shipping_category_1) { create(:shipping_category, name: 'A') }
+        let(:shipping_category_2) { create(:shipping_category, name: 'B') }
+
+        def inventory_unit1
+          build(:inventory_unit, variant: variant1, order: order, line_item: line_item).tap do |inventory_unit|
+            inventory_unit.variant.product.shipping_category = shipping_category_1
+          end
+        end
+
+        def inventory_unit2
+          build(:inventory_unit, variant: variant2, order: order, line_item: line_item).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,29 @@
+require 'spec_helper'
+
+module Spree
+  module Stock
+    module Splitter
+      describe Weight, :type => :model do
+        let(:packer) { build(:stock_packer) }
+        let(:variant) { build(:base_variant, :weight => 100) }
+
+        subject { Weight.new(packer) }
+
+        it 'splits and keeps splitting until all packages are underweight' do
+          package = Package.new(packer.stock_location)
+          4.times { package.add build(:inventory_unit, variant: 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