spree_core 3.3.0.rc1 → 3.3.0.rc2

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

@@ -1,70 +0,0 @@
-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

@@ -1,125 +0,0 @@
-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

@@ -1,126 +0,0 @@
-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

@@ -1,29 +0,0 @@
-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

@@ -1,21 +0,0 @@
-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

@@ -1,47 +0,0 @@
-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

@@ -1,32 +0,0 @@
-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