# encoding: utf-8 require 'spec_helper' describe Spree::Variant, type: :model do let!(:variant) { create(:variant) } it_behaves_like 'default_price' context "validations" do it "should validate price is greater than 0" do variant.price = -1 expect(variant).to be_invalid end it "should validate price is 0" do variant.price = 0 expect(variant).to be_valid end it "should require a product" do expect(variant).to be_valid variant.product = nil expect(variant).to be_invalid end end context "after create" do let!(:product) { create(:product) } it "propagate to stock items" do expect_any_instance_of(Spree::StockLocation).to receive(:propagate_variant) product.variants.create! end context "stock location has disable propagate all variants" do before { Spree::StockLocation.update_all propagate_all_variants: false } it "propagate to stock items" do expect_any_instance_of(Spree::StockLocation).not_to receive(:propagate_variant) product.variants.create! end end describe 'mark_master_out_of_stock' do before do product.master.stock_items.first.set_count_on_hand(5) end context 'when product is created without variants but with stock' do it { expect(product.master).to be_in_stock } end context 'when a variant is created' do before(:each) do product.variants.create! end it { expect(product.master).to_not be_in_stock } end end context "when several countries have VAT" do let(:germany) { create(:country, iso: "DE") } let(:denmark) { create(:country, iso: "DK") } let(:france) { create(:country, iso: "FR") } let(:high_vat_zone) { create(:zone, countries: [germany, denmark]) } let(:low_vat_zone) { create(:zone, countries: [france]) } let(:tax_category) { create(:tax_category) } let!(:high_vat) { create(:tax_rate, included_in_price: true, amount: 0.25, zone: high_vat_zone, tax_categories: [tax_category]) } let!(:low_vat) { create(:tax_rate, included_in_price: true, amount: 0.15, zone: low_vat_zone, tax_categories: [tax_category]) } let(:product) { build(:product, tax_category: tax_category) } subject(:new_variant) { build(:variant, price: 15) } it "creates the appropriate prices for them" do product.variants << new_variant product.save! expect(new_variant.prices.find_by(country_iso: "FR").amount).to eq(17.25) expect(new_variant.prices.find_by(country_iso: "DE").amount).to eq(18.75) expect(new_variant.prices.find_by(country_iso: "DK").amount).to eq(18.75) expect(new_variant.prices.find_by(country_iso: nil).amount).to eq(15.00) # default price + FR, DE, DK expect(new_variant.prices.count).to eq(4) end context "when the products price changes" do context "and rebuild_vat_prices is set to true" do subject { variant.update(price: 99, rebuild_vat_prices: true, tax_category: tax_category) } it "creates new appropriate prices for this variant" do expect { subject }.to change { Spree::Price.count }.by(3) expect(variant.prices.find_by(country_iso: "FR").amount).to eq(113.85) expect(variant.prices.find_by(country_iso: "DE").amount).to eq(123.75) expect(variant.prices.find_by(country_iso: "DK").amount).to eq(123.75) expect(variant.prices.find_by(country_iso: nil).amount).to eq(99.00) end end context "and rebuild_vat_prices is not set" do subject { variant.update(price: 99, tax_category: tax_category) } it "does not create new prices" do expect { subject }.not_to change { Spree::Price.count } end end end end end context "product has other variants" do describe "option value accessors" do before { @multi_variant = FactoryGirl.create :variant, product: variant.product variant.product.reload } let(:multi_variant) { @multi_variant } it "should set option value" do expect(multi_variant.option_value('media_type')).to be_nil multi_variant.set_option_value('media_type', 'DVD') expect(multi_variant.option_value('media_type')).to eql 'DVD' multi_variant.set_option_value('media_type', 'CD') expect(multi_variant.option_value('media_type')).to eql 'CD' end it "should not duplicate associated option values when set multiple times" do multi_variant.set_option_value('media_type', 'CD') expect { multi_variant.set_option_value('media_type', 'DVD') }.to_not change(multi_variant.option_values, :count) expect { multi_variant.set_option_value('coolness_type', 'awesome') }.to change(multi_variant.option_values, :count).by(1) end context "and a variant is soft-deleted" do let!(:old_options_text) { variant.options_text } before { variant.destroy } it "still keeps the option values for that variant" do expect(variant.reload.options_text).to eq(old_options_text) end end context "and a variant is really deleted" do let!(:old_option_values_variant_ids) { variant.option_values_variants.pluck(:id) } before { variant.really_destroy! } it "leaves no stale records behind" do expect(old_option_values_variant_ids).to be_present expect(Spree::OptionValuesVariant.where(id: old_option_values_variant_ids)).to be_empty end end end end context "#cost_price=" do it "should use LocalizedNumber.parse" do expect(Spree::LocalizedNumber).to receive(:parse).with('1,599.99') subject.cost_price = '1,599.99' end end context "#price=" do it "should use LocalizedNumber.parse" do expect(Spree::LocalizedNumber).to receive(:parse).with('1,599.99') subject.price = '1,599.99' end end context "#weight=" do it "should use LocalizedNumber.parse" do expect(Spree::LocalizedNumber).to receive(:parse).with('1,599.99') subject.weight = '1,599.99' end end context "#display_amount" do it "returns a Spree::Money" do variant.price = 21.22 expect(variant.display_amount.to_s).to eql "$21.22" end end context "#cost_currency" do context "when cost currency is nil" do before { variant.cost_currency = nil } it "populates cost currency with the default value on save" do variant.save! expect(variant.cost_currency).to eql "USD" end end end context "#default_price" do context "when multiple prices are present in addition to a default price" do let(:pricing_options_germany) { Spree::Config.pricing_options_class.new(currency: "EUR") } let(:pricing_options_united_states) { Spree::Config.pricing_options_class.new(currency: "USD") } before do variant.prices.create(currency: "EUR", amount: 29.99) variant.reload end it "the default stays the same" do expect(variant.default_price.amount).to eq(19.99) end it "displays default price" do expect(variant.price_for(pricing_options_united_states).to_s).to eq("$19.99") expect(variant.price_for(pricing_options_germany).to_s).to eq("€29.99") end end context "when adding multiple prices" do it "it can reassign a default price" do expect(variant.default_price.amount).to eq(19.99) variant.prices.create(currency: "USD", amount: 12.12) expect(variant.reload.default_price.amount).to eq(12.12) end end end context "#price_selector" do subject { variant.price_selector } it "returns an instance of a price selector" do expect(variant.price_selector).to be_a(Spree::Config.variant_price_selector_class) end it "is instacached" do expect(variant.price_selector.object_id).to eq(variant.price_selector.object_id) end end context "#price_for(price_options)" do let(:price_options) { Spree::Config.variant_price_selector_class.pricing_options_class.new } it "calls the price selector with the given options object" do expect(variant.price_selector).to receive(:price_for).with(price_options) variant.price_for(price_options) end end context "#price_difference_from_master" do let(:pricing_options) { Spree::Config.default_pricing_options } subject { variant.price_difference_from_master(pricing_options) } it "can be called without pricing options" do expect(variant.price_difference_from_master).to eq(Spree::Money.new(0)) end context "for the master variant" do let(:variant) { create(:product).master } it { is_expected.to eq(Spree::Money.new(0, currency: Spree::Config.currency)) } end context "when both variants have a price" do let(:product) { create(:product, price: 25) } let(:variant) { create(:variant, product: product, price: 35) } it { is_expected.to eq(Spree::Money.new(10, currency: Spree::Config.currency)) } end context "when the master variant does not have a price" do let(:product) { create(:product, price: 25) } let(:variant) { create(:variant, product: product, price: 35) } before do allow(product.master).to receive(:price_for).and_return(nil) end it { is_expected.to be_nil } end context "when the variant does not have a price" do let(:product) { create(:product, price: 25) } let(:variant) { create(:variant, product: product, price: 35) } before do allow(variant).to receive(:price_for).and_return(nil) end it { is_expected.to be_nil } end end context "#price_same_as_master?" do context "when the price is the same as the master price" do let(:master) { create(:product, price: 10).master } let(:variant) { create(:variant, price: 10, product: master.product) } subject { variant.price_same_as_master? } it { is_expected.to be true } end context "when the price is different from the master price" do let(:master) { create(:product, price: 11).master } let(:variant) { create(:variant, price: 10, product: master.product) } subject { variant.price_same_as_master? } it { is_expected.to be false } end context "when the master variant does not have a price" do let(:master) { create(:product).master } let(:variant) { create(:variant, price: 10, product: master.product) } before do allow(master).to receive(:price_for).and_return(nil) end subject { variant.price_same_as_master? } it { is_expected.to be_falsey } end context "when the variant itself does not have a price" do let(:master) { create(:product).master } let(:variant) { create(:variant, price: 10, product: master.product) } before do allow(variant).to receive(:price_for).and_return(nil) end subject { variant.price_same_as_master? } it { is_expected.to be_falsey } end end describe '.price_in' do before do variant.prices << create(:price, variant: variant, currency: "EUR", amount: 33.33) end subject do Spree::Deprecation.silence { variant.price_in(currency) } end context "when currency is not specified" do let(:currency) { nil } it "returns nil" do expect(subject).to be_nil end end context "when currency is EUR" do let(:currency) { 'EUR' } it "returns the value in the EUR" do expect(subject.display_price.to_s).to eql "€33.33" end end context "when currency is USD" do let(:currency) { 'USD' } it "returns the value in the USD" do expect(subject.display_price.to_s).to eql "$19.99" end end end describe '.amount_in' do before do variant.prices << create(:price, variant: variant, currency: "EUR", amount: 33.33) end subject do Spree::Deprecation.silence { variant.amount_in(currency) } end context "when currency is not specified" do let(:currency) { nil } it "returns the amount in the default currency" do expect(subject).to be_nil end end context "when currency is EUR" do let(:currency) { 'EUR' } it "returns the value in the EUR" do expect(subject).to eql 33.33 end end context "when currency is USD" do let(:currency) { 'USD' } it "returns the value in the USD" do expect(subject).to eql 19.99 end end end # Regression test for https://github.com/spree/spree/issues/2432 describe 'options_text' do let!(:variant) { create(:variant, option_values: []) } let!(:master) { create(:master_variant) } before do # Order bar than foo variant.option_values << create(:option_value, { name: 'Foo', presentation: 'Foo', option_type: create(:option_type, position: 2, name: 'Foo Type', presentation: 'Foo Type') }) variant.option_values << create(:option_value, { name: 'Bar', presentation: 'Bar', option_type: create(:option_type, position: 1, name: 'Bar Type', presentation: 'Bar Type') }) end it 'should order by bar than foo' do expect(variant.options_text).to eql 'Bar Type: Bar, Foo Type: Foo' end end describe 'exchange_name' do let!(:variant) { create(:variant, option_values: []) } let!(:master) { create(:master_variant) } before do variant.option_values << create(:option_value, { name: 'Foo', presentation: 'Foo', option_type: create(:option_type, position: 2, name: 'Foo Type', presentation: 'Foo Type') }) end context 'master variant' do it 'should return name' do expect(master.exchange_name).to eql master.name end end context 'variant' do it 'should return options text' do expect(variant.exchange_name).to eql 'Foo Type: Foo' end end end describe 'exchange_name' do let!(:variant) { create(:variant, option_values: []) } let!(:master) { create(:master_variant) } before do variant.option_values << create(:option_value, { name: 'Foo', presentation: 'Foo', option_type: create(:option_type, position: 2, name: 'Foo Type', presentation: 'Foo Type') }) end context 'master variant' do it 'should return name' do expect(master.exchange_name).to eql master.name end end context 'variant' do it 'should return options text' do expect(variant.exchange_name).to eql 'Foo Type: Foo' end end end describe 'descriptive_name' do let!(:variant) { create(:variant, option_values: []) } let!(:master) { create(:master_variant) } before do variant.option_values << create(:option_value, { name: 'Foo', presentation: 'Foo', option_type: create(:option_type, position: 2, name: 'Foo Type', presentation: 'Foo Type') }) end context 'master variant' do it 'should return name with Master identifier' do expect(master.descriptive_name).to eql master.name + ' - Master' end end context 'variant' do it 'should return options text with name' do expect(variant.descriptive_name).to eql variant.name + ' - Foo Type: Foo' end end end # Regression test for https://github.com/spree/spree/issues/2744 describe "set_position" do it "sets variant position after creation" do variant = create(:variant) expect(variant.position).to_not be_nil end end describe '#in_stock?' do before do Spree::Config.track_inventory_levels = true end context 'when stock_items are not backorderable' do before do allow_any_instance_of(Spree::StockItem).to receive_messages(backorderable: false) end context 'when stock_items in stock' do before do variant.stock_items.first.update_column(:count_on_hand, 10) end it 'returns true if stock_items in stock' do expect(variant.in_stock?).to be true end end context 'when stock_items out of stock' do before do allow_any_instance_of(Spree::StockItem).to receive_messages(backorderable: false) allow_any_instance_of(Spree::StockItem).to receive_messages(count_on_hand: 0) end it 'return false if stock_items out of stock' do expect(variant.in_stock?).to be false end end end describe "#can_supply?" do it "calls out to quantifier" do expect(Spree::Stock::Quantifier).to receive(:new).and_return(quantifier = double) expect(quantifier).to receive(:can_supply?).with(10) variant.can_supply?(10) end end context 'when stock_items are backorderable' do before do allow_any_instance_of(Spree::StockItem).to receive_messages(backorderable: true) end context 'when stock_items out of stock' do before do allow_any_instance_of(Spree::StockItem).to receive_messages(count_on_hand: 0) end it 'in_stock? returns false' do expect(variant.in_stock?).to be false end it 'can_supply? return true' do expect(variant.can_supply?).to be true end end end describe "cache clearing on update" do it "correctly reports after updating track_inventory" do variant.stock_items.first.set_count_on_hand 0 expect(variant).not_to be_in_stock variant.update!(track_inventory: false) expect(variant).to be_in_stock end end end describe '#is_backorderable' do let(:variant) { build(:variant) } subject { variant.is_backorderable? } it 'should invoke Spree::Stock::Quantifier' do expect_any_instance_of(Spree::Stock::Quantifier).to receive(:backorderable?) { true } subject end end describe '#total_on_hand' do it 'should be infinite if track_inventory_levels is false' do Spree::Config[:track_inventory_levels] = false expect(build(:variant).total_on_hand).to eql(Float::INFINITY) end it 'should match quantifier total_on_hand' do variant = build(:variant) expect(variant.total_on_hand).to eq(Spree::Stock::Quantifier.new(variant).total_on_hand) end end describe '#tax_category' do context 'when tax_category is nil' do let(:product) { build(:product) } let(:variant) { build(:variant, product: product, tax_category_id: nil) } it 'returns the parent products tax_category' do expect(variant.tax_category).to eq(product.tax_category) end end context 'when tax_category is set' do let(:tax_category) { create(:tax_category) } let(:variant) { build(:variant, tax_category: tax_category) } it 'returns the tax_category set on itself' do expect(variant.tax_category).to eq(tax_category) end end end describe "touching" do it "updates a product" do variant.product.update_column(:updated_at, 1.day.ago) variant.touch expect(variant.product.reload.updated_at).to be_within(3.seconds).of(Time.current) end it "clears the in_stock cache key" do expect(Rails.cache).to receive(:delete).with(variant.send(:in_stock_cache_key)) variant.touch end end describe "#should_track_inventory?" do it 'should not track inventory when global setting is off' do Spree::Config[:track_inventory_levels] = false expect(build(:variant).should_track_inventory?).to eq(false) end it 'should not track inventory when variant is turned off' do Spree::Config[:track_inventory_levels] = true expect(build(:on_demand_variant).should_track_inventory?).to eq(false) end it 'should track inventory when global and variant are on' do Spree::Config[:track_inventory_levels] = true expect(build(:variant).should_track_inventory?).to eq(true) end end describe "deleted_at scope" do let!(:previous_variant_price) { variant.display_price } before { variant.destroy } it "should keep its price if deleted" do expect(variant.display_price).to eq(previous_variant_price) end context 'when loading with pre-fetching of default_price' do it 'also keeps the previous price' do reloaded_variant = Spree::Variant.with_deleted.includes(:default_price).find_by(id: variant.id) expect(reloaded_variant.display_price).to eq(previous_variant_price) end end end describe "stock movements" do let!(:movement) { create(:stock_movement, stock_item: variant.stock_items.first) } it "builds out collection just fine through stock items" do expect(variant.stock_movements.to_a).not_to be_empty end end describe "in_stock scope" do subject { Spree::Variant.in_stock } let!(:in_stock_variant) { create(:variant) } let!(:out_of_stock_variant) { create(:variant) } let!(:stock_location) { create(:stock_location) } context "a stock location is provided" do subject { Spree::Variant.in_stock([stock_location]) } context "there's stock in the location" do before do in_stock_variant. stock_items.find_by(stock_location: stock_location). update_column(:count_on_hand, 10) out_of_stock_variant. stock_items.where.not(stock_location: stock_location).first. update_column(:count_on_hand, 10) end it "returns all in stock variants in the provided stock location" do expect(subject).to eq [in_stock_variant] end end context "there's no stock in the location" do it "returns an empty list" do expect(subject).to eq [] end end end context "a stock location is not provided" do before do in_stock_variant.stock_items.first.update_column(:count_on_hand, 10) end it "returns all in stock variants" do expect(subject).to eq [in_stock_variant] end end context "inventory levels globally not tracked" do before { Spree::Config.track_inventory_levels = false } after { Spree::Config.track_inventory_levels = true } it 'includes items without inventory' do expect( subject ).to include out_of_stock_variant end end end describe "#display_image" do subject { variant.display_image } context "variant has associated images" do let(:attachment) { File.open(File.expand_path('../../../fixtures/thinking-cat.jpg', __FILE__)) } let(:image_params) { { viewable_id: variant.id, viewable_type: 'Spree::Variant', attachment: attachment, alt: "position 1", position: 1 } } let!(:first_image) { Spree::Image.create(image_params) } let!(:second_image) { image_params.merge(alt: "position 2", position: 2) } it "returns the first image" do expect(subject).to eq first_image end end context "variant does not have any associated images" do it "returns an image" do expect(subject).to be_a(Spree::Image) end it "returns unpersisted record" do expect(subject).to be_new_record end end end describe "#variant_properties" do let(:option_value_1) { create(:option_value) } let(:option_value_2) { create(:option_value) } let(:variant) { create(:variant, option_values: [option_value_1, option_value_2]) } subject { variant.variant_properties } context "variant has properties" do let!(:rule_1) { create(:variant_property_rule, product: variant.product, option_value: option_value_1) } let!(:rule_2) { create(:variant_property_rule, product: variant.product, option_value: option_value_2) } it "returns the variant property rule's values" do expect(subject).to match_array rule_1.values + rule_2.values end end context "variant doesn't have any properties" do it "returns an empty list" do expect(subject).to eq [] end end end end