removes users.validation_status

app/models/user.rb

@@ -6,28 +6,16 @@ class User < ApplicationRecord
   has_many :projects, through: :assignments
   has_many :work_weeks, through: :assignments
 
-  VALIDATION_STATUS_PENDING = 'pending'.freeze
-  VALIDATION_STATUS_VALIDATED = 'validated'.freeze
-
   validates :name, presence: true
   validates :current_company, presence: true
   validates :email,
     presence: true,
     uniqueness: { case_sensitive: false },
     format: { with: URI::MailTo::EMAIL_REGEXP }
-  validates :validation_status, inclusion: { in: [VALIDATION_STATUS_VALIDATED, VALIDATION_STATUS_PENDING] }
 
   passwordless_with :email
 
   def owner?
     memberships.find_by(company: current_company).role == Membership::OWNER
   end
-
-  def email_validated?
-    validation_status == VALIDATION_STATUS_VALIDATED
-  end
-
-  def email_validation_pending?
-    validation_status == VALIDATION_STATUS_PENDING
-  end
 end

db/migrate/20240114005440_remove_users_validation_status.rb

@@ -0,0 +1,5 @@
+class RemoveUsersValidationStatus < ActiveRecord::Migration[7.1]
+  def change
+    remove_column :users, :validation_status
+  end
+end

spec/factories.rb

@@ -4,10 +4,6 @@
     name { Faker::Name.name + " #{rand(1..100).to_s}" }
     sequence(:email) { |n| "#{n}#{Faker::Internet.email}" }
 
-    trait(:needs_validation) do
-      validation_status { User::VALIDATION_STATUS_PENDING }
-    end
-
     after(:build) do |user, options|
       # a current company is required for the user to be valid
       # unless one is already set, create one

spec/models/user_spec.rb

@@ -50,20 +50,4 @@
       end
     end
   end
-
-  describe "#email_validated?" do
-    context "when user has a validated email address" do
-      it "returns true" do
-        user = build(:user, validation_status: User::VALIDATION_STATUS_VALIDATED)
-        expect(user.email_validated?).to be_truthy
-      end
-    end
-
-    context "when user does not have a validated email address" do
-      it "returns false" do
-        user = build(:user, validation_status: User::VALIDATION_STATUS_PENDING)
-        expect(user.email_validated?).to be_falsey
-      end
-    end
-  end
 end