remove_concurrent_index.rb 927 Bytes
Newer Older
1 2
# frozen_string_literal: true

3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20
require_relative '../../migration_helpers'

module RuboCop
  module Cop
    module Migration
      # Cop that checks if `remove_concurrent_index` is used with `up`/`down` methods
      # and not `change`.
      class RemoveConcurrentIndex < RuboCop::Cop::Cop
        include MigrationHelpers

        MSG = '`remove_concurrent_index` is not reversible so you must manually define ' \
          'the `up` and `down` methods in your migration class, using `add_concurrent_index` in `down`'.freeze

        def on_send(node)
          return unless in_migration?(node)
          return unless node.children[1] == :remove_concurrent_index

          node.each_ancestor(:def) do |def_node|
Rémy Coutable's avatar
Rémy Coutable committed
21
            add_offense(def_node, location: :name) if method_name(def_node) == :change
22 23 24 25 26 27 28 29 30 31
          end
        end

        def method_name(node)
          node.children[0]
        end
      end
    end
  end
end