solidus_core 2.10.1 → 2.11.2

data/spec/lib/spree/migrations_spec.rb

@@ -1,32 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-module Spree
-  RSpec.describe Migrations do
-    let(:app_migrations) { [".", "34_add_title.rb", "52_add_text.rb"] }
-    let(:engine_migrations) { [".", "334_create_orders.spree.rb", "777_create_products.spree.rb"] }
-
-    let(:config) { double("Config", root: "dir") }
-
-    let(:engine_dir) { "dir/db/migrate" }
-    let(:app_dir) { "#{Rails.root}/db/migrate" }
-
-    subject { described_class.new(config, "spree") }
-
-    it "detects missing migrations" do
-      expect(Dir).to receive(:entries).with(app_dir).and_return app_migrations
-      expect(Dir).to receive(:entries).with(engine_dir).and_return engine_migrations
-
-      expect(subject.missing_migrations.size).to eq 2
-    end
-
-    context "no missing migrations" do
-      it "says nothing" do
-        expect(Dir).to receive(:entries).with(engine_dir).and_return engine_migrations
-        expect(Dir).to receive(:entries).with(app_dir).and_return(app_migrations + engine_migrations)
-        expect(subject.missing_migrations.size).to eq 0
-      end
-    end
-  end
-end

data/spec/lib/spree/money_spec.rb

@@ -1,298 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::Money do
-  before do
-    stub_spree_preferences(currency: "USD")
-  end
-
-  describe '#initialize' do
-    subject do
-      Spree::Deprecation.silence do
-        described_class.new(amount, currency: currency, with_currency: true).to_s
-      end
-    end
-
-    context 'with no currency' do
-      let(:currency) { nil }
-      let(:amount){ 10 }
-      it { should == "$10.00 USD" }
-    end
-
-    context 'with currency' do
-      let(:currency){ 'USD' }
-
-      context "CAD" do
-        let(:amount){ '10.00' }
-        let(:currency){ 'CAD' }
-        it { should == "$10.00 CAD" }
-      end
-
-      context "with string amount" do
-        let(:amount){ '10.00' }
-        it { should == "$10.00 USD" }
-      end
-
-      context "with no decimal point" do
-        let(:amount){ '10' }
-        it { should == "$10.00 USD" }
-      end
-
-      context "with symbol" do
-        let(:amount){ '$10.00' }
-        it { should == "$10.00 USD" }
-      end
-
-      context "with extra currency" do
-        let(:amount){ '$10.00 USD' }
-        it { should == "$10.00 USD" }
-      end
-
-      context "with different currency" do
-        let(:currency){ 'USD' }
-        let(:amount){ '$10.00 CAD' }
-        it { should == "$10.00 CAD" }
-      end
-
-      context "with commas" do
-        let(:amount){ '1,000.00' }
-        it { should == "$1,000.00 USD" }
-      end
-
-      context "with comma for decimal point" do
-        let(:amount){ '10,00' }
-        it { should == "$10.00 USD" }
-      end
-
-      context 'with fixnum' do
-        let(:amount){ 10 }
-        it { should == "$10.00 USD" }
-      end
-
-      context 'with float' do
-        let(:amount){ 10.00 }
-        it { should == "$10.00 USD" }
-      end
-
-      context 'with BigDecimal' do
-        let(:amount){ BigDecimal('10.00') }
-        it { should == "$10.00 USD" }
-      end
-    end
-  end
-
-  it "formats correctly" do
-    money = Spree::Money.new(10)
-    expect(money.to_s).to eq("$10.00")
-  end
-
-  it "can get cents" do
-    money = Spree::Money.new(10)
-    expect(money.cents).to eq(1000)
-  end
-
-  context "with currency" do
-    it "passed in option" do
-      money = Spree::Money.new(10, with_currency: true, html_wrap: false)
-      expect(money.to_s).to eq("$10.00 USD")
-    end
-  end
-
-  context "hide cents" do
-    it "hides cents suffix" do
-      money = Spree::Money.new(10, no_cents: true)
-      expect(money.to_s).to eq("$10")
-    end
-
-    it "shows cents suffix" do
-      money = Spree::Money.new(10)
-      expect(money.to_s).to eq("$10.00")
-    end
-  end
-
-  context "currency parameter" do
-    context "when currency is specified in Canadian Dollars" do
-      it "uses the currency param over the global configuration" do
-        money = Spree::Money.new(10, currency: 'CAD', with_currency: true, html_wrap: false)
-        expect(money.to_s).to eq("$10.00 CAD")
-      end
-    end
-
-    context "when currency is specified in Japanese Yen" do
-      it "uses the currency param over the global configuration" do
-        money = Spree::Money.new(100, currency: 'JPY', html_wrap: false)
-        expect(money.to_s).to eq("¥100")
-      end
-    end
-  end
-
-  context "symbol positioning" do
-    it "passed in option" do
-      money = Spree::Money.new(10, format: '%n %u', html_wrap: false)
-      expect(money.to_s).to eq("10.00 $")
-    end
-
-    it "config option" do
-      money = Spree::Money.new(10, format: '%n %u', html_wrap: false)
-      expect(money.to_s).to eq("10.00 $")
-    end
-  end
-
-  context "sign before symbol" do
-    it "defaults to -$10.00" do
-      money = Spree::Money.new(-10)
-      expect(money.to_s).to eq("-$10.00")
-    end
-
-    it "passed in option" do
-      money = Spree::Money.new(-10, sign_before_symbol: false)
-      expect(money.to_s).to eq("$-10.00")
-    end
-  end
-
-  context "JPY" do
-    before do
-      stub_spree_preferences(currency: "JPY")
-    end
-
-    it "formats correctly" do
-      money = Spree::Money.new(1000, html_wrap: false)
-      expect(money.to_s).to eq("¥1,000")
-    end
-  end
-
-  context "EUR" do
-    before do
-      stub_spree_preferences(currency: "EUR")
-    end
-
-    # Regression test for https://github.com/spree/spree/issues/2634
-    it "formats as plain by default" do
-      money = Spree::Money.new(10, format: '%n %u')
-      expect(money.to_s).to eq("10.00 €")
-    end
-
-    it "formats as HTML if asked (nicely) to" do
-      money = Spree::Money.new(10, format: '%n %u')
-      # The HTML'ified version of "10.00 €"
-      expect(money.to_html).to eq("<span class=\"money-whole\">10</span><span class=\"money-decimal-mark\">.</span><span class=\"money-decimal\">00</span> <span class=\"money-currency-symbol\">&#x20AC;</span>")
-    end
-
-    it "formats as HTML with currency" do
-      money = Spree::Money.new(10, format: '%n %u', with_currency: true)
-      # The HTML'ified version of "10.00 €"
-      expect(money.to_html).to eq("<span class=\"money-whole\">10</span><span class=\"money-decimal-mark\">.</span><span class=\"money-decimal\">00</span> <span class=\"money-currency-symbol\">&#x20AC;</span> <span class=\"money-currency\">EUR</span>")
-    end
-  end
-
-  describe "#as_json" do
-    let(:options) { double('options') }
-
-    it "returns the expected string" do
-      money = Spree::Money.new(10)
-      expect(money.as_json(options)).to eq("$10.00")
-    end
-  end
-
-  describe 'subtraction' do
-    context "for money objects with same currency" do
-      let(:money_1) { Spree::Money.new(32.00, currency: "USD") }
-      let(:money_2) { Spree::Money.new(15.00, currency: "USD") }
-
-      it "subtracts correctly" do
-        expect(money_1 - money_2).to eq(Spree::Money.new(17.00, currency: "USD"))
-      end
-    end
-
-    context "when trying to subtract money objects in different currencies" do
-      let(:money_1) { Spree::Money.new(32.00, currency: "EUR") }
-      let(:money_2) { Spree::Money.new(15.00, currency: "USD") }
-
-      it "will not work" do
-        expect { money_1 - money_2 }.to raise_error(Money::Bank::UnknownRate)
-      end
-    end
-
-    context "if other does not respond to .money" do
-      let(:money_1) { Spree::Money.new(32.00, currency: "EUR") }
-      let(:money_2) { ::Money.new(1500) }
-
-      it 'raises a TypeError' do
-        expect { money_1 - money_2 }.to raise_error(TypeError)
-      end
-    end
-  end
-
-  describe 'addition' do
-    context "for money objects with same currency" do
-      let(:money_1) { Spree::Money.new(37.00, currency: "USD") }
-      let(:money_2) { Spree::Money.new(15.00, currency: "USD") }
-
-      it "subtracts correctly" do
-        expect(money_1 + money_2).to eq(Spree::Money.new(52.00, currency: "USD"))
-      end
-    end
-
-    context "when trying to subtract money objects in different currencies" do
-      let(:money_1) { Spree::Money.new(32.00, currency: "EUR") }
-      let(:money_2) { Spree::Money.new(15.00, currency: "USD") }
-
-      it "will not work" do
-        expect { money_1 + money_2 }.to raise_error(Money::Bank::UnknownRate)
-      end
-    end
-
-    context "if other does not respond to .money" do
-      let(:money_1) { Spree::Money.new(32.00, currency: "EUR") }
-      let(:money_2) { ::Money.new(1500) }
-
-      it 'raises a TypeError' do
-        expect { money_1 + money_2 }.to raise_error(TypeError)
-      end
-    end
-  end
-
-  describe 'equality checks' do
-    context "if other does not respond to .money" do
-      let(:money_1) { Spree::Money.new(32.00, currency: "EUR") }
-      let(:money_2) { ::Money.new(1500) }
-
-      it 'raises a TypeError' do
-        expect { money_1 == money_2 }.to raise_error(TypeError)
-      end
-    end
-  end
-
-  it "includes Comparable" do
-    expect(described_class).to include(Comparable)
-  end
-
-  describe "<=>" do
-    let(:usd_10) { Spree::Money.new(10, currency: "USD") }
-    let(:usd_20) { Spree::Money.new(20, currency: "USD") }
-    let(:usd_30) { Spree::Money.new(30, currency: "USD") }
-
-    it "compares the two amounts" do
-      expect(usd_20 <=> usd_20).to eq 0
-      expect(usd_20 <=> usd_10).to be > 0
-      expect(usd_20 <=> usd_30).to be < 0
-    end
-
-    context "with a non Spree::Money object" do
-      it "raises an error" do
-        expect { usd_10 <=> 20 }.to raise_error(TypeError)
-      end
-    end
-
-    context "with differing currencies" do
-      let(:cad) { Spree::Money.new(10, currency: "CAD") }
-
-      it "raises an error" do
-        expect { usd_10 <=> cad }.to raise_error(
-          Spree::Money::DifferentCurrencyError
-        )
-      end
-    end
-  end
-end

data/spec/lib/spree/permission_sets/default_customer_spec.rb

@@ -1,22 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::PermissionSets::DefaultCustomer do
-  context 'as Guest User' do
-    context 'for Order' do
-      context 'guest_token is empty string' do
-        let(:ability) { Spree::Ability.new(nil) }
-        let(:resource) { build(:order) }
-        let(:token) { '' }
-
-        it 'should not be allowed to read or update the order' do
-          allow(resource).to receive_messages(guest_token: '')
-
-          expect(ability).to_not be_able_to(:read, resource, token)
-          expect(ability).to_not be_able_to(:update, resource, token)
-        end
-      end
-    end
-  end
-end

data/spec/lib/spree/promo/environment_spec.rb

@@ -1,53 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-require 'spree/core/environment/promotions'
-require 'spree/promo/environment'
-
-RSpec.describe 'Spree::Promo::Environment' do
-  it 'is deprecated' do
-    expect(Spree::Deprecation).to receive(:warn)
-
-    Spree::Promo::Environment.new
-  end
-
-  context 'when customized' do
-    after do
-      # This is needed to cleanup classes after the following specs run.
-      # They will add two new methods but we don't want them to be defined
-      # on the described class for specs that will run later.
-      Spree::Core::Environment::Promotions.remove_method :custom_rules, :custom_rules=
-    end
-
-    context 'with class_eval' do
-      it 'raises a deprecation warning but keep changes' do
-        expect(Spree::Deprecation).to receive(:warn)
-
-        Spree::Promo::Environment.class_eval do
-          add_class_set :custom_rules
-        end
-
-        promo_environment_instance = Spree::Core::Environment::Promotions.new
-        expect(promo_environment_instance).to respond_to(:custom_rules)
-        expect(promo_environment_instance).to respond_to(:custom_rules=)
-      end
-    end
-
-    context 'with prepend' do
-      it 'raises a deprecation warning but keep changes' do
-        expect(Spree::Deprecation).to receive(:warn)
-
-        module CustomRules
-          def self.prepended(base)
-            base.add_class_set :custom_rules
-          end
-        end
-        Spree::Promo::Environment.prepend CustomRules
-
-        promo_environment_instance = Spree::Core::Environment::Promotions.new
-        expect(promo_environment_instance).to respond_to(:custom_rules)
-        expect(promo_environment_instance).to respond_to(:custom_rules=)
-      end
-    end
-  end
-end

data/spec/lib/tasks/dummy_task.rake

@@ -1,12 +0,0 @@
-# frozen_string_literal: true
-
-# This is a dummy task used for generic rake task testing
-
-task dummy_task: :environment do
-  DummyTaskRunner.run
-end
-
-class DummyTaskRunner
-  def self.run
-  end
-end

data/spec/lib/tasks/dummy_task_spec.rb

@@ -1,30 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe 'dummy_task' do
-  include_context(
-    'rake',
-    task_name: 'dummy_task',
-    task_path: Spree::Core::Engine.root.join('spec/lib/tasks/dummy_task.rake'),
-  )
-
-  it 'calls the dummy task exactly once' do
-    expect(DummyTaskRunner).to receive(:run).once
-    task.invoke
-  end
-
-  # This tests:
-  #   1) that tasks get reenabled between examples
-  #   2) that tasks aren't loaded in the wrong way, causing them to execute
-  #      an extra time for every example that's defined.
-  # We need at least two specs to trigger the error conditions and spec order is
-  # random so we just create the same spec twice. We could probably combine this
-  # with the generic spec above but this seems clearer.
-  2.times do |i|
-    it "still calls the dummy task exactly once when more than one example is defined - #{i}" do
-      expect(DummyTaskRunner).to receive(:run).once
-      task.invoke
-    end
-  end
-end