blob: b5724ef1efd1362527dcece3f20ec0dbf02c4808 (
plain)
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
48
49
50
51
52
53
54
55
56
57
58
59
|
# frozen_string_literal: true
require 'fast_spec_helper'
require_relative '../../../../rubocop/cop/migration/background_migration_record'
RSpec.describe RuboCop::Cop::Migration::BackgroundMigrationRecord do
subject(:cop) { described_class.new }
context 'outside of a migration' do
it 'does not register any offenses' do
expect_no_offenses(<<~SOURCE)
class MigrateProjectRecords
class Project < ActiveRecord::Base
end
end
SOURCE
end
end
context 'in migration' do
before do
allow(cop).to receive(:in_background_migration?).and_return(true)
end
it 'adds an offense if inheriting from ActiveRecord::Base' do
expect_offense(<<~RUBY)
class MigrateProjectRecords
class Project < ActiveRecord::Base
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Don't use or inherit from ActiveRecord::Base.[...]
end
end
RUBY
end
it 'adds an offense if create dynamic model from ActiveRecord::Base' do
expect_offense(<<~RUBY)
class MigrateProjectRecords
def define_model(table_name)
Class.new(ActiveRecord::Base) do
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Don't use or inherit from ActiveRecord::Base.[...]
self.table_name = table_name
self.inheritance_column = :_type_disabled
end
end
end
RUBY
end
it 'adds an offense if inheriting from ::ActiveRecord::Base' do
expect_offense(<<~RUBY)
class MigrateProjectRecords
class Project < ::ActiveRecord::Base
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Don't use or inherit from ActiveRecord::Base.[...]
end
end
RUBY
end
end
end
|