solidus_core 4.6.2 → 4.7.0

data/lib/spree/testing_support/common_rake.rb

@@ -1,10 +1,11 @@
 # frozen_string_literal: true
 
-require 'generators/spree/dummy/dummy_generator'
-require 'logger'
+require "generators/spree/dummy/dummy_generator"
+require "logger"
 
 class CommonRakeTasks
   include Rake::DSL
+
   attr_reader :logger
 
   def initialize
@@ -12,7 +13,7 @@ class CommonRakeTasks
     namespace :common do
       task :test_app, :user_class do |_t, args|
         args.with_defaults(user_class: "Spree::LegacyUser")
-        lib_name = ENV['LIB_NAME'] or
+        lib_name = ENV["LIB_NAME"] or
           raise "Please provide a library name via the LIB_NAME environment variable."
 
         require lib_name
@@ -21,30 +22,30 @@ class CommonRakeTasks
 
         Spree::DummyGenerator.start [
           "--lib-name=#{lib_name}",
-          "--quiet",
+          "--quiet"
         ]
 
         # While the dummy app is generated the current directory
         # within ruby is changed to that of the dummy app.
         sh({
-          'FRONTEND' => ENV['FRONTEND'] || 'none',
+          "FRONTEND" => ENV["FRONTEND"] || "none"
         }, [
-          'bin/rails',
-          'generate',
-          'solidus:install',
+          "bin/rails",
+          "generate",
+          "solidus:install",
           Dir.pwd, # use the current dir as Rails.root
           "--auto-accept",
-          "--admin-preview=#{ENV.fetch('ADMIN_PREVIEW', 'false')}",
+          "--admin-preview=#{ENV.fetch("ADMIN_PREVIEW", "false")}",
           "--authentication=none",
           "--payment-method=none",
           "--migrate=false",
           "--seed=false",
           "--sample=false",
           "--user-class=#{args[:user_class]}",
-          "--quiet",
+          "--quiet"
         ].shelljoin)
 
-        if Bundler.locked_gems.dependencies['solidus_frontend']
+        if Bundler.locked_gems.dependencies["solidus_frontend"]
           sh "bin/rails g solidus_frontend:install --auto-accept"
         end
 
@@ -54,7 +55,7 @@ class CommonRakeTasks
         sh "bin/rails db:drop db:create db:migrate VERBOSE=false RAILS_ENV=test"
 
         if extension_installation_generator_exists?
-          logger.info 'Running extension installation generator...'
+          logger.info "Running extension installation generator..."
           sh "bin/rails generate #{rake_generator_namespace}:install --auto-run-migrations"
         end
       end
@@ -70,8 +71,8 @@ class CommonRakeTasks
   private
 
   def force_rails_environment_to_test
-    ENV["RAILS_ENV"] = 'test'
-    Rails.env = 'test'
+    ENV["RAILS_ENV"] = "test"
+    Rails.env = "test"
   end
 
   def extension_installation_generator_exists?
@@ -83,11 +84,11 @@ class CommonRakeTasks
   end
 
   def generator_namespace
-    "#{ENV['LIB_NAMESPACE']&.underscore || ENV['LIB_NAME']}"
+    (ENV["LIB_NAMESPACE"]&.underscore || ENV["LIB_NAME"]).to_s
   end
 
   def rake_generator_namespace
-    generator_namespace.gsub("/", ":")
+    generator_namespace.tr("/", ":")
   end
 end
 

data/lib/spree/testing_support/dummy_ability.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'cancan'
+require "cancan"
 
 class DummyAbility
   include CanCan::Ability

data/lib/spree/testing_support/dummy_app/migrations.rb

@@ -11,7 +11,7 @@ module DummyApp
         # Disconnect to avoid "database is being accessed by other users" on postgres
         ActiveRecord::Base.remove_connection
 
-        sh 'rake db:reset VERBOSE=false'
+        sh "rake db:reset VERBOSE=false"
 
         # We have a brand new database, so we must re-establish our connection
         ActiveRecord::Base.establish_connection

data/lib/spree/testing_support/dummy_app/rake_tasks.rb

@@ -6,9 +6,9 @@ module DummyApp
 
     def initialize(gem_root:, lib_name:)
       task :dummy_environment do
-        ENV['RAILS_ENV'] = 'test'
+        ENV["RAILS_ENV"] = "test"
         require lib_name
-        require 'spree/testing_support/dummy_app'
+        require "spree/testing_support/dummy_app"
         DummyApp.setup(
           gem_root:,
           lib_name:,
@@ -45,12 +45,12 @@ namespace :db do
   end
 
   desc "Recreates the test database and re-runs all migrations"
-  task reset: ['db:drop', 'db:create', 'db:migrate']
+  task reset: ["db:drop", "db:create", "db:migrate"]
 end
 
 desc "Open a sandboxed console in the test environment"
 task console: :dummy_environment do
-  require 'rails/commands'
-  require 'rails/commands/console/console_command'
+  require "rails/commands"
+  require "rails/commands/console/console_command"
   Rails::Console.new(Rails.application, sandbox: true, environment: "test").start
 end

data/lib/spree/testing_support/dummy_app/routes.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 DummyApp::Application.routes.draw do
-  mount Spree::Core::Engine, at: '/'
+  mount Spree::Core::Engine, at: "/"
 end

data/lib/spree/testing_support/dummy_app.rb

@@ -1,17 +1,17 @@
 # frozen_string_literal: true
 
-ENV['RAILS_ENV'] = 'test'
-ENV['DISABLE_DATABASE_ENVIRONMENT_CHECK'] = '1'
+ENV["RAILS_ENV"] = "test"
+ENV["DISABLE_DATABASE_ENVIRONMENT_CHECK"] = "1"
 
-require 'rails'
-require 'active_record/railtie'
-require 'action_controller/railtie'
-require 'action_mailer/railtie'
-require 'active_storage/engine'
+require "rails"
+require "active_record/railtie"
+require "action_controller/railtie"
+require "action_mailer/railtie"
+require "active_storage/engine"
 
-Rails.env = 'test'
+Rails.env = "test"
 
-require 'solidus_core'
+require "solidus_core"
 
 # @private
 class ApplicationController < ActionController::Base
@@ -39,7 +39,7 @@ end
 module DummyApp
   def self.setup(gem_root:, lib_name:, auto_migrate: true)
     ENV["LIB_NAME"] = lib_name
-    root = Pathname(gem_root).join('spec/dummy')
+    root = Pathname(gem_root).join("spec/dummy")
     root.join("app/assets/config").mkpath
     root.join("app/assets/config/manifest.js").write("// Intentionally empty\n")
 
@@ -48,7 +48,7 @@ module DummyApp
 
     # Raise on deprecation warnings.
     # NOTE: This needs to happen after the application is initialized.
-    if ENV['SOLIDUS_RAISE_DEPRECATIONS'].present?
+    if ENV["SOLIDUS_RAISE_DEPRECATIONS"].present?
       Spree.deprecator.behavior = :raise
     end
 
@@ -77,19 +77,19 @@ module DummyApp
     config.cache_classes = true
 
     # Make debugging easier:
-    if Rails.gem_version >= Gem::Version.new('7.1')
+    if Rails.gem_version >= Gem::Version.new("7.1")
       config.action_controller.raise_on_missing_callback_actions = true
       config.action_dispatch.show_exceptions = :none
     else
       config.action_dispatch.show_exceptions = false
     end
     config.consider_all_requests_local = true
-    config.active_support.deprecation = ENV['RAILS_DEPRECATIONS_BEHAVIOR'].presence&.to_sym || :stderr
+    config.active_support.deprecation = ENV["RAILS_DEPRECATIONS_BEHAVIOR"].presence&.to_sym || :stderr
     config.log_level = :debug
 
     # Improve test suite performance:
     config.eager_load = false
-    config.public_file_server.headers = { 'Cache-Control' => 'public, max-age=3600' }
+    config.public_file_server.headers = {"Cache-Control" => "public, max-age=3600"}
     config.cache_store = :memory_store
 
     # We don't use a web server, so we let Rails serve assets.
@@ -102,66 +102,66 @@ module DummyApp
     config.active_job.queue_adapter = :test
 
     # No need to use credentials file in a test environment.
-    config.secret_key_base = 'SECRET_TOKEN'
+    config.secret_key_base = "SECRET_TOKEN"
 
     # Set the preview path within the dummy app:
     if ActionMailer::Base.respond_to? :preview_paths # Rails 7.1+
-      config.action_mailer.preview_paths << File.expand_path('dummy_app/mailer_previews', __dir__)
+      config.action_mailer.preview_paths << File.expand_path("dummy_app/mailer_previews", __dir__)
     else
-      config.action_mailer.preview_path = File.expand_path('dummy_app/mailer_previews', __dir__)
+      config.action_mailer.preview_path = File.expand_path("dummy_app/mailer_previews", __dir__)
     end
 
     config.active_record.dump_schema_after_migration = false
 
     # Configure active storage to use storage within tmp folder
-    initializer 'solidus.active_storage' do
+    initializer "solidus.active_storage" do
       config.active_storage.service_configurations = {
         test: {
-          service: 'Disk',
-          root: Rails.root.join('tmp', 'storage')
+          service: "Disk",
+          root: Rails.root.join("tmp", "storage")
         }
       }
       config.active_storage.service = :test
-      config.active_storage.variant_processor = ENV.fetch('ACTIVE_STORAGE_VARIANT_PROCESSOR', :vips).to_sym
+      config.active_storage.variant_processor = ENV.fetch("ACTIVE_STORAGE_VARIANT_PROCESSOR", :vips).to_sym
     end
 
     # Avoid issues if an old spec/dummy still exists
-    config.paths['config/initializers'] = []
-    config.paths['config/environments'] = []
+    config.paths["config/initializers"] = []
+    config.paths["config/environments"] = []
 
     migration_dirs = Rails.application.migration_railties.flat_map do |engine|
       if engine.respond_to?(:paths)
-        engine.paths['db/migrate'].to_a
+        engine.paths["db/migrate"].to_a
       else
         []
       end
     end
-    config.paths['db/migrate'] = migration_dirs
+    config.paths["db/migrate"] = migration_dirs
     ActiveRecord::Migrator.migrations_paths = migration_dirs
     ActiveRecord::Migration.verbose = false
 
-    config.assets.paths << File.expand_path('dummy_app/assets/javascripts', __dir__)
-    config.assets.paths << File.expand_path('dummy_app/assets/stylesheets', __dir__)
+    config.assets.paths << File.expand_path("dummy_app/assets/javascripts", __dir__)
+    config.assets.paths << File.expand_path("dummy_app/assets/stylesheets", __dir__)
     config.assets.css_compressor = nil
 
-    config.paths["config/database"] = File.expand_path('dummy_app/database.yml', __dir__)
-    config.paths['config/routes.rb'] = File.expand_path('dummy_app/routes.rb', __dir__)
+    config.paths["config/database"] = File.expand_path("dummy_app/database.yml", __dir__)
+    config.paths["config/routes.rb"] = File.expand_path("dummy_app/routes.rb", __dir__)
 
     ActionMailer::Base.default from: "store@example.com"
   end
 end
 
-require 'spree/testing_support/dummy_app/migrations'
+require "spree/testing_support/dummy_app/migrations"
 
 ActiveSupport.on_load(:action_controller) do
   wrap_parameters format: [:json]
 end
 
-Spree.user_class = 'Spree::LegacyUser'
+Spree.user_class = "Spree::LegacyUser"
 Spree.load_defaults(Spree.solidus_version)
 Spree.config do |config|
-  if (ENV['DISABLE_ACTIVE_STORAGE'] == 'true')
-    config.image_attachment_module = 'Spree::Image::PaperclipAttachment'
-    config.taxon_attachment_module = 'Spree::Taxon::PaperclipAttachment'
+  if ENV["DISABLE_ACTIVE_STORAGE"] == "true"
+    config.image_attachment_module = "Spree::Image::PaperclipAttachment"
+    config.taxon_attachment_module = "Spree::Taxon::PaperclipAttachment"
   end
 end

data/lib/spree/testing_support/extension_rake.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
-require 'spree/testing_support/common_rake'
+require "spree/testing_support/common_rake"
 
 desc "Generates a dummy app for testing an extension"
 namespace :extension do
   task :test_app, [:user_class] do |_t, args|
     Spree::DummyGeneratorHelper.inject_extension_requirements = true
-    Rake::Task['common:test_app'].invoke(args[:user_class])
+    Rake::Task["common:test_app"].invoke(args[:user_class])
   end
 end

data/lib/spree/testing_support/factories/address_factory.rb

@@ -1,21 +1,21 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :address, class: 'Spree::Address' do
+  factory :address, class: "Spree::Address" do
     transient do
       # There's `Spree::Address#country_iso=`, prohibiting me from using `country_iso` here
-      country_iso_code { 'US' }
-      state_code { 'AL' }
+      country_iso_code { "US" }
+      state_code { "AL" }
     end
 
-    name { 'John Von Doe' }
-    company { 'Company' }
-    address1 { '10 Lovely Street' }
-    address2 { 'Northwest' }
-    city { 'Herndon' }
+    name { "John Von Doe" }
+    company { "Company" }
+    address1 { "10 Lovely Street" }
+    address2 { "Northwest" }
+    city { "Herndon" }
     sequence(:zipcode, 10001) { |i| i.to_s }
-    phone { '555-555-0199' }
-    alternative_phone { '555-555-0199' }
+    phone { "555-555-0199" }
+    alternative_phone { "555-555-0199" }
 
     country do |address|
       if address.state
@@ -35,10 +35,10 @@ FactoryBot.define do
   end
 
   factory :ship_address, parent: :address do
-    address1 { 'A Different Road' }
+    address1 { "A Different Road" }
   end
 
   factory :bill_address, parent: :address do
-    address1 { 'PO Box 1337' }
+    address1 { "PO Box 1337" }
   end
 end

data/lib/spree/testing_support/factories/adjustment_factory.rb

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :adjustment, class: 'Spree::Adjustment' do
+  factory :adjustment, class: "Spree::Adjustment" do
     order
     adjustable { order }
     amount { 100.0 }
-    label { 'Shipping' }
+    label { "Shipping" }
     association(:source, factory: :tax_rate)
 
     after(:build) do |adjustment|
@@ -15,19 +15,19 @@ FactoryBot.define do
       end
     end
 
-    factory :tax_adjustment, class: 'Spree::Adjustment' do
+    factory :tax_adjustment, class: "Spree::Adjustment" do
       order { adjustable.order }
       association(:adjustable, factory: :line_item)
       amount { 10.0 }
-      label { 'VAT 5%' }
+      label { "VAT 5%" }
 
       after(:create) do |adjustment|
         # Set correct tax category, so that adjustment amount is not 0
         if adjustment.adjustable.is_a?(Spree::LineItem)
-          if adjustment.adjustable.tax_category.present?
-            adjustment.source.tax_categories = [adjustment.adjustable.tax_category]
+          adjustment.source.tax_categories = if adjustment.adjustable.tax_category.present?
+            [adjustment.adjustable.tax_category]
           else
-            adjustment.source.tax_categories = []
+            []
           end
           adjustment.source.save
           adjustment.update!(amount: adjustment.source.compute_amount(adjustment.adjustable))

data/lib/spree/testing_support/factories/adjustment_reason_factory.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :adjustment_reason, class: 'Spree::AdjustmentReason' do
+  factory :adjustment_reason, class: "Spree::AdjustmentReason" do
     sequence(:name) { |n| "Refund for return ##{n}" }
     sequence(:code) { |n| "Code #{n}" }
   end

data/lib/spree/testing_support/factories/calculator_factory.rb

@@ -1,25 +1,25 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :flat_rate_calculator, aliases: [:calculator], class: 'Spree::Calculator::FlatRate' do
+  factory :flat_rate_calculator, aliases: [:calculator], class: "Spree::Calculator::FlatRate" do
     preferred_amount { 10.0 }
   end
 
-  factory :no_amount_calculator, class: 'Spree::Calculator::FlatRate' do
+  factory :no_amount_calculator, class: "Spree::Calculator::FlatRate" do
     preferred_amount { 0 }
   end
 
-  factory :default_tax_calculator, class: 'Spree::Calculator::DefaultTax' do
+  factory :default_tax_calculator, class: "Spree::Calculator::DefaultTax" do
   end
 
-  factory :flat_fee_calculator, class: 'Spree::Calculator::FlatFee' do
+  factory :flat_fee_calculator, class: "Spree::Calculator::FlatFee" do
   end
 
-  factory :shipping_calculator, class: 'Spree::Calculator::Shipping::FlatRate' do
+  factory :shipping_calculator, class: "Spree::Calculator::Shipping::FlatRate" do
     preferred_amount { 10.0 }
   end
 
-  factory :shipping_no_amount_calculator, class: 'Spree::Calculator::Shipping::FlatRate' do
+  factory :shipping_no_amount_calculator, class: "Spree::Calculator::Shipping::FlatRate" do
     preferred_amount { 0 }
   end
 end

data/lib/spree/testing_support/factories/carton_factory.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :carton, class: 'Spree::Carton' do
+  factory :carton, class: "Spree::Carton" do
     address
     stock_location
     shipping_method

data/lib/spree/testing_support/factories/country_factory.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
-require 'carmen'
+require "carmen"
 
 FactoryBot.define do
-  factory :country, class: 'Spree::Country' do
-    iso { 'US' }
+  factory :country, class: "Spree::Country" do
+    iso { "US" }
 
     transient do
       carmen_country { Carmen::Country.coded(iso) || fail("Unknown country iso code: #{iso.inspect}") }

data/lib/spree/testing_support/factories/credit_card_factory.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :credit_card, class: 'Spree::CreditCard' do
+  factory :credit_card, class: "Spree::CreditCard" do
     verification_value { 123 }
     month { 12 }
     year { 1.year.from_now.year }
-    number { '4111111111111111' }
-    name { 'Spree Commerce' }
+    number { "4111111111111111" }
+    name { "Spree Commerce" }
     association(:payment_method, factory: :credit_card_payment_method)
     association(:address)
 

data/lib/spree/testing_support/factories/customer_return_factory.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :customer_return, class: 'Spree::CustomerReturn' do
+  factory :customer_return, class: "Spree::CustomerReturn" do
     association(:stock_location, factory: :stock_location)
 
     transient do
@@ -25,7 +25,7 @@ FactoryBot.define do
   end
 
   # for the case when you want to supply existing return items instead of generating some
-  factory :customer_return_without_return_items, class: 'Spree::CustomerReturn' do
+  factory :customer_return_without_return_items, class: "Spree::CustomerReturn" do
     association(:stock_location, factory: :stock_location)
   end
 end

data/lib/spree/testing_support/factories/image_factory.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :image, class: 'Spree::Image' do
-    attachment { Spree::Core::Engine.root.join('lib', 'spree', 'testing_support', 'fixtures', 'blank.jpg').open }
+  factory :image, class: "Spree::Image" do
+    attachment { Spree::Core::Engine.root.join("lib", "spree", "testing_support", "fixtures", "blank.jpg").open }
   end
 end

data/lib/spree/testing_support/factories/inventory_unit_factory.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :inventory_unit, class: 'Spree::InventoryUnit' do
+  factory :inventory_unit, class: "Spree::InventoryUnit" do
     transient do
       order { nil }
       stock_location { nil }
@@ -15,12 +15,12 @@ FactoryBot.define do
         build(:line_item, variant:)
       end
     end
-    state { 'on_hand' }
+    state { "on_hand" }
     shipment do
       if stock_location
-        build(:shipment, state: 'pending', order: line_item.order, stock_location:)
+        build(:shipment, state: "pending", order: line_item.order, stock_location:)
       else
-        build(:shipment, state: 'pending', order: line_item.order)
+        build(:shipment, state: "pending", order: line_item.order)
       end
     end
     # return_authorization

data/lib/spree/testing_support/factories/line_item_factory.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :line_item, class: 'Spree::LineItem' do
+  factory :line_item, class: "Spree::LineItem" do
     quantity { 1 }
-    price { BigDecimal('10.00') }
+    price { BigDecimal("10.00") }
     order
     transient do
       product { nil }

data/lib/spree/testing_support/factories/option_type_factory.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :option_type, class: 'Spree::OptionType' do
+  factory :option_type, class: "Spree::OptionType" do
     sequence(:name) { |n| "foo-size-#{n}" }
-    presentation { 'Size' }
+    presentation { "Size" }
   end
 end

data/lib/spree/testing_support/factories/option_value_factory.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :option_value, class: 'Spree::OptionValue' do
+  factory :option_value, class: "Spree::OptionValue" do
     sequence(:name) { |n| "Size-#{n}" }
 
-    presentation { 'S' }
+    presentation { "S" }
     option_type
   end
 end