solidus_core 2.10.5 → 3.0.8

data/spec/helpers/products_helper_spec.rb

@@ -1,202 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-module Spree
-  RSpec.describe ProductsHelper, type: :helper do
-    include ProductsHelper
-
-    let(:product) { create(:product, price: product_price) }
-    let(:product_price) { 10 }
-    let(:variant) { create(:variant, product: product, price: variant_price) }
-    let(:currency) { 'USD' }
-    let(:pricing_options) do
-      Spree::Config.pricing_options_class.new(currency: currency)
-    end
-
-    before do
-      without_partial_double_verification do
-        allow(helper).to receive(:current_pricing_options) { pricing_options }
-      end
-    end
-
-    context "#variant_price_diff" do
-      let(:variant_price) { 10 }
-
-      subject { helper.variant_price(variant) }
-
-      context "when variant is same as master" do
-        it { is_expected.to be_nil }
-      end
-
-      context "when currency is default" do
-        context "when variant is more than master" do
-          let(:variant_price) { 15 }
-
-          it { is_expected.to eq("(Add: <span class=\"money-currency-symbol\">$</span><span class=\"money-whole\">5</span><span class=\"money-decimal-mark\">.</span><span class=\"money-decimal\">00</span>)") }
-          # Regression test for https://github.com/spree/spree/issues/2737
-          it { is_expected.to be_html_safe }
-        end
-
-        context "when variant is less than master" do
-          let(:product_price) { 15 }
-
-          it { is_expected.to eq("(Subtract: <span class=\"money-currency-symbol\">$</span><span class=\"money-whole\">5</span><span class=\"money-decimal-mark\">.</span><span class=\"money-decimal\">00</span>)") }
-        end
-      end
-
-      context "when currency is JPY" do
-        let(:variant_price) { 100 }
-        let(:product_price) { 100 }
-        let(:currency) { 'JPY' }
-
-        before do
-          variant
-          product.prices.update_all(currency: currency)
-        end
-
-        context "when variant is more than master" do
-          let(:variant_price) { 150 }
-
-          it { is_expected.to eq("(Add: <span class=\"money-currency-symbol\">&#x00A5;</span><span class=\"money-whole\">50</span>)") }
-        end
-
-        context "when variant is less than master" do
-          let(:product_price) { 150 }
-
-          it { is_expected.to eq("(Subtract: <span class=\"money-currency-symbol\">&#x00A5;</span><span class=\"money-whole\">50</span>)") }
-        end
-      end
-    end
-
-    context "#variant_price_full" do
-      let!(:variant_2) { create(:variant, product: product, price: variant_2_price) }
-      let(:variant_2_price) { 20 }
-      let(:variant_price) { 15 }
-
-      before do
-        stub_spree_preferences(show_variant_full_price: true)
-        variant
-      end
-
-      context "when currency is default" do
-        it "should return the variant price if the price is different than master" do
-          expect(helper.variant_price(variant)).to eq("<span class=\"money-currency-symbol\">$</span><span class=\"money-whole\">15</span><span class=\"money-decimal-mark\">.</span><span class=\"money-decimal\">00</span>")
-          expect(helper.variant_price(variant_2)).to eq("<span class=\"money-currency-symbol\">$</span><span class=\"money-whole\">20</span><span class=\"money-decimal-mark\">.</span><span class=\"money-decimal\">00</span>")
-        end
-      end
-
-      context "when currency is JPY" do
-        let(:currency) { 'JPY' }
-        let(:product_price) { 100 }
-        let(:variant_price) { 150 }
-
-        before do
-          variant
-          product.prices.update_all(currency: currency)
-        end
-
-        it "should return the variant price if the price is different than master" do
-          expect(helper.variant_price(variant)).to eq("<span class=\"money-currency-symbol\">&#x00A5;</span><span class=\"money-whole\">150</span>")
-        end
-      end
-
-      context "when all variant prices are equal" do
-        let(:product_price) { 10 }
-        let(:variant_price) { 10 }
-        let(:variant_2_price) { 10 }
-
-        it "should be nil" do
-          expect(helper.variant_price(variant)).to be_nil
-          expect(helper.variant_price(variant_2)).to be_nil
-        end
-      end
-    end
-
-    context "#product_description" do
-      # Regression test for https://github.com/spree/spree/issues/1607
-      it "renders a product description without excessive paragraph breaks" do
-        product.description = %{
-<p>Lorem ipsum dolor sit amet, consectetur adipiscing elit. Vivamus a ligula leo. Proin eu arcu at ipsum dapibus ullamcorper. Pellentesque egestas orci nec magna condimentum luctus. Cum sociis natoque penatibus et magnis dis parturient montes, nascetur ridiculus mus. Ut ac ante et mauris bibendum ultricies non sed massa. Fusce facilisis dui eget lacus scelerisque eget aliquam urna ultricies. Duis et rhoncus quam. Praesent tellus nisi, ultrices sed iaculis quis, euismod interdum ipsum.</p>
-<ul>
-<li>Lorem ipsum dolor sit amet</li>
-<li>Lorem ipsum dolor sit amet</li>
-</ul>
-        }
-        description = product_description(product)
-        expect(description.strip).to eq(product.description.strip)
-      end
-
-      it "renders a product description with automatic paragraph breaks" do
-        product.description = %{
-THIS IS THE BEST PRODUCT EVER!
-
-"IT CHANGED MY LIFE" - Sue, MD}
-
-        description = product_description(product)
-        expect(description.strip).to eq(%{<p>\nTHIS IS THE BEST PRODUCT EVER!</p>"IT CHANGED MY LIFE" - Sue, MD})
-      end
-
-      it "renders a product description without any formatting based on configuration" do
-        description = %{
-            <p>hello world</p>
-
-            <p>tihs is completely awesome and it works</p>
-
-            <p>why so many spaces in the code. and why some more formatting afterwards?</p>
-        }
-
-        product.description = description
-
-        stub_spree_preferences(show_raw_product_description: true)
-        description = product_description(product)
-        expect(description).to eq(description)
-      end
-    end
-
-    context '#line_item_description_text' do
-      subject { line_item_description_text description }
-      context 'variant has a blank description' do
-        let(:description) { nil }
-        it { is_expected.to eq(I18n.t('spree.product_has_no_description')) }
-      end
-      context 'variant has a description' do
-        let(:description) { 'test_desc' }
-        it { is_expected.to eq(description) }
-      end
-      context 'description has nonbreaking spaces' do
-        let(:description) { 'test&nbsp;desc' }
-        it { is_expected.to eq('test desc') }
-      end
-    end
-
-    context '#cache_key_for_products' do
-      subject { helper.cache_key_for_products }
-      before(:each) do
-        @products = double('products collection')
-        allow(helper).to receive(:params) { { page: 10 } }
-      end
-
-      context 'when there is a maximum updated date' do
-        let(:updated_at) { Date.new(2011, 12, 13) }
-        before :each do
-          allow(@products).to receive(:count) { 5 }
-          allow(@products).to receive(:maximum).with(:updated_at) { updated_at }
-        end
-
-        it { is_expected.to eq('en/USD/spree/products/all-10-20111213-5') }
-      end
-
-      context 'when there is no considered maximum updated date' do
-        let(:today) { Date.new(2013, 12, 11) }
-        before :each do
-          allow(@products).to receive(:count) { 1_234_567 }
-          allow(@products).to receive(:maximum).with(:updated_at) { nil }
-          allow(Date).to receive(:today) { today }
-        end
-
-        it { is_expected.to eq('en/USD/spree/products/all-10-20131211-1234567') }
-      end
-    end
-  end
-end

data/spec/helpers/taxons_helper_spec.rb

@@ -1,43 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::TaxonsHelper, type: :helper do
-  let(:currency) { 'USD' }
-  let(:pricing_options) do
-    Spree::Config.pricing_options_class.new(currency: currency)
-  end
-  before do
-    without_partial_double_verification do
-      allow(helper).to receive(:current_pricing_options) { pricing_options }
-    end
-  end
-
-  describe "#taxon_preview" do
-    let!(:taxon) { create(:taxon) }
-    let!(:child_taxon) { create(:taxon, parent: taxon) }
-    let!(:product_1) { create(:product) }
-    let!(:product_2) { create(:product) }
-    let!(:product_3) { create(:product) }
-
-    before do
-      taxon.products << product_1
-      taxon.products << product_2
-      child_taxon.products << product_3
-      taxon.reload
-    end
-
-    # Regression test for https://github.com/spree/spree/issues/4382
-    it "returns products" do
-      expect(helper.taxon_preview(taxon)).to eql([product_1, product_2, product_3])
-    end
-
-    context "with different currency" do
-      let(:currency) { 'CAD' }
-
-      it "returns no products" do
-        expect(helper.taxon_preview(taxon)).to be_empty
-      end
-    end
-  end
-end

data/spec/jobs/promotion_code_batch_job_spec.rb

@@ -1,96 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-RSpec.describe Spree::PromotionCodeBatchJob, type: :job do
-  let(:email) { "test@email.com" }
-  let(:promotion_code_batch) do
-    Spree::PromotionCodeBatch.create!(
-      promotion_id: create(:promotion).id,
-      base_code: "test",
-      number_of_codes: 10,
-      email: email
-    )
-  end
-  context "with a successful build" do
-    before do
-      allow(Spree::PromotionCodeBatchMailer)
-        .to receive(:promotion_code_batch_finished)
-        .and_call_original
-    end
-
-    def codes
-      Spree::PromotionCode.pluck(:value)
-    end
-
-    context 'with the default join character' do
-      it 'uses the default join characters', :aggregate_failures do
-        subject.perform(promotion_code_batch)
-        codes.each do |code|
-          expect(code).to match(/^test_/)
-        end
-      end
-    end
-    context 'with a custom join character' do
-      let(:promotion_code_batch) do
-        Spree::PromotionCodeBatch.create!(
-          promotion_id: create(:promotion).id,
-          base_code: "test",
-          number_of_codes: 10,
-          email: email,
-          join_characters: '-'
-        )
-      end
-      it 'uses the custom join characters', :aggregate_failures do
-        subject.perform(promotion_code_batch)
-        codes.each do |code|
-          expect(code).to match(/^test-/)
-        end
-      end
-    end
-    context "with an email address" do
-      it "sends an email" do
-        subject.perform(promotion_code_batch)
-        expect(Spree::PromotionCodeBatchMailer)
-          .to have_received(:promotion_code_batch_finished)
-      end
-    end
-    context "with no email address" do
-      let(:email) { nil }
-      it "sends an email" do
-        subject.perform(promotion_code_batch)
-        expect(Spree::PromotionCodeBatchMailer)
-          .to_not have_received(:promotion_code_batch_finished)
-      end
-    end
-  end
-
-  context "with a failed build" do
-    before do
-      allow_any_instance_of(Spree::PromotionCode::BatchBuilder)
-        .to receive(:build_promotion_codes)
-        .and_raise("Error")
-
-      allow(Spree::PromotionCodeBatchMailer)
-        .to receive(:promotion_code_batch_errored)
-        .and_call_original
-
-      expect { subject.perform(promotion_code_batch) }
-        .to raise_error RuntimeError
-    end
-
-    context "with an email address" do
-      it "sends an email" do
-        expect(Spree::PromotionCodeBatchMailer)
-          .to have_received(:promotion_code_batch_errored)
-      end
-    end
-
-    context "with no email address" do
-      let(:email) { nil }
-      it "sends an email" do
-        expect(Spree::PromotionCodeBatchMailer)
-          .to_not have_received(:promotion_code_batch_errored)
-      end
-    end
-  end
-end

data/spec/lib/calculated_adjustments_spec.rb

@@ -1,133 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::CalculatedAdjustments do
-  let(:calculator_class) { Spree::Calculator::FlatRate }
-
-  with_model :Calculable, scope: :all do
-    model do
-      include Spree::CalculatedAdjustments
-    end
-  end
-
-  it "should add has_one :calculator relationship" do
-    expect(Calculable.reflect_on_all_associations(:has_one).map(&:name)).to include(:calculator)
-  end
-
-  describe 'initialization' do
-    context 'with no calculator' do
-      subject { Calculable.new }
-
-      it 'can be initialized' do
-        expect(subject.calculator).to be nil
-      end
-
-      it 'is not valid' do
-        expect(subject).not_to be_valid
-        expect(subject.errors[:calculator]).to eq ["can't be blank"]
-      end
-    end
-
-    context 'with calculator object' do
-      subject { Calculable.new(calculator: calculator_class.new) }
-
-      it 'can be initialized' do
-        expect(subject.calculator).to be_a(calculator_class)
-        expect(calculator_class.count).to eq 0 # not yet saved
-      end
-
-      it 'can be created' do
-        subject.save!
-        expect(subject.calculator).to be_a(calculator_class)
-        expect(calculator_class.count).to eq 1 # saved in database
-      end
-    end
-
-    context 'with calculator_type' do
-      subject { Calculable.new(calculator_type: calculator_class.to_s) }
-
-      it 'can be initialized' do
-        expect(subject.calculator).to be_a(calculator_class)
-        expect(calculator_class.count).to eq 0 # not yet saved
-      end
-
-      it 'can be created' do
-        subject.save!
-        expect(subject.calculator).to be_a(calculator_class)
-        expect(calculator_class.count).to eq 1 # saved in database
-      end
-    end
-
-    context 'with calculator_type and calculator_attributes' do
-      subject { Calculable.new(calculator_type: calculator_class.to_s, calculator_attributes: { preferred_amount: 123 }) }
-
-      it 'can be initialized' do
-        expect(subject.calculator).to be_a(calculator_class)
-        expect(subject.calculator.preferred_amount).to eq 123
-        expect(calculator_class.count).to eq 0 # not yet saved
-      end
-
-      it 'can be created' do
-        subject.save!
-        expect(subject.calculator).to be_a(calculator_class)
-        expect(subject.calculator.preferred_amount).to eq 123
-        expect(calculator_class.count).to eq 1 # saved in database
-      end
-    end
-  end
-
-  describe 'update' do
-    subject { Calculable.create!(calculator_type: calculator_class.to_s) }
-
-    it "can update calculator attributes with id" do
-      subject.update!(calculator_attributes: {
-        id: subject.calculator.id,
-        preferred_amount: 123
-      })
-      expect(subject.calculator.preferred_amount).to eq(123)
-      subject.reload
-      expect(subject.calculator.preferred_amount).to eq(123)
-    end
-
-    it "can update calculator attributes without id" do
-      subject.update!(calculator_attributes: {
-        preferred_amount: 123
-      })
-      expect(subject.calculator.preferred_amount).to eq(123)
-      subject.reload
-      expect(subject.calculator.preferred_amount).to eq(123)
-    end
-
-    it "can update both calculator type and attributes" do
-      subject.update!(calculator_type: 'Spree::Calculator::FlexiRate', calculator_attributes: {
-        preferred_first_item: 123
-      })
-      expect(subject.calculator.class).to eq(Spree::Calculator::FlexiRate)
-      expect(subject.calculator.preferred_first_item).to eq(123)
-      subject.reload
-      expect(subject.calculator.class).to eq(Spree::Calculator::FlexiRate)
-      expect(subject.calculator.preferred_first_item).to eq(123)
-    end
-  end
-
-  describe '#calculator_type=' do
-    subject { Calculable.new }
-
-    let(:calculator_subclass) { Spree::Calculator::Shipping::FlatRate }
-    let(:calculator_superclass) { Spree::ShippingCalculator }
-
-    before(:each) do
-      subject.calculator_type = calculator_subclass.to_s
-    end
-
-    it 'sets calculator type' do
-      expect(subject.calculator_type).to eq(calculator_subclass.to_s)
-    end
-
-    it 'switches from calculator subclass to calculator superclass' do
-      subject.calculator_type = calculator_superclass.to_s
-      expect(subject.calculator_type).to eq(calculator_superclass.to_s)
-    end
-  end
-end

data/spec/lib/i18n_spec.rb

@@ -1,97 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-require 'spree/i18n'
-
-RSpec.describe "i18n" do
-  before do
-    # This reload avoids an issue with I18n.available_locales being cached
-    I18n.reload!
-
-    I18n.backend.store_translations(:en,
-    {
-      spree: {
-        i18n: {
-          this_file_language: "English"
-        },
-        foo: "bar",
-        bar: {
-          foo: "bar within bar scope",
-          invalid: nil,
-          legacy_translation: "back in the day..."
-        },
-        invalid: nil,
-        legacy_translation: "back in the day..."
-      }
-    })
-  end
-  after do
-    I18n.reload!
-  end
-
-  it "translates within the spree scope" do
-    expect(Spree::Deprecation).to receive(:warn).twice
-    expect(Spree.t(:foo)).to eql("bar")
-    expect(Spree.translate(:foo)).to eql("bar")
-  end
-
-  it "prepends a string scope" do
-    expect(Spree::Deprecation).to receive(:warn)
-    expect(Spree.t(:foo, scope: "bar")).to eql("bar within bar scope")
-  end
-
-  it "prepends to an array scope" do
-    expect(Spree::Deprecation).to receive(:warn)
-    expect(Spree.t(:foo, scope: ["bar"])).to eql("bar within bar scope")
-  end
-
-  it "returns two translations" do
-    expect(Spree::Deprecation).to receive(:warn)
-    expect(Spree.t([:foo, 'bar.foo'])).to eql(["bar", "bar within bar scope"])
-  end
-
-  it "returns reasonable string for missing translations" do
-    expect(Spree::Deprecation).to receive(:warn)
-    expect(Spree.t(:missing_entry)).to include("<span")
-  end
-
-  it "should have a Spree::I18N_GENERIC_PLURAL constant" do
-    expect(Spree::I18N_GENERIC_PLURAL).to eq 2.1
-  end
-
-  describe "i18n_available_locales" do
-    it "should only return :en" do
-      expect(Spree.i18n_available_locales).to eq([:en])
-    end
-
-    context 'with unprefixed translations in another locale' do
-      before do
-        I18n.backend.store_translations(:fr, { cheese: "fromage" })
-      end
-
-      it "should only return :en" do
-        expect(Spree.i18n_available_locales).to eq([:en])
-      end
-    end
-
-    context 'with spree-prefixed translations in another locale' do
-      before do
-        I18n.backend.store_translations(:fr, spree: { cheese: "fromage" })
-      end
-
-      it "should return :en and :fr" do
-        expect(Spree.i18n_available_locales).to eq([:en])
-      end
-    end
-
-    context 'with specific desired key' do
-      before do
-        I18n.backend.store_translations(:fr, spree: { i18n: { this_file_language: "Français" } })
-      end
-
-      it "should return :en and :fr" do
-        expect(Spree.i18n_available_locales).to eq([:en, :fr])
-      end
-    end
-  end
-end

data/spec/lib/search/base_spec.rb

@@ -1,91 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require 'spree/core/product_filters'
-
-RSpec.describe Spree::Core::Search::Base do
-  before do
-    include Spree::Core::ProductFilters
-    @taxon = create(:taxon, name: "Ruby on Rails")
-
-    @product1 = create(:product, name: "RoR Mug", price: 9.00)
-    @product1.taxons << @taxon
-    @product2 = create(:product, name: "RoR Shirt", price: 11.00)
-  end
-
-  it "returns all products by default" do
-    params = { per_page: "" }
-    searcher = Spree::Core::Search::Base.new(params)
-    expect(searcher.retrieve_products.count).to eq(2)
-  end
-
-  context "when include_images is included in the initalization params" do
-    let(:params) { { include_images: true, keyword: @product1.name, taxon: @taxon.id } }
-    subject { described_class.new(params).retrieve_products }
-
-    before do
-      @product1.master.images.create(attachment_file_name: "Test", position: 2)
-      @product1.master.images.create(attachment_file_name: "Test1", position: 1)
-      @product1.reload
-    end
-
-    it "returns images in correct order" do
-      expect(subject.first).to eq @product1
-      expect(subject.first.images).to eq @product1.master.images
-    end
-  end
-
-  it "switches to next page according to the page parameter" do
-    @product3 = create(:product, name: "RoR Pants", price: 14.00)
-
-    params = { per_page: "2" }
-    searcher = Spree::Core::Search::Base.new(params)
-    expect(searcher.retrieve_products.count).to eq(2)
-
-    params[:page] = "2"
-    searcher = Spree::Core::Search::Base.new(params)
-    expect(searcher.retrieve_products.count).to eq(1)
-  end
-
-  it "maps search params to named scopes" do
-    params = { per_page: "",
-               search: { "price_range_any" => ["Under $10.00"] } }
-    searcher = Spree::Core::Search::Base.new(params)
-    expect(searcher.send(:get_base_scope).to_sql).to match /<= 10/
-    expect(searcher.retrieve_products.count).to eq(1)
-  end
-
-  it "maps multiple price_range_any filters" do
-    params = { per_page: "",
-               search: { "price_range_any" => ["Under $10.00", "$10.00 - $15.00"] } }
-    searcher = Spree::Core::Search::Base.new(params)
-    expect(searcher.send(:get_base_scope).to_sql).to match /<= 10/
-    if Rails.gem_version >= Gem::Version.new('6.0.0')
-      expect(searcher.send(:get_base_scope).to_sql).to match /between 10\.0 and 15\.0/i
-    else
-      expect(searcher.send(:get_base_scope).to_sql).to match /between 10 and 15/i
-    end
-    expect(searcher.retrieve_products.count).to eq(2)
-  end
-
-  it "uses ransack if scope not found" do
-    params = { per_page: "",
-               search: { "name_not_cont" => "Shirt" } }
-    searcher = Spree::Core::Search::Base.new(params)
-    expect(searcher.retrieve_products.count).to eq(1)
-  end
-
-  it "accepts a current user" do
-    user = double
-    searcher = Spree::Core::Search::Base.new({})
-    searcher.current_user = user
-    expect(searcher.current_user).to eql(user)
-  end
-
-  it "finds products in alternate currencies" do
-    create(:price, currency: 'EUR', variant: @product1.master)
-    searcher = Spree::Core::Search::Base.new({})
-    searcher.pricing_options = Spree::Config.pricing_options_class.new(currency: 'EUR')
-    expect(searcher.retrieve_products).to eq([@product1])
-  end
-end