rubocop/cop/migration/add_concurrent_index.rb


1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36

# frozen_string_literal: true

require_relative '../../migration_helpers'

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

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

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

          name = node.children[1]

          return unless name == :add_concurrent_index

          node.each_ancestor(:def) do |def_node|
            next unless method_name(def_node) == :change

            add_offense(def_node, location: :name)
          end
        end

        def method_name(node)
          node.children.first
        end
      end
    end
  end
end