solidus_core 2.9.6 → 2.11.0

data/spec/models/spree/calculator_spec.rb

@@ -1,43 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::Calculator, type: :model do
-  class SimpleCalculator < Spree::Calculator
-    def compute_simple_computable(_line_item)
-      'computed'
-    end
-  end
-
-  class SimpleComputable
-  end
-
-  describe "#calculators" do
-    subject { Spree::Calculator.calculators }
-
-    it 'returns the (deprecated) calculator step' do
-      Spree::Deprecation.silence do
-        expect(subject).to be_a Spree::Core::Environment::Calculators
-      end
-    end
-  end
-
-  context "with computable" do
-    let(:calculator) { SimpleCalculator.new }
-    let(:computable) { SimpleComputable.new }
-
-    subject { SimpleCalculator.new.compute computable }
-
-    it 'calls compute method of class type' do
-      expect(subject).to eq  'computed'
-    end
-
-    context 'computable does not implement right function name' do
-      let(:computable) { Spree::LineItem.new }
-
-      it 'raises an error' do
-        expect { subject }.to raise_error NotImplementedError, /Please implement \'compute_line_item\(line_item\)\' in your calculator/
-      end
-    end
-  end
-end

data/spec/models/spree/carton_spec.rb

@@ -1,137 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::Carton do
-  let(:carton) { create(:carton) }
-
-  describe "#create" do
-    subject { carton }
-
-    it "raises no errors" do
-      subject
-    end
-  end
-
-  describe "#tracking_url" do
-    subject do
-      carton.tracking_url
-    end
-
-    let(:carton) { create(:carton, shipping_method: shipping_method) }
-    let(:shipping_method) do
-      create(:shipping_method, tracking_url: "https://example.com/:tracking")
-    end
-
-    context "when tracking is not present" do
-      it { is_expected.to be_nil }
-    end
-
-    context "when tracking is present" do
-      let(:carton) do
-        create(:carton, shipping_method: shipping_method, tracking: "1Z12345")
-      end
-
-      it "uses shipping method to determine url" do
-        is_expected.to eq("https://example.com/1Z12345")
-      end
-    end
-  end
-
-  describe "#to_param" do
-    subject do
-      carton.to_param
-    end
-
-    it { is_expected.to eq carton.number }
-  end
-
-  describe "#order_numbers" do
-    subject { carton.order_numbers }
-    let(:order) { carton.orders.first }
-
-    it "returns a list of the order numbers it is associated to" do
-      expect(subject).to eq [order.number]
-    end
-  end
-
-  describe "#shipment_numbers" do
-    subject { carton.shipment_numbers }
-    let(:shipment) { carton.shipments.first }
-
-    it "returns a list of the shipment numbers it is associated to" do
-      expect(subject).to eq [shipment.number]
-    end
-  end
-
-  describe "#order_emails" do
-    subject { carton.order_emails }
-
-    let(:carton) { create(:carton, inventory_units: [first_order.inventory_units, second_order.inventory_units].flatten) }
-    let(:first_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:second_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:email) { 'something@something.com' }
-
-    before do
-      first_order.update_attributes!(email: email)
-      second_order.update_attributes!(email: email)
-    end
-
-    it "returns a unique list of the order emails it is associated to" do
-      expect(subject).to eq [email]
-    end
-  end
-
-  describe "#manifest" do
-    subject { carton.manifest }
-
-    let(:carton) { create(:carton, inventory_units: [first_order.inventory_units, second_order.inventory_units].flatten) }
-    let(:first_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:first_line_item) { first_order.line_items.first }
-    let(:second_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:second_line_item) { second_order.line_items.first }
-
-    it "contains only the items in both the carton and order" do
-      expect(subject.map(&:line_item)).to match_array([first_line_item, second_line_item])
-    end
-  end
-
-  describe "#manifest_for_order" do
-    subject { carton.manifest_for_order(first_order) }
-
-    let(:carton) { create(:carton, inventory_units: [first_order.inventory_units, second_order.inventory_units].flatten) }
-    let(:first_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:first_line_item) { first_order.line_items.first }
-    let(:second_order) { create(:order_ready_to_ship, line_items_count: 1) }
-
-    it "contains only the items in both the carton and order" do
-      expect(subject.map(&:line_item)).to eq [first_line_item]
-    end
-  end
-
-  describe "#any_exchanges?" do
-    subject { carton.any_exchanges? }
-
-    let(:carton) { create(:carton, inventory_units: [first_order.inventory_units, second_order.inventory_units].flatten) }
-    let(:first_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:second_order) { create(:order_ready_to_ship, line_items_count: 1) }
-
-    context "when any of the inventory has an original return item" do
-      let(:return_item) { create(:return_item) }
-      before do
-        first_order.inventory_units.first.original_return_item = return_item
-        first_order.save
-      end
-
-      it "is true" do
-        expect(subject).to be_truthy
-      end
-    end
-
-    context "when none of the inventory has an original return item" do
-      it "is false" do
-        expect(subject).to be_falsey
-      end
-    end
-  end
-end

data/spec/models/spree/classification_spec.rb

@@ -1,114 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-module Spree
-  RSpec.describe Classification, type: :model do
-    # Regression test for https://github.com/spree/spree/issues/3494
-    it "cannot link the same taxon to the same product more than once" do
-      product = create(:product)
-      taxon = create(:taxon)
-      add_taxon = lambda { product.taxons << taxon }
-      add_taxon.call
-      expect(add_taxon).to raise_error(ActiveRecord::RecordInvalid)
-    end
-
-    let(:taxon_with_5_products) do
-      products = []
-      5.times do
-        products << create(:base_product)
-      end
-
-      create(:taxon, products: products)
-    end
-
-    def positions_to_be_valid(taxon)
-      positions = taxon.reload.classifications.map(&:position)
-      expect(positions).to eq((1..taxon.classifications.count).to_a)
-    end
-
-    it "has a valid fixtures" do
-      expect positions_to_be_valid(taxon_with_5_products)
-      expect(Spree::Classification.count).to eq 5
-    end
-
-    context "removing product from taxon" do
-      before :each do
-        p = taxon_with_5_products.products[1]
-        expect(p.classifications.first.position).to eq(2)
-        taxon_with_5_products.products.destroy(p)
-      end
-
-      it "resets positions" do
-        expect positions_to_be_valid(taxon_with_5_products)
-      end
-    end
-
-    context "replacing taxon's products" do
-      before :each do
-        products = taxon_with_5_products.products.to_a
-        products.pop(1)
-        taxon_with_5_products.products = products
-        taxon_with_5_products.save!
-      end
-
-      it "resets positions" do
-        expect positions_to_be_valid(taxon_with_5_products)
-      end
-    end
-
-    context "removing taxon from product" do
-      before :each do
-        p = taxon_with_5_products.products[1]
-        p.taxons.destroy(taxon_with_5_products)
-        p.save!
-      end
-
-      it "resets positions" do
-        expect positions_to_be_valid(taxon_with_5_products)
-      end
-    end
-
-    context "replacing product's taxons" do
-      before :each do
-        p = taxon_with_5_products.products[1]
-        p.taxons = []
-        p.save!
-      end
-
-      it "resets positions" do
-        expect positions_to_be_valid(taxon_with_5_products)
-      end
-    end
-
-    context "destroying classification" do
-      before :each do
-        classification = taxon_with_5_products.classifications[1]
-        classification.destroy
-      end
-
-      it "resets positions" do
-        expect positions_to_be_valid(taxon_with_5_products)
-      end
-    end
-
-    it "touches the product" do
-      taxon = taxon_with_5_products
-      classification = taxon.classifications.first
-      product = classification.product
-      product.update_columns(updated_at: 1.day.ago)
-      expect {
-        classification.touch
-      }.to change { product.reload.updated_at }
-    end
-
-    it "touches the taxon" do
-      taxon = taxon_with_5_products
-      classification = taxon.classifications.first
-      taxon.update_columns(updated_at: 1.day.ago)
-      expect {
-        classification.touch
-      }.to change { taxon.reload.updated_at }
-    end
-  end
-end

data/spec/models/spree/concerns/display_money_spec.rb

@@ -1,44 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-module Spree
-  RSpec.describe DisplayMoney do
-    let(:test_class) do
-      Class.new do
-        extend DisplayMoney
-        def total; 10.0; end
-      end
-    end
-
-    describe ".money_methods" do
-      before { test_class.money_methods :total }
-
-      context "currency is not defined" do
-        it "generates a display_method that builds a Spree::Money without options" do
-          expect(test_class.new.display_total).to eq Spree::Money.new(10.0)
-        end
-      end
-
-      context "currency is defined" do
-        before { test_class.class_eval { def currency; "USD"; end } }
-
-        it "generates a display_* method that builds a Spree::Money with currency" do
-          expect(test_class.new.display_total).to eq Spree::Money.new(10.0, currency: "USD")
-        end
-      end
-
-      context "with multiple + options" do
-        before do
-          test_class.class_eval { def amount; 20.0; end }
-          test_class.money_methods :total, amount: { no_cents: true }
-        end
-
-        it "generates a display_* method that builds a Spree::Money with the specified options" do
-          expect(test_class.new.display_total).to eq Spree::Money.new(10.0)
-          expect(test_class.new.display_amount).to eq Spree::Money.new(20.0, no_cents: true)
-        end
-      end
-    end
-  end
-end

data/spec/models/spree/concerns/ordered_property_value_list_spec.rb

@@ -1,52 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::OrderedPropertyValueList do
-  #
-  # Using ProductProperty as a subject
-  # since it includes OrderedPropertyValueList
-  #
-
-  context 'positioning' do
-    let(:product_1) { create(:product) }
-    let!(:property_1) { create(:product_property, product: product_1) }
-    let!(:property_2) { create(:product_property, product: product_1) }
-
-    let(:product_2) { create(:product) }
-    let!(:property_3) { create(:product_property, product: product_2) }
-    let!(:property_4) { create(:product_property, product: product_2) }
-
-    before do
-      property_1.update_attribute(:position, 0)
-      property_2.update_attribute(:position, 1)
-      property_3.update_attribute(:position, 0)
-      property_4.update_attribute(:position, 1)
-    end
-
-    it 'scopes position to the product' do
-      expect(property_1.reload.position).to eq(0)
-      expect(property_2.reload.position).to eq(1)
-      expect(property_3.reload.position).to eq(0)
-      expect(property_4.reload.position).to eq(1)
-    end
-  end
-
-  context "validations" do
-    let(:product_property) { create(:product_property) }
-
-    # Only MySQL stores or stores that were migrated prior to the Rails 4.2
-    # upgrade have length limitations on "value":
-    # > The PostgreSQL and SQLite adapters no longer add a default limit of 255
-    # > characters on string columns.
-    # http://guides.rubyonrails.org/4_2_release_notes.html#active-record-notable-changes
-    # https://github.com/rails/rails/pull/14579
-    if Spree::ProductProperty.columns_hash['value'].limit
-      it "should validate length of value" do
-        overflow_length = Spree::ProductProperty.columns_hash['value'].limit + 1
-        product_property.value = "x" * overflow_length
-        expect(product_property).not_to be_valid
-      end
-    end
-  end
-end

data/spec/models/spree/concerns/user_address_book_spec.rb

@@ -1,358 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-module Spree
-  RSpec.describe UserAddressBook do
-    #
-    # Using LegacyUser as a subject
-    # since it uses the UserAddressBookExtension
-    #
-    let!(:user) { create(:user) }
-
-    describe "#save_in_address_book" do
-      context "saving a default address" do
-        let(:user_address) { user.user_addresses.find_first_by_address_values(address.attributes) }
-
-        subject { user.save_in_address_book(address.attributes, true) }
-
-        context "the address is a new record" do
-          let(:address) { build(:address) }
-
-          it "creates a new Address" do
-            expect { subject }.to change { Spree::Address.count }.by(1)
-          end
-
-          it "creates a UserAddress" do
-            expect { subject }.to change { Spree::UserAddress.count }.by(1)
-          end
-
-          it "sets the UserAddress default flag to true" do
-            subject
-            expect(user_address.default).to eq true
-          end
-
-          it "adds the address to the user's the associated addresses" do
-            expect { subject }.to change { user.reload.addresses.count }.by(1)
-          end
-        end
-
-        context "user already has a default address" do
-          let(:address) { create(:address) }
-          let(:original_default_address) { create(:ship_address) }
-          let(:original_user_address) { user.user_addresses.find_first_by_address_values(original_default_address.attributes) }
-
-          before do
-            user.user_addresses.create(address: original_default_address, default: true)
-          end
-
-          it "makes all the other associated addresses not be the default" do
-            expect { subject }.to change { original_user_address.reload.default }.from(true).to(false)
-          end
-
-          context "an odd flip-flop corner case discovered running backfill rake task" do
-            before do
-              user.save_in_address_book(original_default_address.attributes, true)
-              user.save_in_address_book(address.attributes, true)
-            end
-
-            it "handles setting 2 addresses as default without a reload of user" do
-              user.save_in_address_book(original_default_address.attributes, true)
-              user.save_in_address_book(address.attributes, true)
-              expect(user.addresses.count).to eq 2
-              expect(user.default_address.address1).to eq address.address1
-            end
-          end
-        end
-
-        context "changing existing address to default" do
-          let(:address) { create(:address) }
-
-          before do
-            user.user_addresses.create(address: address, default: false)
-          end
-
-          it "properly sets the default flag" do
-            expect(subject).to eq user.default_address
-          end
-
-          context "and changing another address field at the same time" do
-            let(:updated_address_attributes) { address.attributes.tap { |a| a[:first_name] = "Newbie" } }
-
-            subject { user.save_in_address_book(updated_address_attributes, true) }
-
-            it "changes first name" do
-              expect(subject.first_name).to eq updated_address_attributes[:first_name]
-            end
-
-            it "preserves last name" do
-              expect(subject.last_name).to eq address.last_name
-            end
-
-            it "is a new immutable address instance" do
-              expect(subject.id).to_not eq address.id
-            end
-
-            it "is the new default" do
-              expect(subject).to eq user.default_address
-            end
-          end
-        end
-      end
-
-      context "updating an address and making default at once" do
-        let(:address1) { create(:address) }
-        let(:address2) { create(:address, firstname: "Different") }
-        let(:updated_attrs) do
-          address2.attributes.tap { |a| a[:firstname] = "Johnny" }
-        end
-
-        before do
-          user.save_in_address_book(address1.attributes, true)
-          user.save_in_address_book(address2.attributes, false)
-        end
-
-        it "returns the edit as the first address" do
-          user.save_in_address_book(updated_attrs, true)
-          expect(user.user_addresses.first.address.firstname).to eq "Johnny"
-        end
-      end
-
-      context "saving a non-default address" do
-        let(:user_address) { user.user_addresses.find_first_by_address_values(address.attributes) }
-
-        subject { user.save_in_address_book(address.attributes) }
-
-        context "the address is a new record" do
-          let(:address) { build(:address) }
-
-          it "creates a new Address" do
-            expect { subject }.to change { Spree::Address.count }.by(1)
-          end
-
-          it "creates a UserAddress" do
-            expect { subject }.to change { Spree::UserAddress.count }.by(1)
-          end
-
-          context "it is not the first address" do
-            before { user.user_addresses.create!(address: create(:address)) }
-            it "sets the UserAddress default flag to false" do
-              expect { subject }.to change { Spree::UserAddress.count }.by(1)
-              expect(user_address.default).to eq false
-            end
-          end
-
-          context "it is the first address" do
-            it "sets the UserAddress default flag to true" do
-              subject
-              expect(user_address.default).to eq true
-            end
-          end
-
-          it "adds the address to the user's the associated addresses" do
-            expect { subject }.to change { user.reload.addresses.count }.by(1)
-          end
-        end
-      end
-
-      context "resurrecting a previously saved (but now archived) address" do
-        let(:address) { create(:address) }
-        before do
-          user.save_in_address_book(address.attributes, true)
-          user.remove_from_address_book(address.id)
-        end
-        subject { user.save_in_address_book(address.attributes, true) }
-
-        it "returns the address" do
-          expect(subject).to eq address
-        end
-
-        it "sets it as default" do
-          subject
-          expect(user.default_address).to eq address
-        end
-
-        context "via an edit to another address" do
-          let(:address2) { create(:address, firstname: "Different") }
-          let(:edited_attributes) do
-            # conceptually edit address2 to match the values of address
-            edited_attributes = address.attributes
-            edited_attributes[:id] = address2.id
-            edited_attributes
-          end
-
-          before { user.save_in_address_book(address2.attributes, true) }
-
-          subject { user.save_in_address_book(edited_attributes) }
-
-          it "returns the address" do
-            expect(subject).to eq address
-          end
-
-          it "archives address2" do
-            subject
-            user_address2 = user.user_addresses.all_historical.find_by(address_id: address2.id)
-            expect(user_address2.archived).to be true
-          end
-
-          context "via a new address that matches an archived one" do
-            let(:added_attributes) do
-              added_attributes = address.attributes
-              added_attributes.delete(:id)
-              added_attributes
-            end
-
-            subject { user.save_in_address_book(added_attributes) }
-
-            it "returns the address" do
-              expect(subject).to eq address
-            end
-
-            it "no longer has archived user_addresses" do
-              subject
-              expect(user.user_addresses.all_historical).to eq user.user_addresses
-            end
-          end
-        end
-      end
-    end
-
-    context "#remove_from_address_book" do
-      let(:address1) { create(:address) }
-      let(:address2) { create(:address, firstname: "Different") }
-      let(:remove_id) { address1.id }
-      subject { user.remove_from_address_book(remove_id) }
-
-      before do
-        user.save_in_address_book(address1.attributes)
-        user.save_in_address_book(address2.attributes)
-      end
-
-      it "removes the address from user_addresses" do
-        subject
-        expect(user.user_addresses.find_first_by_address_values(address1.attributes)).to be_nil
-      end
-
-      it "leaves user_address record in an archived state" do
-        subject
-        archived_user_address = user.user_addresses.all_historical.find_first_by_address_values(address1.attributes)
-        expect(archived_user_address).to be_archived
-      end
-
-      it "returns false if the addresses is not there" do
-        expect(user.remove_from_address_book(0)).to be false
-      end
-    end
-
-    context "#persist_order_address" do
-      it 'will save the bill/ship_address reference if it can' do
-        order = create :order
-        user.persist_order_address(order)
-
-        expect( user.bill_address_id ).to eq order.bill_address_id
-        expect( user.ship_address_id ).to eq order.ship_address_id
-        expect( user.bill_address_id ).not_to eq user.ship_address_id
-      end
-
-      context "when automatic_default_address preference is at a default of true" do
-        before do
-          stub_spree_preferences(automatic_default_address: true)
-          expect(user).to receive(:save_in_address_book).with(kind_of(Hash), true)
-          expect(user).to receive(:save_in_address_book).with(kind_of(Hash), false)
-        end
-
-        it "does set the default: true flag" do
-          order = build(:order)
-          user.persist_order_address(order)
-        end
-      end
-
-      context "when automatic_default_address preference is false" do
-        before do
-          stub_spree_preferences(automatic_default_address: false)
-          expect(user).to receive(:save_in_address_book).with(kind_of(Hash), false).twice
-          # and not the optional 2nd argument
-        end
-
-        it "does not set the default: true flag" do
-          order = build(:order)
-          user.persist_order_address(order)
-        end
-      end
-
-      context "when address is nil" do
-        context "when automatic_default_address preference is at a default of true" do
-          before do
-            stub_spree_preferences(automatic_default_address: true)
-            expect(user).to receive(:save_in_address_book).with(kind_of(Hash), true).once
-          end
-
-          it "does not call save_in_address_book on ship address" do
-            order = build(:order)
-            order.ship_address = nil
-
-            user.persist_order_address(order)
-          end
-
-          it "does not call save_in_address_book on bill address" do
-            order = build(:order)
-            order.bill_address = nil
-
-            user.persist_order_address(order)
-          end
-        end
-      end
-
-      context "when automatic_default_address preference is false" do
-        before do
-          stub_spree_preferences(automatic_default_address: false)
-          expect(user).to receive(:save_in_address_book).with(kind_of(Hash), false).once
-        end
-
-        it "does not call save_in_address_book on ship address" do
-          order = build(:order)
-          order.ship_address = nil
-
-          user.persist_order_address(order)
-        end
-
-        it "does not call save_in_address_book on bill address" do
-          order = build(:order)
-          order.bill_address = nil
-
-          user.persist_order_address(order)
-        end
-      end
-    end
-
-    describe "generating a new user with a ship_address at once" do
-      let(:ship_address) { build(:ship_address) }
-      subject { create(:user, ship_address: ship_address) }
-
-      it "stores the ship_address" do
-        expect(subject.ship_address).to eq ship_address
-      end
-
-      it "is also available as default_address" do
-        expect(subject.default_address).to eq ship_address
-      end
-    end
-
-    describe "ship_address=" do
-      let!(:user) { create(:user) }
-      let!(:address) { create(:address) }
-
-      # https://github.com/solidusio/solidus/issues/1241
-      it "resets the association and persists" do
-        # Load (which will cache) the has_one association
-        expect(user.ship_address).to be_nil
-
-        user.update!(ship_address: address)
-        expect(user.ship_address).to eq(address)
-
-        user.reload
-        expect(user.ship_address).to eq(address)
-      end
-    end
-  end
-end