spec/rubocop/cop/migration/hash_index_spec.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
37
38
39
40
41
42
43
44
45
46
47

# frozen_string_literal: true

require 'fast_spec_helper'
require_relative '../../../../rubocop/cop/migration/hash_index'

RSpec.describe RuboCop::Cop::Migration::HashIndex do
  subject(:cop) { described_class.new }

  context 'when in migration' do
    before do
      allow(cop).to receive(:in_migration?).and_return(true)
    end

    it 'registers an offense when creating a hash index' do
      expect_offense(<<~RUBY)
        def change
          add_index :table, :column, using: :hash
                                     ^^^^^^^^^^^^ hash indexes should be avoided at all costs[...]
        end
      RUBY
    end

    it 'registers an offense when creating a concurrent hash index' do
      expect_offense(<<~RUBY)
        def change
          add_concurrent_index :table, :column, using: :hash
                                                ^^^^^^^^^^^^ hash indexes should be avoided at all costs[...]
        end
      RUBY
    end

    it 'registers an offense when creating a hash index using t.index' do
      expect_offense(<<~RUBY)
        def change
          t.index :table, :column, using: :hash
                                   ^^^^^^^^^^^^ hash indexes should be avoided at all costs[...]
        end
      RUBY
    end
  end

  context 'when outside of migration' do
    it 'registers no offense' do
      expect_no_offenses('def change; index :table, :column, using: :hash; end')
    end
  end
end