Class: RuboCop::Cop::Neeto::UnsafeTableDeletion

Inherits:
Base
  • Object
show all
Defined in:
lib/rubocop/cop/neeto/unsafe_table_deletion.rb

Overview

‘drop_table` is considered as a dangerous operation. If not used correctly, it could cause irreversible data loss. It is advised to rename the table to `table_name_deprecated_on_yyyy_mm_dd` instead. The renamed table can be safely dropped in a later migration. The cop permits dropping tables that are named in the above format.

Examples:

UnsafeTableDeletion: true (default)

# Enforces the usage of `rename_table` over `drop_table`.

# bad
drop_table :users

# bad
drop_table :users do |t|
  t.string :email, null: false
  t.string :first_name, null: false
end

# good
drop_table :users_deprecated_on_2024_08_09

# good
drop_table :users_deprecated_on_2024_08_09 do |t|
  t.string :email, null: false
  t.string :first_name, null: false
end

Constant Summary collapse

SAFE_TABLE_NAME_REGEX = 
/\w+_deprecated_on_\d{4}_\d{2}_\d{2}/
CURRENT_DATE = 
DateTime.now.strftime("%Y_%m_%d")
MSG = 
"'drop_table' is a dangerous operation. If not used correctly, " \
"it could cause irreversible data loss. You must perform " \
"'rename_table :%{table_name}, :%{table_name}_deprecated_on_#{CURRENT_DATE}' " \
"instead. The renamed table can be safely dropped in a future migration."
RESTRICT_ON_SEND = 
%i[drop_table].freeze

Instance Method Summary collapse

Instance Method Details

#on_send(node) ⇒ Object 



48
49
50
51
52
53
54
55
 
# File 'lib/rubocop/cop/neeto/unsafe_table_deletion.rb', line 48

def on_send(node)
  return unless unsafe_drop_table?(node)

  unsafe_drop_table?(node) do |table_name|
    message = format(MSG, table_name:)
    add_offense(node, message:) unless SAFE_TABLE_NAME_REGEX.match?(table_name)
  end
end