2017-02-08 17:17:15 -05:00
|
|
|
require_relative '../../migration_helpers'
|
|
|
|
|
|
|
|
module RuboCop
|
|
|
|
module Cop
|
|
|
|
module Migration
|
|
|
|
# Cop that checks if `add_column_with_default` is used with `up`/`down` methods
|
|
|
|
# and not `change`.
|
2017-04-24 13:16:33 -04:00
|
|
|
class ReversibleAddColumnWithDefault < RuboCop::Cop::Cop
|
2017-02-08 17:17:15 -05:00
|
|
|
include MigrationHelpers
|
|
|
|
|
2017-04-24 13:14:09 -04:00
|
|
|
def_node_matcher :add_column_with_default?, <<~PATTERN
|
|
|
|
(send nil :add_column_with_default $...)
|
|
|
|
PATTERN
|
|
|
|
|
|
|
|
def_node_matcher :defines_change?, <<~PATTERN
|
|
|
|
(def :change ...)
|
|
|
|
PATTERN
|
|
|
|
|
2017-02-08 17:17:15 -05:00
|
|
|
MSG = '`add_column_with_default` is not reversible so you must manually define ' \
|
2017-02-21 18:32:18 -05:00
|
|
|
'the `up` and `down` methods in your migration class, using `remove_column` in `down`'.freeze
|
2017-02-08 17:17:15 -05:00
|
|
|
|
|
|
|
def on_send(node)
|
|
|
|
return unless in_migration?(node)
|
2017-04-24 13:14:09 -04:00
|
|
|
return unless add_column_with_default?(node)
|
2017-02-08 17:17:15 -05:00
|
|
|
|
|
|
|
node.each_ancestor(:def) do |def_node|
|
2017-04-24 13:14:09 -04:00
|
|
|
next unless defines_change?(def_node)
|
2017-02-08 17:17:15 -05:00
|
|
|
|
|
|
|
add_offense(def_node, :name)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|