Ignore case sensitivity in validation

Fixes MySQL error
boomerdigital · May 5, 2023 · f8ddf18 · f8ddf18
1 parent 9000bcd
commit f8ddf18
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/decorators/models/solidus_user_roles/spree/role_decorator.rb b/app/decorators/models/solidus_user_roles/spree/role_decorator.rb
@@ -8,7 +8,7 @@ def self.prepended(base)
         base.has_many :permission_sets, through: :role_permissions
 
         base.scope :non_base_roles, -> { where.not(name: ['admin', 'user']) }
-        base.validates :name, uniqueness: true
+        base.validates_uniqueness_of :name, case_sensitive: false
         base.after_save :assign_permissions
       end
 

diff --git a/spec/models/spree/role_spec.rb b/spec/models/spree/role_spec.rb
@@ -9,7 +9,7 @@
 
   it { is_expected.to have_many :role_permissions }
   it { is_expected.to have_many(:permission_sets).through(:role_permissions) }
-  it { is_expected.to validate_uniqueness_of :name }
+  it { should validate_uniqueness_of(:name).ignoring_case_sensitivity }
 
   describe "#assign_permissions" do
     it 'creates new Spree::RoleConfiguration::Role' do