solidus_core 2.8.0 → 2.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6cfc7af157e2912ba67be54f77a8bbcc9c053609b8aac47a95cf94f8411023a5
-  data.tar.gz: 2c05f398e332a2a57e031b3009b0ca06508b4209760b145423716300bedad757
+  metadata.gz: f6a0e506dd5c20cd114029a4d8f7cf6c9dff0ef0ce08c3b92cdd24e39b58f97e
+  data.tar.gz: 1936cc0f6fda95d9a4931b584eeec1c3529d64dec7aee80c92e0b58183d0c19b
 SHA512:
-  metadata.gz: dc600844d181e8d722f84c456e673170287ad5c1eb6a983808fc29af90459b0cb5835e306c16d7ba8ef2c4123ac6bbd7d1485179656b905696f05c1076ee0835
-  data.tar.gz: 79493de4d9e86c27260795466c63f314d82fce1b92da34ab56b6b48c137612ec7f24ad408a45d0790f09274e1cc1cea7cb8ff0abb987f33f609ddbb95bbc1354
+  metadata.gz: fd93440711741bf2970e57db6cc86325f0ae2faa28642651a959f40004bf9b384cd39343d23836ef6bdb5565c04aa818822b1443284c3feee93ebdc51b4b4b84
+  data.tar.gz: 46461444d4462294444911f9ec1a7da35e75e82a3713e62879ef7e08cc4f58cc02b01eefd2678cee624cd3158ad165a28c0ecefb9c456cf1df5167af1b812550

data/app/models/spree/stock/inventory_units_finalizer.rb

@@ -38,7 +38,7 @@ module Spree
           inventory_units_for_shipment.group_by(&:line_item_id).each_value do |units|
             shipment = units.first.shipment
             line_item = units.first.line_item
-            shipment.stock_location.unstock line_item.variant, inventory_units.count, shipment
+            shipment.stock_location.unstock line_item.variant, units.count, shipment
           end
         end
       end

data/lib/spree/core/version.rb

@@ -2,7 +2,7 @@
 
 module Spree
   def self.solidus_version
-    "2.8.0"
+    "2.8.1"
   end
 
   def self.solidus_gem_version

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

@@ -5,29 +5,53 @@ require 'rails_helper'
 module Spree
   module Stock
     RSpec.describe InventoryUnitsFinalizer, type: :model do
-      let(:order)          { build(:order_with_line_items) }
-      let(:inventory_unit) { build(:inventory_unit, order: order, variant: order.line_items.first.variant, shipment: order.shipments.first) }
-      let(:stock_item) { inventory_unit.variant.stock_items.first }
-
-      before do
-        stock_item.set_count_on_hand(10)
-        stock_item.update_attributes!(backorderable: false)
-        inventory_unit.update_attributes!(pending: true)
-      end
+      context "when finalizing an order with one line_item" do
+        let(:order)          { build(:order_with_line_items) }
+        let(:inventory_unit) { build(:inventory_unit, order: order, variant: order.line_items.first.variant, shipment: order.shipments.first) }
+        let(:stock_item) { inventory_unit.variant.stock_items.first }
 
-      subject { described_class.new([inventory_unit]).run! }
+        before do
+          stock_item.set_count_on_hand(10)
+          stock_item.update_attributes!(backorderable: false)
+          inventory_unit.update_attributes!(pending: true)
+        end
 
-      it "updates the associated inventory units" do
-        inventory_unit.update_columns(updated_at: 1.hour.ago)
-        expect { subject }.to change { inventory_unit.reload.updated_at }
-      end
+        subject { described_class.new([inventory_unit]).run! }
+
+        it "updates the associated inventory units" do
+          inventory_unit.update_columns(updated_at: 1.hour.ago)
+          expect { subject }.to change { inventory_unit.reload.updated_at }
+        end
+
+        it "updates the inventory units to not be pending" do
+          expect { subject }.to change { inventory_unit.reload.pending }.to(false)
+        end
 
-      it "updates the inventory units to not be pending" do
-        expect { subject }.to change { inventory_unit.reload.pending }.to(false)
+        it "unstocks the variant" do
+          expect { subject }.to change { stock_item.reload.count_on_hand }.from(10).to(9)
+        end
       end
 
-      it "unstocks the variant" do
-        expect { subject }.to change { stock_item.reload.count_on_hand }.from(10).to(9)
+      context "when finalizing an order with multiple line_items" do
+        let(:order)          { build(:order_with_line_items, line_items_count: 2) }
+        let(:inventory_unit) { build(:inventory_unit, order: order, variant: order.line_items.first.variant, shipment: order.shipments.first) }
+        let(:inventory_unit_2) { build(:inventory_unit, order: order, variant: order.line_items.second.variant, shipment: order.shipments.first) }
+        let(:stock_item) { inventory_unit.variant.stock_items.first }
+        let(:stock_item_2) { inventory_unit.variant.stock_items.first }
+
+        before do
+          stock_item.set_count_on_hand(10)
+          stock_item_2.set_count_on_hand(10)
+          inventory_unit.update_attributes!(pending: true)
+          inventory_unit_2.update_attributes!(pending: true)
+        end
+
+        subject { described_class.new([inventory_unit, inventory_unit_2]).run! }
+
+        it "unstocks the variant with the correct quantity" do
+          expect { subject }.to change { stock_item.reload.count_on_hand }.from(10).to(9)
+          .and change { stock_item_2.reload.count_on_hand }.from(10).to(9)
+        end
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: solidus_core
 version: !ruby/object:Gem::Version
-  version: 2.8.0
+  version: 2.8.1
 platform: ruby
 authors:
 - Solidus Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-01-29 00:00:00.000000000 Z
+date: 2019-02-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionmailer