solidus_volume_pricing 1.2.0 → 2.0.0

data/spec/models/spree/line_item_spec.rb

@@ -1,36 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe Spree::LineItem, type: :model do
-  let(:order) { create(:order) }
-  let(:variant) { create(:variant, price: 10) }
-  let(:line_item) { order.line_items.first }
-  let(:role) { create(:role) }
-
-  before do
-    variant.volume_prices.create! amount: 9, discount_type: 'price', range: '(2+)'
-    order.contents.add(variant, 1)
-  end
-
-  it 'updates the line item price when the quantity changes to match a range and has no role' do
-    expect(line_item.price.to_f).to be(10.00)
-    order.contents.add(variant, 1)
-    expect(order.line_items.first.price.to_f).to be(9.00)
-  end
-
-  it 'updates the line item price when the quantity changes to match a range and role matches' do
-    order.user.spree_roles << role
-    expect(order.user.has_spree_role?(role.name.to_sym)).to be(true)
-    variant.volume_prices.first.update(role_id: role.id)
-    expect(line_item.price.to_f).to be(10.00)
-    order.contents.add(variant, 1)
-    expect(order.line_items.first.price.to_f).to be(9.00)
-  end
-
-  it 'does not update the line item price when the variant role and order role don`t match' do
-    expect(order.user.has_spree_role?(role.name.to_sym)).to be(false)
-    variant.volume_prices.first.update(role_id: role.id)
-    expect(line_item.price.to_f).to be(10.00)
-    order.contents.add(variant, 1)
-    expect(order.line_items.first.price.to_f).to be(10.00)
-  end
-end

data/spec/models/spree/order_spec.rb

@@ -1,51 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe Spree::Order, type: :model do
-  before do
-    @order = create(:order)
-    @variant = create(:variant, price: 10)
-
-    @variant_with_prices = create(:variant, price: 10)
-    @variant_with_prices.volume_prices << create(:volume_price, range: '1..5', amount: 9, position: 2)
-    @variant_with_prices.volume_prices << create(:volume_price, range: '(5..9)', amount: 8, position: 1)
-  end
-
-  context 'add_variant' do
-    it 'uses the variant price if there are no volume prices' do
-      @order.contents.add(@variant)
-      expect(@order.line_items.first.price).to eq(10)
-    end
-
-    it 'uses the volume price if quantity falls within a quantity range of a volume price' do
-      @variant.volume_prices << create(:volume_price, range: '(5..10)', amount: 9)
-      @order.contents.add(@variant_with_prices, 7)
-      expect(@order.line_items.first.price).to eq(8)
-    end
-
-    it 'uses the variant price if the quantity fails to satisfy any of the volume price ranges' do
-      @order.contents.add(@variant, 10)
-      expect(@order.line_items.first.price).to eq(10)
-    end
-
-    it 'uses the first matching volume price in the event of more then one matching volume prices' do
-      @order.contents.add(@variant_with_prices, 5)
-      expect(@order.line_items.first.price).to eq(8)
-    end
-
-    it 'uses the master variant volume price in case variant has no volume price if config is true' do
-      stub_spree_preferences(use_master_variant_volume_pricing: true)
-      @master = @variant.product.master
-      @master.volume_prices << create(:volume_price, range: '(1..5)', amount: 9, position: 2)
-      @order.contents.add(@variant, 5)
-      expect(@order.line_items.first.price).to eq(9)
-    end
-
-    it 'doesnt use the master variant volume price in case variant has no volume price if config is false' do
-      stub_spree_preferences(use_master_variant_volume_pricing: false)
-      @master = @variant.product.master
-      @master.volume_prices << create(:volume_price, range: '(1..5)', amount: 9, position: 2)
-      @order.contents.add(@variant, 5)
-      expect(@order.line_items.first.price).to eq(10)
-    end
-  end
-end

data/spec/models/spree/variant_spec.rb

@@ -1,6 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe Spree::Variant, type: :model do
-  it { is_expected.to have_many(:volume_prices) }
-  it { is_expected.to have_many(:model_volume_prices) }
-end

data/spec/models/spree/volume_price_spec.rb

@@ -1,216 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe Spree::VolumePrice, type: :model do
-  let(:volume_price) do
-    described_class.new(variant: Spree::Variant.new, amount: 10, discount_type: 'price')
-  end
-
-  it { is_expected.to belong_to(:variant).touch(true).optional }
-  it { is_expected.to belong_to(:volume_price_model).touch(true).optional }
-  it { is_expected.to belong_to(:spree_role).class_name('Spree::Role').with_foreign_key('role_id').optional }
-  it { is_expected.to validate_presence_of(:discount_type) }
-  it { is_expected.to validate_presence_of(:amount) }
-
-  it { is_expected.to validate_inclusion_of(:discount_type).in_array(%w[price dollar percent]) }
-
-  describe '.for_variant' do
-    subject { described_class.for_variant(variant, user: user) }
-
-    let(:user) { nil }
-    let(:variant) { create(:variant) }
-    let!(:volume_prices) { create_list(:volume_price, 2, variant: variant) }
-
-    context 'if no user is given' do
-      it 'returns all volume prices for given variant that are not related to a specific role' do
-        expect(subject).to eq(volume_prices)
-      end
-    end
-
-    context 'if user is given' do
-      let(:role) do
-        create(:role, name: 'merchant')
-      end
-
-      let(:user) do
-        create(:user)
-      end
-
-      let!(:volume_prices_for_user_role) do
-        create_list(:volume_price, 2, variant: variant, role_id: role.id)
-      end
-
-      context 'whose role matches' do
-        before do
-          user.spree_roles = [role]
-        end
-
-        it 'returns role specific volume prices' do
-          expect(subject).to include(*volume_prices_for_user_role)
-        end
-
-        it 'returns non-role specific volume prices' do
-          expect(subject).to include(*volume_prices)
-        end
-      end
-
-      context 'whose role does not match' do
-        before do
-          user.spree_roles = []
-        end
-
-        it 'does not include role specific volume prices' do
-          expect(subject).not_to include(*volume_prices_for_user_role)
-        end
-
-        it 'returns non-role specific volume prices' do
-          expect(subject).to include(*volume_prices)
-        end
-      end
-    end
-
-    context 'if volume prices are not related to the variant but to a volume price model' do
-      let(:volume_price_model) do
-        create(:volume_price_model)
-      end
-
-      let!(:volume_prices_from_model) do
-        create_list(:volume_price, 2, volume_price_model: volume_price_model, variant: nil)
-      end
-
-      context 'and these volume prices are also related to the given variant' do
-        let(:variant) do
-          create(:variant, volume_price_models: [volume_price_model])
-        end
-
-        it 'includes these volume prices' do
-          expect(subject).to include(*volume_prices_from_model)
-        end
-      end
-
-      context 'and these volume prices are not related to the given variant' do
-        it 'does not include these volume prices' do
-          expect(subject).not_to include(*volume_prices_from_model)
-        end
-      end
-    end
-  end
-
-  describe 'valid range format' do
-    it 'requires the presence of a variant' do
-      volume_price.variant = nil
-      expect(volume_price).not_to be_valid
-    end
-
-    it 'consider a range of (1..2) to be valid' do
-      volume_price.range = '(1..2)'
-      expect(volume_price).to be_valid
-    end
-
-    it 'consider a range of (1...2) to be valid' do
-      volume_price.range = '(1...2)'
-      expect(volume_price).to be_valid
-    end
-
-    it 'consider a range of 1..2 to be valid' do
-      volume_price.range = '1..2'
-      expect(volume_price).to be_valid
-    end
-
-    it 'consider a range of 1...2 to be valid' do
-      volume_price.range = '1...2'
-      expect(volume_price).to be_valid
-    end
-
-    it 'consider a range of (10+) to be valid' do
-      volume_price.range = '(10+)'
-      expect(volume_price).to be_valid
-    end
-
-    it 'consider a range of 10+ to be valid' do
-      volume_price.range = '10+'
-      expect(volume_price).to be_valid
-    end
-
-    it 'does not consider a range of 1-2 to valid' do
-      volume_price.range = '1-2'
-      expect(volume_price).not_to be_valid
-    end
-
-    it 'does not consider a range of 1 to valid' do
-      volume_price.range = '1'
-      expect(volume_price).not_to be_valid
-    end
-
-    it 'does not consider a range of foo to valid' do
-      volume_price.range = 'foo'
-      expect(volume_price).not_to be_valid
-    end
-  end
-
-  describe 'display_range' do
-    subject(:display_range) { volume_price.display_range }
-
-    let(:volume_price) { described_class.new(range: range) }
-
-    context 'with parens' do
-      let(:range) { '(48+)' }
-
-      it { is_expected.to eq('48+') }
-    end
-
-    context 'with range dots' do
-      let(:range) { '1..3' }
-
-      it { is_expected.to eq('1-3') }
-    end
-
-    context 'with range dots and parens' do
-      let(:range) { '(1..3)' }
-
-      it { is_expected.to eq('1-3') }
-    end
-  end
-
-  describe 'include?' do
-    ['10..20', '(10..20)'].each do |range|
-      it "does not match a quantity that fails to fall within the specified range of #{range}" do
-        volume_price.range = range
-        expect(volume_price).not_to include(21)
-      end
-
-      it "matches a quantity that is within the specified range of #{range}" do
-        volume_price.range = range
-        expect(volume_price).to include(12)
-      end
-
-      it 'matches the upper bound of ranges that include the upper bound' do
-        volume_price.range = range
-        expect(volume_price).to include(20)
-      end
-    end
-
-    ['10...20', '(10...20)'].each do |range|
-      it 'does not match the upper bound for ranges that exclude the upper bound' do
-        volume_price.range = range
-        expect(volume_price).not_to include(20)
-      end
-    end
-
-    ['50+', '(50+)'].each do |range|
-      it "matches a quantity that exceeds the value of an open ended range of #{range}" do
-        volume_price.range = range
-        expect(volume_price).to include(51)
-      end
-
-      it "matches a quantity that equals the value of an open ended range of #{range}" do
-        volume_price.range = range
-        expect(volume_price).to include(50)
-      end
-
-      it "does not match a quantity that is less then the value of an open ended range of #{range}" do
-        volume_price.range = range
-        expect(volume_price).not_to include(40)
-      end
-    end
-  end
-end

data/spec/spec_helper.rb

@@ -1,32 +0,0 @@
-# frozen_string_literal: true
-
-# Configure Rails Environment
-ENV['RAILS_ENV'] = 'test'
-
-# Run Coverage report
-require 'solidus_dev_support/rspec/coverage'
-
-# Create the dummy app if it's still missing.
-dummy_env = "#{__dir__}/dummy/config/environment.rb"
-system 'bin/rake extension:test_app' unless File.exist? dummy_env
-require dummy_env
-
-# Requires factories and other useful helpers defined in spree_core.
-require 'solidus_dev_support/rspec/feature_helper'
-
-# Requires supporting ruby files with custom matchers and macros, etc,
-# in spec/support/ and its subdirectories.
-Dir["#{__dir__}/support/**/*.rb"].sort.each { |f| require f }
-
-# Requires factories defined in Solidus core and this extension.
-# See: lib/solidus_volume_pricing/testing_support/factories.rb
-SolidusDevSupport::TestingSupport::Factories.load_for(SolidusVolumePricing::Engine)
-
-RSpec.configure do |config|
-  config.infer_spec_type_from_file_location!
-  config.use_transactional_fixtures = false
-
-  if Spree.solidus_gem_version < Gem::Version.new('2.11')
-    config.extend Spree::TestingSupport::AuthorizationHelpers::Request, type: :system
-  end
-end

data/spec/support/shoulda.rb

@@ -1,11 +0,0 @@
-# frozen_string_literal: true
-
-require 'shoulda-matchers'
-
-# From: https://github.com/thoughtbot/shoulda-matchers/issues/384
-Shoulda::Matchers.configure do |config|
-  config.integrate do |with|
-    with.test_framework :rspec
-    with.library :rails
-  end
-end