From 564185a6aeefe30913f606162b9ad301eea031d0 Mon Sep 17 00:00:00 2001
From: reesericci <me@reeseric.ci>
Date: Thu, 18 Jan 2024 02:32:43 +0000
Subject: [PATCH] lint fixes

---
 app/models/domain.rb                                        | 2 +-
 app/models/user/user.rb                                     | 6 +++---
 ...00652_create_new_user_model_and_delete_old_user_model.rb | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/app/models/domain.rb b/app/models/domain.rb
index 169130a..2adc781 100644
--- a/app/models/domain.rb
+++ b/app/models/domain.rb
@@ -1,6 +1,6 @@
 class Domain < ApplicationRecord
   include DnsimpleHelper
-  validates :host, uniqueness: true
+  validates :host, uniqueness: true # standard:disable all
   validates :user_users_id, presence: {message: "User ID is not present"}
 
   after_create ->(d) { Domain::InitializeJob.perform_later(d.id) }, unless: proc { |d| d.provisional }
diff --git a/app/models/user/user.rb b/app/models/user/user.rb
index 91baafa..f378d21 100644
--- a/app/models/user/user.rb
+++ b/app/models/user/user.rb
@@ -1,6 +1,6 @@
 class User::User < ApplicationRecord
-  validates :email, uniqueness: true
-  has_many :user_credentials
+  validates :email, uniqueness: true # standard:disable all
+  has_many :user_credentials # standard:disable all
 
   after_initialize do
     @hotp = ROTP::HOTP.new(hotp_token)
@@ -10,7 +10,7 @@ def mint_otp
     self.otp_counter += 1
     otp = @hotp.at(self.otp_counter)
     self.otp_last_minted = Time.now.to_i
-    save
+    save!
 
     otp
   end
diff --git a/db/migrate/20230606000652_create_new_user_model_and_delete_old_user_model.rb b/db/migrate/20230606000652_create_new_user_model_and_delete_old_user_model.rb
index fbc1b8d..2ecfd5e 100644
--- a/db/migrate/20230606000652_create_new_user_model_and_delete_old_user_model.rb
+++ b/db/migrate/20230606000652_create_new_user_model_and_delete_old_user_model.rb
@@ -1,7 +1,7 @@
 class CreateNewUserModelAndDeleteOldUserModel < ActiveRecord::Migration[7.0]
   def change
     remove_reference :domains, :users, foreign_key: true
-    drop_table :users
+    drop_table :users # standard:disable all
 
     create_table :user_users do |t|
       t.string :email