Validate uniqnuess of member role name

What does this MR do and why?

It adds a requirement that a name is unique per namespace:

• it adds a validation to the model
• it removes existing duplicates by changing the name of duplicates
• it adds unique index to the member_roles table

Database queries

• duplicates removal
• we have around 10 member role records on Gitlab.com and we expect maximum of hundreds of records on self-managed instances (more likely tons)

MR acceptance checklist

Screenshots or screen recordings

How to set up and validate locally

1. Before checking out this branch, create few duplicates member roles (duplicated = same name and namespace), eg. in console:

mr1 = MemberRole.create(name: 'member role', base_access_level: 10, read_code: true)
mr2 = MemberRole.create(name: 'member role', base_access_level: 10, read_code: true)
mr3 = MemberRole.create(name: 'member role', base_access_level: 10, read_code: true)

2. Check out this branch
3. Run the migration
4. Check if the duplicated names were updated, eg. by checking in the console: MemberRole.all.map(&:name)
5. Try to create the duplicated records again, this should not work

Related to #438472 (closed)

db/post_migrate/20240221134504_add_name_unique_index_to_member_roles.rb

@@ -4,18 +4,18 @@ class AddNameUniqueIndexToMemberRoles < Gitlab::Database::Migration[2.2]
   disable_ddl_transaction!
   milestone '16.10'
 
-  OLD_INDEX_NAME = 'index_member_roles_on_namespace_id'
-  INDEX_NAME = 'index_member_roles_on_namespace_id_name_unique'
+  INDEX_WITH_NAMESPACE_NAME = 'index_member_roles_on_namespace_id_name_unique'
+  INDEX_NO_NAMESPACE_NAME = 'index_member_roles_on_name_unique'
 
   def up
-    remove_concurrent_index_by_name :member_roles, OLD_INDEX_NAME
-
-    add_concurrent_index :member_roles, [:namespace_id, :name], name: INDEX_NAME, unique: true
+    add_concurrent_index :member_roles, [:namespace_id, :name], name: INDEX_WITH_NAMESPACE_NAME,
+      unique: true, where: 'namespace_id IS NOT NULL'
+    add_concurrent_index :member_roles, [:name], name: INDEX_NO_NAMESPACE_NAME,
+      unique: true, where: 'namespace_id IS NULL'
   end
 
   def down
-    remove_concurrent_index_by_name :member_roles, INDEX_NAME
-
-    add_concurrent_index :member_roles, [:namespace_id], name: OLD_INDEX_NAME
+    remove_concurrent_index_by_name :member_roles, INDEX_WITH_NAMESPACE_NAME
+    remove_concurrent_index_by_name :member_roles, INDEX_NO_NAMESPACE_NAME
   end
 end