solidus_acts_as_tenant 0.0.5

data/db/migrate/20241204043609_add_tenant_id_to_solidus_tables.rb

@@ -0,0 +1,103 @@
+class AddTenantIdToSolidusTables < ActiveRecord::Migration[7.0]
+  # If using a different tenant model, you will need to change the ::SolidusActsAsTenant.config.tenant_column_name
+  # that is added to the tables.
+
+  # These are tables that don't need a tenancy but may be desired:
+  # spree_countries (also remove "iso" uniqueness index)
+  # spree_states
+  # spree_permission_sets
+  # spree_roles (also remove "name" uniqueness index)
+  # spree_role_permissions
+
+  TABLES = %w[
+    spree_addresses
+    spree_adjustments
+    spree_credit_cards
+    spree_payments
+    spree_refunds
+    spree_roles_users
+    spree_shipping_method_categories
+    spree_store_credit_types
+    spree_store_credits
+    spree_calculators
+    spree_users
+    spree_adjustment_reasons
+    spree_assets
+    spree_cartons
+    spree_customer_returns
+    spree_inventory_units
+    spree_line_item_actions
+    spree_line_items
+    spree_log_entries
+    spree_option_type_prototypes
+    spree_option_types
+    spree_option_values
+    spree_option_values_variants
+    spree_orders
+    spree_payment_capture_events
+    spree_payment_methods
+    spree_preferences
+    spree_prices
+    spree_product_option_types
+    spree_product_properties
+    spree_products
+    spree_products_taxons
+    spree_properties
+    spree_property_prototypes
+    spree_prototype_taxons
+    spree_prototypes
+    spree_refund_reasons
+    spree_reimbursement_credits
+    spree_reimbursement_types
+    spree_reimbursements
+    spree_return_authorizations
+    spree_return_items
+    spree_return_reasons
+    spree_shipments
+    spree_shipping_categories
+    spree_shipping_method_stock_locations
+    spree_shipping_method_zones
+    spree_shipping_methods
+    spree_shipping_rate_taxes
+    spree_shipping_rates
+    spree_state_changes
+    spree_stock_items
+    spree_stock_locations
+    spree_stock_movements
+    spree_store_credit_categories
+    spree_store_credit_events
+    spree_store_payment_methods
+    spree_stores
+    spree_store_shipping_methods
+    spree_store_credit_reasons
+    spree_tax_categories
+    spree_tax_rates
+    spree_taxonomies
+    spree_taxons
+    spree_tax_rate_tax_categories
+    spree_unit_cancels
+    spree_user_addresses
+    spree_user_stock_locations
+    spree_variant_property_rule_conditions
+    spree_variant_property_rule_values
+    spree_variant_property_rules
+    spree_variants
+    spree_wallet_payment_sources
+    spree_zone_members
+    spree_zones
+  ].push(::Spree.user_class.table_name).uniq.freeze
+
+  def up
+    TABLES.each do |table|
+      add_column table, ::SolidusActsAsTenant.config.tenant_column_name, :bigint
+      add_index table, ::SolidusActsAsTenant.config.tenant_column_name
+    end
+  end
+
+  def down
+    TABLES.each do |table|
+      remove_index table
+      remove_column table, ::SolidusActsAsTenant.config.tenant_column_name
+    end
+  end
+end

data/db/migrate/20241204055845_update_spree_unique_indexes_with_tenant_scope.rb

@@ -0,0 +1,21 @@
+class UpdateSpreeUniqueIndexesWithTenantScope < ActiveRecord::Migration[7.0]
+  # If using a different tenant model, you will need to change the :::SolidusActsAsTenant.config.tenant_column_name,
+  # in the following indexes.
+  def up
+    remove_index :spree_preferences, name: :index_spree_preferences_on_key
+    remove_index :spree_products, name: :index_spree_products_on_slug
+
+    add_index :spree_preferences, [::SolidusActsAsTenant.config.tenant_column_name, :key], unique: true,
+      name: :index_spree_preferences_on_tenant_and_key
+    add_index :spree_products, [::SolidusActsAsTenant.config.tenant_column_name, :slug], unique: true,
+      name: :index_spree_products_on_tenant_and_slug
+  end
+
+  def down
+    remove_index :spree_preferences, name: :index_spree_preferences_on_tenant_and_key
+    remove_index :spree_products, name: :index_spree_products_on_tenant_and_slug
+
+    add_index :spree_preferences, :key, unique: true, name: :index_spree_preferences_on_key
+    add_index :spree_products, :slug, unique: true, name: :index_spree_products_on_slug
+  end
+end

data/db/migrate/20241204105318_create_spree_tenants.rb

@@ -0,0 +1,16 @@
+class CreateSpreeTenants < ActiveRecord::Migration[7.0]
+  # If using a different tenant model, you will need to create a migration for it and remove this one.
+  def change
+    create_table :spree_tenants do |t|
+      t.string :name
+      t.string :subdomain # Only used if setting tenant by subdomain
+      t.string :domain # Only used if setting tenant by domain
+
+      t.timestamps
+    end
+
+    add_index :spree_tenants, :name, unique: true
+    add_index :spree_tenants, :subdomain, unique: true
+    add_index :spree_tenants, :domain, unique: true
+  end
+end

data/lib/generators/solidus_acts_as_tenant/install/install_generator.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module SolidusActsAsTenant
+  module Generators
+    class InstallGenerator < Rails::Generators::Base
+      class_option :auto_run_migrations, type: :boolean, default: false
+      source_root File.expand_path('templates', __dir__)
+
+      def self.exit_on_failure?
+        true
+      end
+
+      def copy_initializer
+        template 'initializer.rb', 'config/initializers/solidus_acts_as_tenant.rb'
+      end
+
+      def copy_config
+        template 'tenant_aware_models.yml', 'config/tenant_aware_models.yml'
+      end
+
+      def add_migrations
+        run 'bin/rails railties:install:migrations FROM=solidus_acts_as_tenant'
+      end
+
+      def run_migrations
+        puts 'Remember to inspect and adapt the migration files before migrating!' # rubocop:disable Rails/Output
+        run_migrations = options[:auto_run_migrations] || ['', 'y', 'Y'].include?(ask('Would you like to run the migrations now? [Y/n]')) # rubocop:disable Layout/LineLength
+        if run_migrations
+          run 'bin/rails db:migrate'
+        else
+          puts 'Skipping bin/rails db:migrate, don\'t forget to run it!' # rubocop:disable Rails/Output
+        end
+      end
+    end
+  end
+end

data/lib/generators/solidus_acts_as_tenant/install/templates/initializer.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+SolidusActsAsTenant.configure do |config|
+  # This configuration is used to dynamically prepend existing models with the acts_as_tenant concern.
+  # See the tenant_aware_classes.yml file for inormation on how to configure this file.
+  config.tenant_aware_models =
+    YAML.safe_load(ERB.new(File.read(Rails.root.join('config/tenant_aware_models.yml'))).result)
+
+  config.tenant_column_name = :tenant_id
+  # Tenant model class name
+  config.class_name = 'Spree::Tenant'
+
+  # These options are passed to the acts_as_tenant class method in the tenant_aware_classes.yml file.
+  config.acts_as_tenant_args = [:tenant, { class_name: 'Spree::Tenant' }]
+end

data/lib/generators/solidus_acts_as_tenant/install/templates/tenant_aware_models.yml

@@ -0,0 +1,98 @@
+# Any model that is already defined and has an a tenant column can be added to this list.
+# It is not restricted to the Solidus models so it can be used for any model that is inherited from a gem.
+
+# It is importnant to also note that any unique indexes on the database will need to replaced with a unique index on scoped to the tenant column.
+
+# Format:
+  # - [Class name, [uniqueness attributes to update with tenant scope]]
+  # The uniqueness attributes array are optional for cases where there is already a uniqueness validation on the model that are not scoped to the tenant column.
+
+# Notes regarding Solidus models:
+  # These are are classes that are created in this extention because they are not defined in Solidus:
+    # [Spree::ProductTaxon, []],
+    # [Spree::ReimbursementCredit, []],
+
+  # These are classes that don't need a tenancy but may be desired:
+    # [Spree::Country, []],
+    # [Spree::State, []],
+    # [Spree::PermissionSet, []],
+    # [Spree::Role, [:name]],
+    # [Spree::RolePermission, []],
+
+  # These are not needed at all as they are internal to Solidus (and may cause issues):
+    # [Spree::OrderMutex, []],
+
+  # The user model conflicts with devise, so it is not included here.
+  # But it may be included if the TenantNotSet error is bypassed using the config added on this fork's branch:
+  # https://github.com/nebulab/acts_as_tenant/tree/Pass-the-ar-relation-for-context-to-should_require_tenant
+    # [<%= ::Spree.user_class.to_s %>, []]
+
+---
+- [Spree::Address, []]
+- [Spree::AdjustmentReason, [name, code]]
+- [Spree::Adjustment, []]
+- [Spree::Asset, []]
+- [Spree::Calculator, []]
+- [Spree::Carton, []]
+- [Spree::CreditCard, []]
+- [Spree::CustomerReturn, []]
+- [Spree::InventoryUnit, []]
+- [Spree::LineItemAction, []]
+- [Spree::LineItem, []]
+- [Spree::LogEntry, []]
+- [Spree::OptionType, [name]]
+- [Spree::OptionValue, [name]]
+- [Spree::OptionValuesVariant, []]
+- [Spree::Order, []]
+- [Spree::PaymentCaptureEvent, []]
+- [Spree::PaymentMethod, []]
+- [Spree::Payment, []]
+- [Spree::Preference, [key]]
+- [Spree::Price, []]
+- [Spree::ProductOptionType, []]
+- [Spree::ProductProperty, []]
+- [Spree::Product, [slug]]
+- [Spree::Property, []]
+- [Spree::RefundReason, [name]]
+- [Spree::Refund, []]
+- [Spree::ReimbursementType, [name]]
+- [Spree::Reimbursement, []]
+- [Spree::ReturnAuthorization, []]
+- [Spree::ReturnItem, []]
+- [Spree::ReturnReason, []]
+- [Spree::RoleUser, []]
+- [Spree::Shipment, []]
+- [Spree::ShippingCategory, []]
+- [Spree::ShippingMethodCategory, []]
+- [Spree::ShippingMethodStockLocation, []]
+- [Spree::ShippingMethodZone, []]
+- [Spree::ShippingMethod, []]
+- [Spree::ShippingRateTax, []]
+- [Spree::StateChange, []]
+- [Spree::ShippingRate, []]
+- [Spree::StockItem, []]
+- [Spree::StockLocation, [code]]
+- [Spree::StockMovement, []]
+- [Spree::StoreCreditCategory, []]
+- [Spree::StoreCreditEvent, []]
+- [Spree::StoreCreditType, []]
+- [Spree::StoreCredit, []]
+- [Spree::StorePaymentMethod, []]
+- [Spree::Store, [code]]
+- [Spree::StoreShippingMethod, []]
+- [Spree::StoreCreditReason, [name]]
+- [Spree::TaxCategory, [name]]
+- [Spree::Taxonomy, [name]]
+- [Spree::Taxon, []]
+- [Spree::TaxRate, []]
+- [Spree::TaxRateTaxCategory, []]
+- [Spree::UnitCancel, []]
+- [Spree::UserAddress, []]
+- [Spree::UserStockLocation, []]
+- [Spree::VariantPropertyRuleCondition, []]
+- [Spree::VariantPropertyRuleValue, []]
+- [Spree::VariantPropertyRule, []]
+- [Spree::Variant, [sku]]
+- [Spree::WalletPaymentSource, []]
+- [Spree::ZoneMember, []]
+- [Spree::Zone, [name]]

data/lib/solidus_acts_as_tenant/configuration.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+module SolidusActsAsTenant
+  class Configuration
+    attr_accessor :tenant_aware_models
+    attr_writer :tenant_column_name, :acts_as_tenant_args, :class_name
+
+    def tenant_column_name
+      @tenant_column_name ||= :tenant_id
+    end
+
+    def acts_as_tenant_args
+      @acts_as_tenant_args ||= [:tenant, { class_name: 'Spree::Tenant' }]
+    end
+
+    def class_name
+      @class_name ||= 'Spree::Tenant'
+    end
+  end
+
+  class << self
+    def configuration
+      @configuration ||= Configuration.new
+    end
+
+    alias config configuration
+
+    def configure
+      yield configuration
+    end
+  end
+end

data/lib/solidus_acts_as_tenant/engine.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require 'solidus_core'
+require 'solidus_support'
+
+module SolidusActsAsTenant
+  class Engine < Rails::Engine
+    include SolidusSupport::EngineExtensions
+
+    isolate_namespace ::Spree
+
+    engine_name 'solidus_acts_as_tenant'
+
+    # use rspec for tests
+    config.generators do |g|
+      g.test_framework :rspec
+    end
+
+    config.to_prepare do
+      SolidusActsAsTenant::TenantAware.setup_tenant_aware_models
+    end
+  end
+end

data/lib/solidus_acts_as_tenant/factories/preference_factory.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+FactoryBot.define do
+  factory :preference, class: 'Spree::Preference' do
+    sequence(:key) { |i| "Key#{i}" }
+  end
+end

data/lib/solidus_acts_as_tenant/factories/tenant_factory.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+FactoryBot.define do
+  factory :tenant, class: ::SolidusActsAsTenant.config.class_name do
+    sequence(:name) { |i| "Demo #{i}" }
+    sequence(:subdomain) { |i| "demo-#{i}" }
+    sequence(:domain) { |i| "demo-#{i}.com" }
+  end
+end

data/lib/solidus_acts_as_tenant/tenant_aware.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+module SolidusActsAsTenant
+  # TenantAware automatically adds multi-tenant support to models
+  # by adding tenant scoping and updating uniqueness validations to be tenant-aware.
+  module TenantAware
+    class << self
+      def setup_tenant_aware_models
+        tenant_aware_models = SolidusActsAsTenant.config.tenant_aware_models || []
+
+        tenant_aware_models.each do |klass, validator_attributes|
+          Module.new do
+            @validator_attributes = validator_attributes
+            define_singleton_method(:prepended) do |base|
+              options = ::SolidusActsAsTenant.config.acts_as_tenant_args
+              base.acts_as_tenant(*options[0..-2], **options.last)
+
+              @validator_attributes&.each do |attribute|
+                TenantAware.update_uniqueness_validation(base, attribute)
+              end
+            end
+
+            klass.constantize.prepend(self)
+          end
+        end
+      end
+
+      def update_uniqueness_validation(base, attribute)
+        attribute = attribute.to_sym
+        validator = base._validators[attribute].find { _1.kind == :uniqueness }
+        raise "No uniqueness validator found for #{attribute} on #{base}" unless validator
+
+        new_options = validator.options.dup
+        new_options[:scope] = Array(new_options[:scope]).push(::SolidusActsAsTenant.config.tenant_column_name)
+
+        remove_existing_validation(base, attribute)
+        base.validates_uniqueness_of attribute, **new_options
+      end
+
+      def remove_existing_validation(base, attribute)
+        base._validators[attribute].reject! { _1.kind == :uniqueness }
+        base._validate_callbacks.each do |callback|
+          next unless callback.filter.try(:attributes)&.include?(attribute) &&
+                      callback.filter.kind == :uniqueness
+
+          callback.filter.attributes.delete(attribute)
+        end
+      end
+    end
+  end
+end

data/lib/solidus_acts_as_tenant/testing_support/factories.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+Dir["#{File.dirname(__FILE__)}/../factories/**"].each do |f|
+  require File.expand_path(f)
+end

data/lib/solidus_acts_as_tenant/utils/tenant_selector.rb

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+module SolidusActsAsTenant
+  module Utils
+    class TenantSelector
+      MESSAGES = {
+        available: "Available tenants: %<tenants>s",
+        prompt: "Select tenant: ",
+        not_found: "Tenant not found in list '%<tenants>s'",
+        current: "You are now Tenant '%<name>s'"
+      }.freeze
+
+      def ask
+        return if tenants.empty?
+
+        puts format(MESSAGES[:available], tenants: tenants) # rubocop:disable Rails/Output
+        print MESSAGES[:prompt] # rubocop:disable Rails/Output
+
+        tenant_name = ENV["DEFAULT_TENANT"] || gets.strip
+        switch_tenant!(tenant_name) unless tenant_name.empty?
+      end
+
+      def tenants
+        @tenants ||= fetch_tenants
+      end
+
+      def switch_tenant!(tenant_name)
+        if exists?(tenant_name)
+          switch(tenant_name)
+        elsif numeric_tenant?(tenant_name)
+          switch(tenants[tenant_name.to_i])
+        else
+          puts format(MESSAGES[:not_found], tenants: tenants) # rubocop:disable Rails/Output
+          return false
+        end
+
+        puts format(MESSAGES[:current], name: current) # rubocop:disable Rails/Output
+        true
+      end
+
+      def current
+        ActsAsTenant.current_tenant&.name
+      end
+
+      private
+
+      def fetch_tenants
+        [*tenant_names].uniq
+                       .each_with_index
+                       .to_h { |name, index| [index, name] }
+      end
+
+      def tenant_names
+        @tenant_names ||= ::SolidusActsAsTenant.config.class_name.constantize.pluck(:name).sort
+      end
+
+      def switch(tenant_name)
+        return false unless tenants.value?(tenant_name)
+
+        tenant = ::SolidusActsAsTenant.config.class_name.constantize.find_by(name: tenant_name)
+        ActsAsTenant.current_tenant = tenant
+        true
+      end
+
+      def exists?(tenant_name)
+        tenant_names.include?(tenant_name)
+      end
+
+      def numeric_tenant?(tenant_name)
+        tenant_name =~ /^\d+$/ && tenants.key?(tenant_name.to_i)
+      end
+    end
+  end
+end

data/lib/solidus_acts_as_tenant/version.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+module SolidusActsAsTenant
+  VERSION = '0.0.5'
+end

data/lib/solidus_acts_as_tenant.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+require 'acts_as_tenant'
+require 'solidus_acts_as_tenant/configuration'
+require 'solidus_acts_as_tenant/tenant_aware'
+require 'solidus_acts_as_tenant/utils/tenant_selector'
+require 'solidus_acts_as_tenant/version'
+require 'solidus_acts_as_tenant/engine'

data/solidus_acts_as_tenant.gemspec

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+require_relative 'lib/solidus_acts_as_tenant/version'
+
+Gem::Specification.new do |spec|
+  spec.name = 'solidus_acts_as_tenant'
+  spec.version = SolidusActsAsTenant::VERSION
+  spec.authors = ['Ikraam Ghoor']
+  spec.email = 'consult.ikraam@gmail.com'
+
+  spec.summary = 'Adds tenant functionality to solidus using the row level acts_as_tenant gem'
+  spec.description = 'Adds tenant functionality to solidus using the row level acts_as_tenant gem'
+  spec.homepage = 'https://github.com/nebulab/solidus_acts_as_tenant#readme'
+  spec.license = 'BSD-3-Clause'
+
+  spec.metadata['homepage_uri'] = spec.homepage
+  spec.metadata['source_code_uri'] = 'https://github.com/nebulab/solidus_acts_as_tenant'
+  spec.metadata['changelog_uri'] = 'https://github.com/nebulab/solidus_acts_as_tenant/blob/main/CHANGELOG.md'
+
+  spec.required_ruby_version = Gem::Requirement.new('>= 2.5', '< 4')
+
+  # Specify which files should be added to the gem when it is released.
+  # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
+  files = Dir.chdir(__dir__) { `git ls-files -z`.split("\x0") }
+
+  spec.files = files.grep_v(%r{^(test|spec|features)/})
+  spec.test_files = files.grep(%r{^(test|spec|features)/})
+  spec.bindir = "exe"
+  spec.executables = files.grep(%r{^exe/}) { |f| File.basename(f) }
+  spec.require_paths = ["lib"]
+
+  spec.add_dependency 'acts_as_tenant', '~> 1.0.1'
+  spec.add_dependency 'solidus_core', ['>= 2.0.0', '< 5']
+  spec.add_dependency 'solidus_support', '>= 0.14.1'
+
+  spec.add_development_dependency 'pry'
+  spec.add_development_dependency 'solidus_dev_support', '~> 2.11'
+end

data/spec/lib/solidus_acts_as_tenant/configuration_spec.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+RSpec.describe SolidusActsAsTenant::Configuration do
+  let(:configuration) { described_class.new }
+
+  describe 'default values' do
+    it 'has default tenant_column_name' do
+      expect(configuration.tenant_column_name).to eq(:tenant_id)
+    end
+
+    it 'has default acts_as_tenant_args' do
+      expect(configuration.acts_as_tenant_args).to eq([:tenant, { class_name: 'Spree::Tenant' }])
+    end
+
+    it 'has default class_name' do
+      expect(configuration.class_name).to eq('Spree::Tenant')
+    end
+  end
+
+  describe 'custom configuration' do
+    it 'allows setting tenant_aware_models' do
+      models = ['Spree::Product', 'Spree::Order']
+      configuration.tenant_aware_models = models
+      expect(configuration.tenant_aware_models).to eq(models)
+    end
+
+    it 'allows setting tenant_column_name' do
+      configuration.tenant_column_name = :custom_tenant_id
+      expect(configuration.tenant_column_name).to eq(:custom_tenant_id)
+    end
+
+    it 'allows setting acts_as_tenant_args' do
+      custom_args = [:custom_tenant, { class_name: 'Custom::Tenant' }]
+      configuration.acts_as_tenant_args = custom_args
+      expect(configuration.acts_as_tenant_args).to eq(custom_args)
+    end
+
+    it 'allows setting class_name' do
+      configuration.class_name = 'Custom::Tenant'
+      expect(configuration.class_name).to eq('Custom::Tenant')
+    end
+  end
+end

data/spec/lib/solidus_acts_as_tenant/solidus_acts_as_tenant_spec.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+RSpec.describe SolidusActsAsTenant do
+  describe '.configuration' do
+    it 'returns the same configuration instance' do
+      expect(described_class.configuration).to eq(described_class.configuration)
+    end
+
+    it 'aliases config to configuration' do
+      expect(described_class.config).to eq(described_class.configuration)
+    end
+  end
+
+  describe '.configure' do
+    it 'yields the configuration instance' do
+      expect { |b| described_class.configure(&b) }.to yield_with_args(SolidusActsAsTenant::Configuration)
+    end
+
+    it 'allows configuring through a block' do
+      described_class.configure do |config|
+        config.tenant_column_name = :custom_tenant_id
+      end
+
+      expect(described_class.configuration.tenant_column_name).to eq(:custom_tenant_id)
+    end
+  end
+end

data/spec/lib/solidus_acts_as_tenant/tenant_aware_spec.rb

@@ -0,0 +1,48 @@
+describe SolidusActsAsTenant::TenantAware do
+  describe '#setup_tenant_aware_models' do
+    let!(:tenant) { ::SolidusActsAsTenant.config.class_name.constantize.find_or_create_by!(name: 'Test') }
+    let!(:tenant2) { create(:tenant, name: 'Test2') }
+
+    SolidusActsAsTenant.config.tenant_aware_models.each do |klass, validator_attributes|
+      describe "for #{klass}" do
+        it 'has a tenant accessor and the current tenant is already set', set_tenant: true do
+          expect(klass.constantize.new.tenant).to eq tenant
+        end
+
+        validator_attributes.each do |attribute|
+          it "validates uniqueness of #{attribute}", set_tenant: true do
+            entry = create(klass.demodulize.underscore.downcase.to_sym, attribute => 'test@test.com')
+            duplicate = entry.dup
+
+            if klass == 'Spree::Product'
+              duplicate.master = entry.master
+              duplicate.slug = entry.slug
+            end
+
+            duplicate.valid?
+
+            expect(duplicate.errors[attribute]).to include('has already been taken')
+          end
+
+          it 'allows duplicate values across different tenants' do
+            ::ActsAsTenant.current_tenant = tenant
+            entry = create(klass.demodulize.underscore.downcase.to_sym, attribute => 'test@test.com',
+              tenant: tenant)
+            duplicate = entry.dup
+
+            if klass == 'Spree::Product'
+              duplicate.master = entry.master
+              duplicate.slug = entry.slug
+            end
+
+            ::ActsAsTenant.current_tenant = tenant2
+            duplicate.tenant = tenant2
+            duplicate.valid?
+
+            expect(duplicate.errors[attribute]).to be_empty
+          end
+        end
+      end
+    end
+  end
+end