solidus_core 2.9.6 → 2.11.0

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

data/spec/lib/tasks/migrations/migrate_shipping_rate_taxes_spec.rb

@@ -1,21 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe 'solidus:migrations:migrate_shipping_rate_taxes' do
-  describe 'up' do
-    include_context(
-      'rake',
-      task_path: Spree::Core::Engine.root.join('lib/tasks/migrations/migrate_shipping_rate_taxes.rake'),
-      task_name: 'solidus:migrations:migrate_shipping_rate_taxes:up',
-    )
-
-    it 'runs' do
-      Spree::Deprecation.silence do
-        expect { task.invoke }.to output(
-          "Adding persisted tax notes to historic shipping rates ... Success.\n"
-        ).to_stdout
-      end
-    end
-  end
-end

data/spec/mailers/carton_mailer_spec.rb

@@ -1,44 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::CartonMailer do
-  let(:carton) { create(:carton) }
-  let(:order) { carton.orders.first }
-
-  # Regression test for https://github.com/spree/spree/issues/2196
-  it "doesn't include out of stock in the email body" do
-    shipment_email = Spree::CartonMailer.shipped_email(order: order, carton: carton)
-    expect(shipment_email.parts.first.body).not_to include(%{Out of Stock})
-    expect(shipment_email.parts.first.body).to include(%{Your order has been shipped})
-    expect(shipment_email.subject).to eq "#{order.store.name} Shipment Notification ##{order.number}"
-  end
-
-  context "with resend option" do
-    subject do
-      Spree::CartonMailer.shipped_email(order: order, carton: carton, resend: true).subject
-    end
-    it { is_expected.to match /^\[RESEND\] / }
-  end
-
-  context "emails must be translatable" do
-    context "shipped_email" do
-      context "pt-BR locale" do
-        before do
-          pt_br_shipped_email = { spree: { shipment_mailer: { shipped_email: { dear_customer: 'Caro Cliente,' } } } }
-          I18n.backend.store_translations :'pt-BR', pt_br_shipped_email
-          I18n.locale = :'pt-BR'
-        end
-
-        after do
-          I18n.locale = I18n.default_locale
-        end
-
-        specify do
-          shipped_email = Spree::CartonMailer.shipped_email(order: order, carton: carton)
-          expect(shipped_email.parts.first.body).to include("Caro Cliente,")
-        end
-      end
-    end
-  end
-end

data/spec/mailers/order_mailer_spec.rb

@@ -1,117 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::OrderMailer, type: :mailer do
-  let(:order) do
-    order = create(:order)
-    product = stub_model(Spree::Product, name: %{The "BEST" product})
-    variant = stub_model(Spree::Variant, product: product)
-    price = stub_model(Spree::Price, variant: variant, amount: 5.00)
-    store = FactoryBot.build :store, mail_from_address: "store@example.com"
-    line_item = stub_model(Spree::LineItem, variant: variant, order: order, quantity: 1, price: 4.99)
-    allow(variant).to receive_messages(default_price: price)
-    allow(order).to receive_messages(line_items: [line_item])
-    allow(order).to receive(:store).and_return(store)
-    order
-  end
-
-  it "uses the order's store for the from address" do
-    message = Spree::OrderMailer.confirm_email(order)
-    expect(message.from).to eq ["store@example.com"]
-  end
-
-  it "doesn't aggressively escape double quotes in confirmation body" do
-    confirmation_email = Spree::OrderMailer.confirm_email(order)
-    expect(confirmation_email.body).not_to include(""")
-  end
-
-  context "only shows eligible adjustments in emails" do
-    before do
-      create(
-        :adjustment,
-        adjustable: order,
-        order:      order,
-        eligible:   true,
-        label:      'Eligible Adjustment'
-      )
-
-      create(
-        :adjustment,
-        adjustable: order,
-        order:      order,
-        eligible:   false,
-        label:      'Ineligible Adjustment'
-      )
-    end
-
-    let!(:confirmation_email) { Spree::OrderMailer.confirm_email(order) }
-    let!(:cancel_email) { Spree::OrderMailer.cancel_email(order) }
-
-    specify do
-      expect(confirmation_email.body).not_to include("Ineligible Adjustment")
-    end
-
-    specify do
-      expect(cancel_email.body).not_to include("Ineligible Adjustment")
-    end
-  end
-
-  context "displays unit costs from line item" do
-    # Regression test for https://github.com/spree/spree/issues/2772
-
-    # Tests mailer view spree/order_mailer/confirm_email.text.erb
-    specify do
-      confirmation_email = Spree::OrderMailer.confirm_email(order)
-      expect(confirmation_email.parts.first.body).to include("4.99")
-      expect(confirmation_email.parts.first.body).to_not include("5.00")
-    end
-
-    # Tests mailer view spree/order_mailer/cancel_email.text.erb
-    specify do
-      cancel_email = Spree::OrderMailer.cancel_email(order)
-      expect(cancel_email.parts.first.body).to include("4.99")
-      expect(cancel_email.parts.first.body).to_not include("5.00")
-    end
-  end
-
-  context "emails must be translatable" do
-    context "pt-BR locale" do
-      before do
-        I18n.enforce_available_locales = false
-        pt_br_confirm_mail = { spree: { order_mailer: { confirm_email: { dear_customer: 'Caro Cliente,' } } } }
-        pt_br_cancel_mail = { spree: { order_mailer: { cancel_email: { order_summary_canceled: 'Resumo da Pedido [CANCELADA]' } } } }
-        I18n.backend.store_translations :'pt-BR', pt_br_confirm_mail
-        I18n.backend.store_translations :'pt-BR', pt_br_cancel_mail
-        I18n.locale = :'pt-BR'
-      end
-
-      after do
-        I18n.locale = I18n.default_locale
-        I18n.enforce_available_locales = true
-      end
-
-      context "confirm_email" do
-        specify do
-          confirmation_email = Spree::OrderMailer.confirm_email(order)
-          expect(confirmation_email.parts.first.body).to include("Caro Cliente,")
-        end
-      end
-
-      context "cancel_email" do
-        specify do
-          cancel_email = Spree::OrderMailer.cancel_email(order)
-          expect(cancel_email.parts.first.body).to include("Resumo da Pedido [CANCELADA]")
-        end
-      end
-    end
-  end
-
-  context "with preference :send_core_emails set to false" do
-    it "sends no email" do
-      stub_spree_preferences(send_core_emails: false)
-      message = Spree::OrderMailer.confirm_email(order)
-      expect(message.body).to be_blank
-    end
-  end
-end

data/spec/mailers/promotion_code_batch_mailer_spec.rb

@@ -1,47 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-RSpec.describe Spree::PromotionCodeBatchMailer, type: :mailer do
-  let(:promotion) { create(:promotion, name: "Promotion Test") }
-  let(:promotion_code_batch) do
-    Spree::PromotionCodeBatch.create!(
-      promotion_id: promotion.id,
-      base_code: "test",
-      number_of_codes: 10,
-      email: "test@email.com"
-    )
-  end
-
-  describe "#promotion_code_batch_finished" do
-    subject { described_class.promotion_code_batch_finished(promotion_code_batch) }
-
-    it "sends the email to the email attached to the promotion code batch " do
-      expect(subject.to).to eq([promotion_code_batch.email])
-    end
-
-    it "contains the number of codes created" do
-      expect(subject.body).to include("All 10 codes have been created")
-    end
-
-    it "contains the name of the promotion" do
-      expect(subject.body).to include(promotion.name)
-    end
-  end
-
-  describe "#promotion_code_batch_errored" do
-    before { promotion_code_batch.update(error: "Test error") }
-    subject { described_class.promotion_code_batch_errored(promotion_code_batch) }
-
-    it "sends the email to the email attached to the promotion code batch " do
-      expect(subject.to).to eq([promotion_code_batch.email])
-    end
-
-    it "contains the error" do
-      expect(subject.body).to include("Test error")
-    end
-
-    it "contains the name of the promotion" do
-      expect(subject.body).to include(promotion.name)
-    end
-  end
-end

data/spec/mailers/reimbursement_mailer_spec.rb

@@ -1,36 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::ReimbursementMailer, type: :mailer do
-  let(:reimbursement) { create(:reimbursement) }
-
-  it "accepts a reimbursement id as an alternative to a Reimbursement object" do
-    expect(Spree::Reimbursement).to receive(:find).with(reimbursement.id).and_return(reimbursement)
-
-    Spree::ReimbursementMailer.reimbursement_email(reimbursement.id).body
-  end
-
-  context "emails must be translatable" do
-    context "reimbursement_email" do
-      context "pt-BR locale" do
-        before do
-          I18n.enforce_available_locales = false
-          pt_br_shipped_email = { spree: { reimbursement_mailer: { reimbursement_email: { dear_customer: 'Caro Cliente,' } } } }
-          I18n.backend.store_translations :'pt-BR', pt_br_shipped_email
-          I18n.locale = :'pt-BR'
-        end
-
-        after do
-          I18n.locale = I18n.default_locale
-          I18n.enforce_available_locales = true
-        end
-
-        specify do
-          reimbursement_email = Spree::ReimbursementMailer.reimbursement_email(reimbursement)
-          expect(reimbursement_email.parts.first.body).to include("Caro Cliente,")
-        end
-      end
-    end
-  end
-end

data/spec/mailers/test_mailer_spec.rb

@@ -1,13 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-
-RSpec.describe Spree::TestMailer, type: :mailer do
-  let(:user) { create(:user) }
-
-  it "confirm_email accepts a user id as an alternative to a User object" do
-    Spree::Deprecation.silence do
-      Spree::TestMailer.test_email('test@example.com')
-    end
-  end
-end

data/spec/migrate/20190106184413_remove_code_from_spree_promotions_spec.rb

@@ -1,148 +0,0 @@
-# frozen_string_literal: true
-
-require 'rails_helper'
-require Spree::Core::Engine.root.join('db/migrate/20190106184413_remove_code_from_spree_promotions.rb')
-
-RSpec.describe RemoveCodeFromSpreePromotions do
-  let(:migrations_paths) { ActiveRecord::Migrator.migrations_paths }
-  let(:migrations) do
-    if ActiveRecord::Base.connection.respond_to?(:migration_context)
-      # Rails >= 5.2
-      ActiveRecord::MigrationContext.new(migrations_paths).migrations
-    else
-      ActiveRecord::Migrator.migrations(migrations_paths)
-    end
-  end
-  let(:previous_version) { 20180710170104 }
-  let(:current_version) { 20190106184413 }
-
-  subject { ActiveRecord::Migrator.new(:up, migrations, current_version).migrate }
-
-  # This is needed for MySQL since it is not able to rollback to the previous
-  # state when database schema changes within that transaction.
-  before(:all) { self.use_transactional_tests = false }
-  after(:all)  { self.use_transactional_tests = true }
-
-  around do |example|
-    DatabaseCleaner.clean_with(:truncation)
-    # Silence migrations output in specs report.
-    ActiveRecord::Migration.suppress_messages do
-      # Migrate back to the previous version
-      ActiveRecord::Migrator.new(:down, migrations, previous_version).migrate
-      # If other tests using Spree::Promotion ran before this one, Rails has
-      # stored information about table's columns and we need to reset those
-      # since the migration changed the database structure.
-      Spree::Promotion.reset_column_information
-
-      example.run
-
-      # Re-update column information after the migration has been executed
-      # again in the example. This will make the promotion attributes cache
-      # ready for other tests.
-      Spree::Promotion.reset_column_information
-    end
-    DatabaseCleaner.clean_with(:truncation)
-  end
-
-  let(:promotion_with_code) { create(:promotion) }
-
-  before do
-    # We can't set code via factory since `code=` is currently raising
-    # an error, see more at:
-    # https://github.com/solidusio/solidus/blob/cf96b03eb9e80002b69736e082fd485c870ab5d9/core/app/models/spree/promotion.rb#L65
-    promotion_with_code.update_column(:code, code)
-  end
-
-  context 'when there are no promotions with code' do
-    let(:code) { '' }
-
-    it 'does not call any promotion with code handler' do
-      expect(described_class).not_to receive(:promotions_with_code_handler)
-
-      subject
-    end
-  end
-
-  context 'when there are promotions with code' do
-    let(:code) { 'Just An Old Promo Code' }
-
-    context 'with the deafult handler (Solidus::Migrations::PromotionWithCodeHandlers::RaiseException)' do
-      it 'raise a StandardError exception' do
-        expect { subject }.to raise_error(StandardError)
-      end
-    end
-
-    context 'changing the default handler' do
-      before do
-        allow(described_class)
-          .to receive(:promotions_with_code_handler)
-          .and_return(promotions_with_code_handler)
-      end
-
-      context 'to Solidus::Migrations::PromotionWithCodeHandlers::MoveToSpreePromotionCode' do
-        let(:promotions_with_code_handler) { Solidus::Migrations::PromotionWithCodeHandlers::MoveToSpreePromotionCode }
-
-        context 'when there are no Spree::PromotionCode with the same value' do
-          it 'moves the code into a Spree::PromotionCode' do
-            migration_context = double('a migration context')
-            allow_any_instance_of(promotions_with_code_handler)
-              .to receive(:migration_context)
-              .and_return(migration_context)
-
-            expect(migration_context)
-              .to receive(:say)
-              .with("Creating Spree::PromotionCode with value 'just an old promo code' for Spree::Promotion with id '#{promotion_with_code.id}'")
-
-            expect { subject }
-              .to change { Spree::PromotionCode.all.size }
-              .from(0)
-              .to(1)
-          end
-        end
-
-        context 'with promotions with type set (legacy feature)' do
-          let(:promotion_with_code) { create(:promotion, type: 'Spree::Promotion') }
-
-          it 'does not raise a STI error' do
-            expect { subject }.not_to raise_error
-          end
-        end
-
-        context 'when there is a Spree::PromotionCode with the same value' do
-          context 'associated with the same promotion' do
-            let!(:existing_promotion_code) { create(:promotion_code, value: 'just an old promo code', promotion: promotion_with_code) }
-
-            it 'does not create a new Spree::PromotionCode' do
-              expect { subject }.not_to change { Spree::PromotionCode.all.size }
-            end
-          end
-
-          context 'associated with another promotion' do
-            let!(:existing_promotion_code) { create(:promotion_code, value: 'just an old promo code') }
-
-            it 'raises an exception' do
-              expect { subject }.to raise_error(StandardError)
-            end
-          end
-        end
-      end
-
-      context 'to Solidus::Migrations::PromotionWithCodeHandlers::DoNothing' do
-        let(:promotions_with_code_handler) { Solidus::Migrations::PromotionWithCodeHandlers::DoNothing }
-
-        it 'just prints a message' do
-          migration_context = double('a migration context')
-          allow_any_instance_of(promotions_with_code_handler)
-            .to receive(:migration_context)
-            .and_return(migration_context)
-
-          expect(migration_context)
-            .to receive(:say)
-            .with("Code 'Just An Old Promo Code' is going to be removed from Spree::Promotion with id '#{promotion_with_code.id}'")
-
-          subject
-        end
-      end
-    end
-  end
-end