fix(CE): remove unique validation of organization name (#477) (#407)

Co-authored-by: datafloyd <[email protected]>

server/app/models/organization.rb

@@ -10,7 +10,7 @@
 #  updated_at  :datetime         not null
 #
 class Organization < ApplicationRecord
-  validates :name, presence: true, uniqueness: { case_sensitive: false }
+  validates :name, presence: true
 
   has_many :workspaces, dependent: :destroy
   has_many :workspace_users, through: :workspaces

server/db/migrate/20241010214745_remove_name_unique_index_from_organization.rb

@@ -0,0 +1,5 @@
+class RemoveNameUniqueIndexFromOrganization < ActiveRecord::Migration[7.1]
+  def change
+    remove_index :organizations, :name
+  end
+end

server/spec/interactors/authentication/signup_spec.rb

@@ -125,16 +125,31 @@
           }
         end
 
-        it "fails" do
-          expect(context).to be_failure
+        it "succeeds" do
+          expect(context).to be_success
+        end
+
+        it "creates and does not confirm the user" do
+          expect { context }.to change(User, :count).by(1)
+          user = User.find_by(email: params[:email])
+          expect(user).not_to be_nil
+          expect(user.confirmed_at).to be_nil
+        end
+
+        it "creates a new user" do
+          expect { context }.to change(User, :count).by(1)
+        end
+
+        it "creates a new organization" do
+          expect { context }.to change(Organization, :count).by(1)
         end
 
-        it "does not create a new user" do
-          expect { context }.not_to change(User, :count)
+        it "creates a new workspace" do
+          expect { context }.to change(Workspace, :count).by(1)
         end
 
-        it "provides error messages related to company name" do
-          expect(context.errors).to eq("Signup failed: Company name has already been taken")
+        it "returns a success message" do
+          expect(context.message).to eq("Signup successful! Please check your email to confirm your account.")
         end
       end
     end

server/spec/models/organization_spec.rb

@@ -13,7 +13,7 @@
   # Test validations
   describe "validations" do
     it { should validate_presence_of(:name) }
-    it { should validate_uniqueness_of(:name).case_insensitive }
+    # it { should validate_uniqueness_of(:name).case_insensitive }
     # Add other validations here
   end