spree_core 4.3.0 → 4.5.1

data/lib/spree/core/version.rb

@@ -1,5 +1,5 @@
 module Spree
-  VERSION = '4.3.0'.freeze
+  VERSION = '4.5.1'.freeze
 
   def self.version
     VERSION

data/lib/spree/core.rb

@@ -14,17 +14,18 @@ require 'friendly_id'
 require 'kaminari'
 require 'monetize'
 require 'paranoia'
-require 'mini_magick'
 require 'ransack'
 require 'state_machines-activerecord'
 require 'active_storage_validations'
+require 'activerecord-typedstore'
 
 # This is required because ActiveModel::Validations#invalid? conflicts with the
 # invalid state of a Payment. In the future this should be removed.
 StateMachines::Machine.ignore_method_conflicts = true
 
 module Spree
-  mattr_accessor :user_class
+  mattr_accessor :user_class, :admin_user_class, :private_storage_service_name,
+                 :public_storage_service_name, :cdn_host, :searcher_class
 
   def self.user_class(constantize: true)
     if @@user_class.is_a?(Class)
@@ -34,19 +35,44 @@ module Spree
     end
   end
 
-  def self.admin_path
-    Spree::Config[:admin_path]
+  def self.admin_user_class(constantize: true)
+    @@admin_user_class ||= @@user_class
+
+    if @@admin_user_class.is_a?(Class)
+      raise 'Spree.admin_user_class MUST be a String or Symbol object, not a Class object.'
+    elsif @@admin_user_class.is_a?(String) || @@admin_user_class.is_a?(Symbol)
+      constantize ? @@admin_user_class.to_s.constantize : @@admin_user_class.to_s
+    end
   end
 
-  # Used to configure admin_path for Spree
-  #
-  # Example:
-  #
-  # write the following line in `config/initializers/spree.rb`
-  #   Spree.admin_path = '/custom-path'
+  def self.private_storage_service_name
+    if @@private_storage_service_name
+      if @@private_storage_service_name.is_a?(String) || @@private_storage_service_name.is_a?(Symbol)
+        @@private_storage_service_name.to_sym
+      else
+        raise 'Spree.private_storage_service_name MUST be a String or Symbol object.'
+      end
+    end
+  end
 
-  def self.admin_path=(path)
-    Spree::Config[:admin_path] = path
+  def self.public_storage_service_name
+    if @@public_storage_service_name
+      if @@public_storage_service_name.is_a?(String) || @@public_storage_service_name.is_a?(Symbol)
+        @@public_storage_service_name.to_sym
+      else
+        raise 'Spree.public_storage_service_name MUST be a String or Symbol object.'
+      end
+    end
+  end
+
+  def self.searcher_class(constantize: true)
+    @@searcher_class ||= 'Spree::Core::Search::Base'
+
+    if @@searcher_class.is_a?(Class)
+      raise 'Spree.searcher_class MUST be a String or Symbol object, not a Class object.'
+    elsif @@searcher_class.is_a?(String) || @@searcher_class.is_a?(Symbol)
+      constantize ? @@searcher_class.to_s.constantize : @@searcher_class.to_s
+    end
   end
 
   # Used to configure Spree.
@@ -97,7 +123,6 @@ require 'spree/localized_number'
 require 'spree/money'
 require 'spree/permitted_attributes'
 require 'spree/service_module'
-require 'spree/dependencies_helper'
 require 'spree/database_type_utilities'
 
 require 'spree/core/importer'
@@ -111,3 +136,6 @@ require 'spree/core/controller_helpers/store'
 require 'spree/core/controller_helpers/strong_parameters'
 require 'spree/core/controller_helpers/locale'
 require 'spree/core/controller_helpers/currency'
+
+require 'spree/core/preferences/store'
+require 'spree/core/preferences/scoped_store'

data/lib/spree/money.rb

@@ -21,7 +21,7 @@ module Spree
 
     def initialize(amount, options = {})
       use_default_currency
-      @money   = Monetize.parse([amount, (options[:currency] || Spree::Config[:currency])].join)
+      @money   = Monetize.parse([amount, (options[:currency] || Spree::Store.default.default_currency)].join)
       @options = Spree::Money.default_formatting_rules.merge(options)
     end
 
@@ -66,7 +66,7 @@ module Spree
     end
 
     def use_default_currency
-      currency = Spree::Store.default.default_currency || Spree::Config[:currency]
+      currency = Spree::Store.default.default_currency
       ::Money.default_currency = currency
     end
 

data/lib/spree/permitted_attributes.rb

@@ -2,9 +2,13 @@ module Spree
   module PermittedAttributes
     ATTRIBUTES = [
       :address_attributes,
-      :classification_attributes,
       :checkout_attributes,
+      :classification_attributes,
+      :cms_page_attributes,
+      :cms_section_attributes,
       :customer_return_attributes,
+      :digital_attributes,
+      :digital_link_attributes,
       :image_attributes,
       :inventory_unit_attributes,
       :line_item_attributes,
@@ -28,19 +32,19 @@ module Spree
       :taxonomy_attributes,
       :user_attributes,
       :variant_attributes,
-      :cms_page_attributes,
-      :cms_section_attributes
+      :wishlist_attributes,
+      :wished_item_attributes
     ]
 
-    mattr_reader *ATTRIBUTES
+    mattr_reader(*ATTRIBUTES)
 
     @@address_attributes = [
       :id, :firstname, :lastname, :first_name, :last_name,
       :address1, :address2, :city, :country_iso, :country_id, :state_id,
       :zipcode, :phone, :state_name, :alternative_phone, :company,
       :user_id, :deleted_at, :label,
-      country: [:iso, :name, :iso3, :iso_name],
-      state: [:name, :abbr]
+      { country: [:iso, :name, :iso3, :iso_name],
+        state: [:name, :abbr] }
     ]
 
     @@checkout_attributes = [
@@ -54,9 +58,16 @@ module Spree
 
     @@cms_page_attributes = [:title, :meta_title, :content, :meta_description, :visible, :slug, :locale]
 
-    @@cms_section_attributes = [:name, :content, :settings, :fit, :destination]
+    @@cms_section_attributes = [:name, :cms_page_id, :fit, :destination, { content: {}, settings: {} }]
 
-    @@customer_return_attributes = [:stock_location_id, return_items_attributes: [:id, :inventory_unit_id, :return_authorization_id, :returned, :pre_tax_amount, :acceptance_status, :exchange_variant_id, :resellable]]
+    @@customer_return_attributes = [:stock_location_id, {
+      return_items_attributes: [:id, :inventory_unit_id, :return_authorization_id, :returned, :pre_tax_amount,
+                                :acceptance_status, :exchange_variant_id, :resellable]
+    }]
+
+    @@digital_attributes = [:attachment, :variant_id]
+
+    @@digital_link_attributes = [:access_counter]
 
     @@image_attributes = [:alt, :attachment, :position, :viewable_type, :viewable_id]
 
@@ -78,17 +89,18 @@ module Spree
     @@product_properties_attributes = [:property_name, :value, :position]
 
     @@product_attributes = [
-      :name, :description, :available_on, :discontinue_on, :permalink, :meta_description,
+      :name, :description, :available_on, :make_active_at, :discontinue_on, :permalink, :meta_description,
       :meta_keywords, :price, :sku, :deleted_at, :prototype_id,
       :option_values_hash, :weight, :height, :width, :depth,
       :shipping_category_id, :tax_category_id,
       :cost_currency, :cost_price, :compare_at_price,
-      option_type_ids: [], taxon_ids: []
+      { option_type_ids: [], taxon_ids: [] }
     ]
 
     @@property_attributes = [:name, :presentation]
 
-    @@return_authorization_attributes = [:amount, :memo, :stock_location_id, :inventory_units_attributes, :return_authorization_reason_id]
+    @@return_authorization_attributes = [:amount, :memo, :stock_location_id, :inventory_units_attributes,
+                                         :return_authorization_reason_id]
 
     @@shipment_attributes = [
       :order, :special_instructions, :stock_location_id, :id,
@@ -114,12 +126,15 @@ module Spree
       :quantity, :stock_item, :stock_item_id, :originator, :action
     ]
 
-    @@store_attributes = [:name, :url, :seo_title, :code, :meta_keywords, :logo,
+    @@store_attributes = [:name, :url, :seo_title, :code, :meta_keywords,
                           :meta_description, :default_currency, :mail_from_address,
                           :customer_support_email, :facebook, :twitter, :instagram,
                           :description, :address, :contact_phone, :supported_locales,
                           :default_locale, :default_country_id, :supported_currencies,
-                          :new_order_notifications_email, :mailer_logo, :favicon_image, :checkout_zone_id, :seo_robots]
+                          :new_order_notifications_email, :checkout_zone_id, :seo_robots,
+                          :digital_asset_authorized_clicks, :digital_asset_authorized_days,
+                          :limit_digital_download_count, :limit_digital_download_days, :digital_asset_link_expire_time,
+                          { mailer_logo_attributes: {}, favicon_image_attributes: {}, logo_attributes: {} }]
 
     @@store_credit_attributes = %i[amount currency category_id memo]
 
@@ -131,14 +146,19 @@ module Spree
     ]
 
     # TODO: Should probably use something like Spree.user_class.attributes
-    @@user_attributes = [:email, :bill_address_id, :ship_address_id, :password, :password_confirmation]
+    @@user_attributes = [:email, :bill_address_id, :ship_address_id, :password, :first_name, :last_name,
+                         :password_confirmation, { public_metadata: {}, private_metadata: {} }]
 
     @@variant_attributes = [
       :name, :presentation, :cost_price, :discontinue_on, :lock_version,
       :position, :track_inventory,
       :product_id, :product, :option_values_attributes, :price, :compare_at_price,
-      :weight, :height, :width, :depth, :sku, :cost_currency,
-      options: [:name, :value], option_value_ids: []
+      :weight, :height, :width, :depth, :sku, :barcode, :cost_currency,
+      { options: [:name, :value], option_value_ids: [] }
     ]
+
+    @@wishlist_attributes = [:name, :is_default, :is_private]
+
+    @@wished_item_attributes = [:variant_id, :quantity]
   end
 end

data/lib/spree/testing_support/authorization_helpers.rb

@@ -37,11 +37,13 @@ module Spree
             ability_class.register_ability(ability)
           end
 
-          let(:admin_token) { Doorkeeper::AccessToken.create!(scopes: 'admin read write').token }
+          let(:admin_app) { Spree::OauthApplication.create!(name: 'Admin Panel', scopes: 'admin') }
+          let(:admin_token) { Spree::OauthAccessToken.create!(application: admin_app, scopes: 'admin').token }
 
           before do
             allow(Spree.user_class).to receive(:find_by).and_return(Spree.user_class.new)
             if defined?(Spree::Admin)
+              allow_any_instance_of(Spree::Admin::BaseController).to receive(:admin_oauth_application).and_return(admin_app)
               allow_any_instance_of(Spree::Admin::BaseController).to receive(:admin_oauth_token).and_return(admin_token)
             end
           end

data/lib/spree/testing_support/capybara_config.rb

@@ -1,28 +1,23 @@
 require 'capybara-screenshot/rspec'
 
-Capybara.save_path = ENV['CIRCLE_ARTIFACTS'] if ENV['CIRCLE_ARTIFACTS']
+Capybara.configure do |config|
+  config.save_path = ENV['CIRCLE_ARTIFACTS'] if ENV['CIRCLE_ARTIFACTS']
+  config.server = :puma
+  config.default_driver = :rack_test
+  config.javascript_driver = :selenium_chrome_headless
+  config.default_max_wait_time = ENV.fetch('CAPYBARA_MAX_WAIT_TIME', 45).to_i
+  config.always_include_port = true
+  config.match = :smart
+  config.ignore_hidden_elements = true
+end
 
 if ENV['WEBDRIVER'] == 'accessible'
   require 'capybara/accessible'
   Capybara.javascript_driver = :accessible
-else
-  Capybara.register_driver :chrome do |app|
-    Selenium::WebDriver.logger.level = :error
-
-    Capybara::Selenium::Driver.new app,
-      browser: :chrome,
-      options: Selenium::WebDriver::Chrome::Options.new(
-        args: %w[no-sandbox disable-dev-shm-usage disable-popup-blocking headless disable-gpu window-size=1920,1080 --enable-features=NetworkService,NetworkServiceInProcess --disable-features=VizDisplayCompositor],
-        log_level: :error
-      )
-  end
-  Capybara.javascript_driver = :chrome
+end
 
-  Capybara::Screenshot.register_driver(:chrome) do |driver, path|
-    driver.browser.save_screenshot(path)
+RSpec.configure do |config|
+  config.before(:each, js: true) do
+    Capybara.page.driver.browser.manage.window.resize_to(1400, 900)
   end
 end
-Capybara.default_max_wait_time = ENV.fetch('CAPYBARA_MAX_WAIT_TIME', 45).to_i
-Capybara.server = :puma
-
-Capybara.always_include_port = true

data/lib/spree/testing_support/capybara_ext.rb

@@ -59,15 +59,10 @@ def wait_for(options = {})
   default_options = { error: nil, seconds: 5 }.merge(options)
 
   Selenium::WebDriver::Wait.new(timeout: default_options[:seconds]).until { yield }
-rescue Selenium::WebDriver::Error::TimeOutError
+rescue Selenium::WebDriver::Error::TimeoutError
   default_options[:error].nil? ? false : raise(default_options[:error])
 end
 
-Capybara.configure do |config|
-  config.match = :smart
-  config.ignore_hidden_elements = true
-end
-
 RSpec::Matchers.define :have_meta do |name, expected|
   match do |_actual|
     has_css?("meta[name='#{name}'][content='#{expected}']", visible: false)

data/lib/spree/testing_support/common_rake.rb

@@ -14,6 +14,13 @@ namespace :common do
     ENV['RAILS_ENV'] = 'test'
     Rails.env = 'test'
 
+    if ENV['LIB_NAME'] == 'spree/backend'
+      puts 'Preparing NPM package...'
+      system('yarn install')
+      system('yarn build')
+      system('yarn link')
+    end
+
     Spree::DummyGenerator.start ["--lib_name=#{ENV['LIB_NAME']}", '--quiet']
     Spree::InstallGenerator.start [
       "--lib_name=#{ENV['LIB_NAME']}",
@@ -29,9 +36,16 @@ namespace :common do
     ]
 
     puts 'Setting up dummy database...'
-    system("bundle exec rake db:drop db:create > #{File::NULL}")
+    system('bin/rails db:environment:set RAILS_ENV=test')
+    system('bundle exec rake db:drop db:create')
     Spree::DummyModelGenerator.start
-    system("bundle exec rake db:migrate > #{File::NULL}")
+    system('bundle exec rake db:migrate')
+
+    unless ['spree/api', 'spree/core', 'spree/sample', 'spree/emails'].include?(ENV['LIB_NAME'])
+      puts 'Setting up node environment'
+      system('bin/rails javascript:install:esbuild')
+      system('bin/rails turbo:install')
+    end
 
     begin
       require "generators/#{ENV['LIB_NAME']}/install/install_generator"
@@ -42,13 +56,18 @@ namespace :common do
     end
 
     unless ['spree/api', 'spree/core', 'spree/sample'].include?(ENV['LIB_NAME'])
+      if ENV['LIB_NAME'] == 'spree/backend'
+        puts 'Installing node dependencies...'
+        system('yarn link @spree/dashboard')
+        system('yarn install')
+      end
       puts 'Precompiling assets...'
-      system("bundle exec rake assets:precompile > #{File::NULL}")
+      system('bundle exec rake assets:precompile')
     end
   end
 
   task :seed do |_t|
     puts 'Seeding ...'
-    system("bundle exec rake db:seed RAILS_ENV=test > #{File::NULL}")
+    system('bundle exec rake db:seed RAILS_ENV=test')
   end
 end

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

@@ -0,0 +1,15 @@
+FactoryBot.define do
+  factory :asset, class: Spree::Asset do
+    viewable_type {}
+    viewable_id {}
+    attachment_width { 340 }
+    attachment_height { 280 }
+    attachment_file_size { 128 }
+    position { 1 }
+    attachment_content_type { '.jpg' }
+    attachment_file_name { 'attachment.jpg' }
+    type {}
+    attachment_updated_at {}
+    alt {}
+  end
+end

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

@@ -1,8 +1,8 @@
 FactoryBot.define do
-  factory :cms_section, class: Spree::CmsSection do
+  factory :base_cms_section, class: Spree::CmsSection do
     name { generate(:random_string) }
 
-    association :cms_page, factory: :base_cms_page
+    association :cms_page, factory: :cms_feature_page
 
     factory :cms_hero_image_section do
       type { 'Spree::Cms::Sections::HeroImage' }

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

@@ -0,0 +1,10 @@
+FactoryBot.define do
+  factory :digital, class: Spree::Digital do
+    after(:build) do |digital|
+      digital.attachment.attach(io: File.new("#{Spree::Core::Engine.root}/spec/fixtures/thinking-cat.jpg"),
+                                filename: 'thinking-cat.jpg')
+    end
+
+    variant
+  end
+end

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

@@ -0,0 +1,6 @@
+FactoryBot.define do
+  factory :digital_link, class: Spree::DigitalLink do
+    digital
+    line_item
+  end
+end

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

@@ -0,0 +1,9 @@
+FactoryBot.define do
+  factory :favicon_image, class: Spree::StoreFaviconImage do
+    transient do
+      filepath { Spree::Core::Engine.root.join('spec', 'fixtures', 'favicon.ico') }
+    end
+
+    attachment { Rack::Test::UploadedFile.new(filepath) }
+  end
+end

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

@@ -1,7 +1,9 @@
 FactoryBot.define do
   factory :icon, class: Spree::Icon do
     before(:create) do |icon|
-      icon.attachment.attach(io: File.new(Spree::Core::Engine.root + 'spec/fixtures' + 'thinking-cat.jpg'), filename: 'thinking-cat.jpg')
+      if icon.respond_to? :attachment
+        icon.attachment.attach(io: File.new(Spree::Core::Engine.root + 'spec/fixtures' + 'thinking-cat.jpg'), filename: 'thinking-cat.jpg')
+      end
     end
   end
 end

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

@@ -1,7 +1,9 @@
 FactoryBot.define do
   factory :image, class: Spree::Image do
     before(:create) do |image|
-      image.attachment.attach(io: File.new(Spree::Core::Engine.root + 'spec/fixtures' + 'thinking-cat.jpg'), filename: 'thinking-cat.jpg')
+      if image.class.method_defined?(:attachment)
+        image.attachment.attach(io: File.new(Spree::Core::Engine.root + 'spec/fixtures' + 'thinking-cat.jpg'), filename: 'thinking-cat.jpg')
+      end
     end
   end
 end

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

@@ -0,0 +1,6 @@
+FactoryBot.define do
+  factory :log_entry, class: Spree::LogEntry do
+    source { build(:order) }
+    details { 'Some details' }
+  end
+end

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

@@ -2,7 +2,7 @@ FactoryBot.define do
   factory :menu_item, class: Spree::MenuItem do
     sequence(:name) { |n| "Link no. #{n} To Somewhere" }
     item_type { 'Link' }
-    linked_resource_type { 'URL' }
+    linked_resource_type { 'Spree::Linkable::Uri' }
 
     menu
     icon

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

@@ -55,7 +55,8 @@ FactoryBot.define do
           order.line_items.reload
         end
 
-        create(:shipment, order: order, cost: evaluator.shipment_cost)
+        stock_location = order.line_items&.first&.variant&.stock_items&.first&.stock_location || create(:stock_location)
+        create(:shipment, order: order, cost: evaluator.shipment_cost, stock_location: stock_location)
         order.shipments.reload
 
         order.update_with_updater!
@@ -77,7 +78,10 @@ FactoryBot.define do
 
         factory :completed_order_with_store_credit_payment do
           after(:create) do |order|
-            create(:store_credit_payment, amount: order.total, order: order)
+            store_credit = create(:store_credit, amount: order.total, store: order.store, user: order.user)
+            payment_method = create(:store_credit_payment_method, stores: [order.store])
+
+            create(:store_credit_payment, amount: order.total, order: order, source: store_credit, payment_method: payment_method)
           end
         end
 

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

@@ -0,0 +1,6 @@
+FactoryBot.define do
+  factory :payment_capture_event, class: Spree::PaymentCaptureEvent do
+    payment
+    amount { 10.0 }
+  end
+end

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

@@ -5,7 +5,7 @@ FactoryBot.define do
     state         { 'checkout' }
     response_code { '12345' }
 
-    association(:payment_method, factory: :credit_card_payment_method)
+    payment_method { create(:credit_card_payment_method, stores: [order.store]) }
     association(:source, factory: :credit_card)
 
     factory :payment_with_refund do
@@ -24,7 +24,7 @@ FactoryBot.define do
   end
 
   factory :store_credit_payment, class: Spree::Payment, parent: :payment do
-    association(:payment_method, factory: :store_credit_payment_method)
-    association(:source, factory: :store_credit)
+    payment_method { create(:store_credit_payment_method, stores: [order.store]) }
+    source { create(:store_credit, store: order.store, user: order.user) }
   end
 end

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

@@ -1,13 +1,15 @@
 FactoryBot.define do
   factory :base_product, class: Spree::Product do
-    sequence(:name)   { |n| "Product ##{n} - #{Kernel.rand(9999)}" }
+    sequence(:name)   { |n| "Product #{n}#{Kernel.rand(9999)}" }
     description       { generate(:random_description) }
     price             { 19.99 }
     cost_price        { 17.00 }
     sku               { generate(:sku) }
     available_on      { 1.year.ago }
+    make_active_at    { 1.year.ago }
     deleted_at        { nil }
     shipping_category { |r| Spree::ShippingCategory.first || r.association(:shipping_category) }
+    status            { 'active' }
 
     # ensure stock item will be created for this products master
     # also attach this product to the default store if no stores are passed in
@@ -21,6 +23,9 @@ FactoryBot.define do
         product.stores << [store]
       end
     end
+    after(:create) do |product|
+      Spree::StockLocation.all.each { |stock_location| stock_location.propagate_variant(product.master) unless stock_location.stock_items.exists?(variant: product.master) }
+    end
 
     factory :custom_product do
       name  { 'Custom Product' }

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

@@ -1,6 +1,7 @@
 FactoryBot.define do
   factory :product_property, class: Spree::ProductProperty do
     product { create(:product, stores: [create(:store)]) }
+    value { "val-#{rand(50)}" }
     property
   end
 end

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

@@ -0,0 +1,5 @@
+FactoryBot.define do
+  factory :promotion_action, class: Spree::PromotionAction do
+    association :promotion
+  end
+end

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

@@ -15,7 +15,7 @@ FactoryBot.define do
   end
 
   factory :refund_reason, class: Spree::RefundReason do
-    sequence(:name) { |n| "Refund for return ##{n}" }
+    sequence(:name) { |n| "Refund for return #{n}" }
     active  { true }
     mutable { false }
   end

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

@@ -0,0 +1,7 @@
+FactoryBot.define do
+  factory :reimbursement_credit, class: Spree::Reimbursement::Credit do
+    reimbursement
+
+    amount { 100.00 }
+  end
+end

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

@@ -14,7 +14,7 @@ FactoryBot.define do
   end
 
   factory :return_authorization_reason, class: Spree::ReturnAuthorizationReason do
-    sequence(:name) { |n| "Defect ##{n}" }
+    sequence(:name) { |n| "Defect #{n}" }
     active          { true }
     mutable         { false }
   end

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

@@ -1,6 +1,6 @@
 FactoryBot.define do
   factory :role, class: Spree::Role do
-    sequence(:name) { |n| "Role ##{n}" }
+    sequence(:name) { |n| "Role #{n}" }
 
     factory :admin_role do
       name { 'admin' }

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

@@ -1,5 +1,5 @@
 FactoryBot.define do
   factory :shipping_category, class: Spree::ShippingCategory do
-    sequence(:name) { |n| "ShippingCategory ##{n}" }
+    sequence(:name) { |n| "ShippingCategory #{n}" }
   end
 end

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

@@ -0,0 +1,7 @@
+FactoryBot.define do
+  factory :shipping_rate, class: Spree::ShippingRate do
+    cost { BigDecimal(10) }
+    shipping_method
+    shipment
+  end
+end

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

@@ -0,0 +1,8 @@
+FactoryBot.define do
+  factory :state_change, class: Spree::StateChange do
+    stateful { build(:order) }
+    previous_state { 'cart' }
+    next_state { 'address' }
+    user { build(:user) }
+  end
+end

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

@@ -4,6 +4,13 @@ FactoryBot.define do
     stock_location
     variant
 
+    before(:create) do |stock_item|
+      Spree::StockItem.find_by(
+        variant: stock_item.variant,
+        stock_location: stock_item.stock_location
+      )&.destroy
+    end
+
     after(:create) { |object| object.adjust_count_on_hand(10) }
   end
 end