solidus_reserved_stock 0.0.1

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

@@ -0,0 +1,42 @@
+require "spec_helper"
+
+# TODO: Need to supply user argument in the appropriate places
+#       - LineItem#sufficient_stock?
+#       - Spree::Stock::AvailabilityValidator
+#       - Variant#can_supply?
+#       - _cart_form (give it the current user)
+describe Spree::Stock::Quantifier, type: :model do
+  let(:user) { create(:user) }
+  let(:other_user) { create(:user) }
+  let(:original_stock_location) { create(:stock_location) }
+  let(:variant) { create(:variant) }
+
+  context "#total_on_hand" do
+    before(:each) do
+      stock_item = original_stock_location.stock_item_or_create(variant)
+      stock_item.adjust_count_on_hand(10)
+      Spree::Stock::Reserver.new.reserve(
+        variant,
+        original_stock_location,
+        user,
+        4
+      )
+    end
+    context "with a user argument" do
+      it "includes reserved stock for user" do
+        subject = Spree::Stock::Quantifier.new(variant, nil, user)
+        expect(subject.total_on_hand).to eq(10)
+      end
+      it "excludes reserved stock for other users" do
+        subject = Spree::Stock::Quantifier.new(variant, nil, other_user)
+        expect(subject.total_on_hand).to eq(6)
+      end
+    end
+    context "with no user argument" do
+      it "excludes reserved stock" do
+        subject = Spree::Stock::Quantifier.new(variant)
+        expect(subject.total_on_hand).to eq(6)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,103 @@
+require "spec_helper"
+
+# Specs ReservedStockItem subclass to StockItem
+describe Spree::Stock::Reserver, type: :model do
+  let(:user) { create(:user) }
+  let(:original_stock_location) { create(:stock_location) }
+  let(:variant) { create(:variant) }
+  let(:reserved_stock_location) { Spree::StockLocation.reserved_items_location }
+  let(:reserved_stock_item) do
+    create(
+      :reserved_stock_item,
+      variant: variant,
+      stock_location: reserved_stock_location,
+      original_stock_location: original_stock_location,
+      user: user,
+      expires_at: 1.day.from_now,
+      backorderable: false
+    )
+  end
+  let(:expired_reserved_stock_item) do
+    create(
+      :reserved_stock_item,
+      variant: variant,
+      stock_location: reserved_stock_location,
+      original_stock_location: original_stock_location,
+      user: user,
+      expires_at: 1.day.ago,
+      backorderable: false
+    )
+  end
+
+  subject { Spree::Stock::Reserver.new }
+
+  context "#reserve" do
+    before do
+      stock_item = original_stock_location.stock_item_or_create(variant)
+      stock_item.adjust_count_on_hand(10)
+    end
+    it "moves stock items to the reserved stock location" do
+      subject.reserve(variant, original_stock_location, user, 4)
+      expect(user.reserved_count_on_hand(variant)).to eq 4
+      expect(original_stock_location.count_on_hand(variant)).to eq 6
+    end
+    it "remembers original stock location" do
+      subject.reserve(variant, original_stock_location, user, 4)
+      reserved_stock_item = user.reserved_stock_item(variant)
+      expect(
+        reserved_stock_item.original_stock_location
+      ).to eq original_stock_location
+    end
+    it "won't reserve more stock than exists" do
+      expect do
+        subject.reserve(variant, original_stock_location, user, 1000)
+      end.to raise_error Spree::Stock::Reserver::InvalidQuantityError
+    end
+    it "won't reserve zero stock" do
+      expect do
+        subject.reserve(variant, original_stock_location, user, 0)
+      end.to raise_error Spree::Stock::Reserver::InvalidQuantityError
+    end
+  end
+
+  context "#restock" do
+    before(:each) do
+      reserved_stock_item
+    end
+    it "can restore all stock to the original stock location" do
+      subject.restock(variant, user)
+      expect(original_stock_location.count_on_hand(variant)).to eq 10
+      expect(user.reserved_count_on_hand(variant)).to eq 0
+    end
+    it "can restore a quantity of stock to the original stock location" do
+      subject.restock(variant, user, 4)
+      expect(original_stock_location.count_on_hand(variant)).to eq 4
+      expect(user.reserved_count_on_hand(variant)).to eq 6
+    end
+    it "won't restock more stock than exists"do
+      expect do
+        subject.restock(variant, user, 1000)
+      end.to raise_error Spree::Stock::Reserver::InvalidQuantityError
+    end
+    it "won't restock zero stock" do
+      expect do
+        subject.restock(variant, user, 0)
+      end.to raise_error Spree::Stock::Reserver::InvalidQuantityError
+    end
+  end
+
+  context "#restock_expired" do
+    it "restores expired ReservedStockItems" do
+      expired_reserved_stock_item
+      subject.restock_expired
+      expect(original_stock_location.count_on_hand(variant)).to eq 10
+      expect(user.reserved_count_on_hand(variant)).to eq 0
+    end
+    it "doesn't restore unexpired ReservedStockItems" do
+      reserved_stock_item
+      subject.restock_expired
+      expect(original_stock_location.count_on_hand(variant)).to eq 0
+      expect(user.reserved_count_on_hand(variant)).to eq 10
+    end
+  end
+end

data/spec/models/spree/stock_location_decorator_spec.rb

@@ -0,0 +1,47 @@
+require "spec_helper"
+
+# Specs modifications in stock_location_decorator
+describe Spree::StockLocation, type: :model do
+  subject do
+    create(
+      :stock_location,
+      backorderable_default: false,
+      propagate_all_variants: false,
+      reserved_items: true
+    )
+  end
+
+  context "validation" do
+    context "when reserved_items is true" do
+      it "is invalid if backorderable_default is true" do
+        reserved_stock_item = build(
+          :stock_location,
+          backorderable_default: true,
+          propagate_all_variants: false,
+          reserved_items: true
+        )
+        expect(reserved_stock_item).to be_invalid
+      end
+      it "is invalid if propagate_all_variants is true" do
+        reserved_stock_item = build(
+          :stock_location,
+          backorderable_default: false,
+          propagate_all_variants: true,
+          reserved_items: true
+        )
+        expect(reserved_stock_item).to be_invalid
+      end
+    end
+  end
+  context ".reserved_items_location" do
+    it "can return the existing reserved stock location" do
+      subject
+      expect(Spree::StockLocation.reserved_items_location).to eq subject
+    end
+    it "creates reserved stock location if not present" do
+      reserved_items_location = Spree::StockLocation.reserved_items_location
+      expect(reserved_items_location).to be_a Spree::StockLocation
+      expect(reserved_items_location.reserved_items?).to be true
+    end
+  end
+end

data/spec/models/spree/user_decorator_spec.rb

@@ -0,0 +1,65 @@
+require "spec_helper"
+
+
+describe Spree.user_class, type: :model do
+  subject { create(:user) }
+  let(:original_stock_location) { create(:stock_location) }
+  let(:variant) { create(:variant) }
+  let(:reserved_stock_location) { Spree::StockLocation.reserved_items_location }
+  let(:reserved_stock_item) do
+    create(
+      :reserved_stock_item,
+      variant: variant,
+      stock_location: reserved_stock_location,
+      original_stock_location: original_stock_location,
+      user: subject,
+      expires_at: 1.day.from_now,
+      backorderable: false
+    )
+  end
+
+  context "#reserved_count_on_hand" do
+    it "returns reserved quantity for a variant" do
+      reserved_stock_item
+      expect(
+        subject.reserved_count_on_hand(variant)
+      ).to eq 10
+    end
+  end
+
+  context "#reserved_stock_item" do
+    it "returns the reserved_stock_item for a variant" do
+      reserved_stock_item
+      expect(
+        subject.reserved_stock_item(variant)
+      ).to eq reserved_stock_item
+    end
+  end
+
+  context "#reserved_stock_item_or_create" do
+    it "returns an existing reserved_stock_item" do
+      reserved_stock_item
+      expect(
+        subject.reserved_stock_item_or_create(variant, original_stock_location)
+      ).to eq reserved_stock_item
+    end
+    it "creates and returns a reserved_stock_item if none exists" do
+      expect(subject.reserved_stock_item(variant)).to be_nil
+      subject.reserved_stock_item_or_create(variant, original_stock_location, 1.week.from_now)
+      expect(subject.reserved_stock_item(variant)).not_to be_nil
+    end
+  end
+
+  context "when account is destroyed" do
+    it "restocks all reserved items" do
+      reserved_stock_item
+      subject.destroy
+      expect(
+        reserved_stock_location.stock_items.where(user_id: subject.id).count
+      ).to eq 0
+      expect(
+        original_stock_location.count_on_hand(variant)
+      ).to eq 10
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,56 @@
+require "simplecov"
+SimpleCov.start "rails" do
+  add_filter "/.gems/"
+end
+
+ENV["RAILS_ENV"] ||= "test"
+
+begin
+  require File.expand_path("../dummy/config/environment", __FILE__)
+rescue LoadError
+  puts "Could not load dummy application."\
+    "Please ensure you have run `bundle exec rake test_app`"
+  exit
+end
+
+require "rspec/rails"
+
+# Requires supporting ruby files with custom matchers and macros, etc,
+# in spec/support/ and its subdirectories.
+Dir[File.dirname(__FILE__) + "/support/**/*.rb"].each {|f| require f}
+require "pry"
+require "ffaker"
+require "factory_girl_rails"
+require "spree/testing_support/factories"
+require "spree/testing_support/preferences"
+require "spree/api/testing_support/helpers"
+require "spree/api/testing_support/setup"
+require 'rspec/active_model/mocks'
+
+RSpec.configure do |config|
+  config.fail_fast = false
+  config.filter_run focus: true
+  config.infer_spec_type_from_file_location!
+  config.mock_with :rspec
+  config.raise_errors_for_deprecations!
+  config.run_all_when_everything_filtered = true
+  config.use_transactional_fixtures = true
+  config.include FactoryGirl::Syntax::Methods
+  config.include Spree::Api::TestingSupport::Helpers, type: :controller
+  config.extend Spree::Api::TestingSupport::Setup, type: :controller
+  config.include Spree::TestingSupport::Preferences
+
+  config.expect_with :rspec do |expectations|
+    expectations.syntax = :expect
+  end
+
+  config.before(:each) do
+    Rails.cache.clear
+    reset_spree_preferences
+    Spree::Api::Config.requires_authentication = true
+  end
+end
+
+Dir[File.join(File.dirname(__FILE__), "/support/**/*.rb")].each do |file|
+  require file
+end

data/spec/support/api_helpers.rb

@@ -0,0 +1,35 @@
+require "active_support/all"
+
+# Convenience methods for API specs
+module ApiHelpers
+  extend ActiveSupport::Concern
+
+  included do
+    routes { Spree::Core::Engine.routes }
+  end
+
+  def api_get(action, params = {}, session = nil, flash = nil)
+    api_process(action, params, session, flash, "GET")
+  end
+
+  def api_post(action, params = {}, session = nil, flash = nil)
+    api_process(action, params, session, flash, "POST")
+  end
+
+  def api_put(action, params = {}, session = nil, flash = nil)
+    api_process(action, params, session, flash, "PUT")
+  end
+
+  def api_delete(action, params = {}, session = nil, flash = nil)
+    api_process(action, params, session, flash, "DELETE")
+  end
+
+  def api_process(action, params = {}, session = nil, flash = nil, method = "get")
+    scoping = respond_to?(:resource_scoping) ? resource_scoping : {}
+    process(action, method, params.merge(scoping).reverse_merge!(format: :json), session, flash)
+  end
+end
+
+RSpec.configure do |config|
+  config.include ApiHelpers, type: :controller
+end

data/spec/support/database_cleaner.rb

@@ -0,0 +1,14 @@
+RSpec.configure do |config|
+  config.before(:suite) do
+    DatabaseCleaner.strategy = :transaction
+    DatabaseCleaner.clean_with(:truncation)
+  end
+
+  config.before(:each) do
+    DatabaseCleaner.start
+  end
+
+  config.after(:each) do
+    DatabaseCleaner.clean
+  end
+end

data/spec/support/have_attributes_matcher.rb

@@ -0,0 +1,10 @@
+# TODO: Remove this file borrowed from Solidus. It doesn't seem like a great
+#       idea to override have_attributes
+RSpec::Matchers.define :have_attributes do |expected_attributes|
+  match do |actual|
+    # actual is a Hash object representing an object, like this:
+    # { "name" => "Product #1" }
+    actual_attributes = actual.keys.map(&:to_sym)
+    expected_attributes.map(&:to_sym).all? { |attr| actual_attributes.include?(attr) }
+  end
+end