Autofix Rubocop Style/RaiseArgs (#23650)
parent
6ed6c41724
commit
593bb8ce99
|
@ -3470,15 +3470,6 @@ Style/PreferredHashMethods:
|
||||||
Exclude:
|
Exclude:
|
||||||
- 'spec/support/matchers/model/model_have_error_on_field.rb'
|
- 'spec/support/matchers/model/model_have_error_on_field.rb'
|
||||||
|
|
||||||
# Offense count: 2
|
|
||||||
# This cop supports safe autocorrection (--autocorrect).
|
|
||||||
# Configuration parameters: EnforcedStyle, AllowedCompactTypes.
|
|
||||||
# SupportedStyles: compact, exploded
|
|
||||||
Style/RaiseArgs:
|
|
||||||
Exclude:
|
|
||||||
- 'db/migrate/20200620164023_add_fixed_lowercase_index_to_accounts.rb'
|
|
||||||
- 'db/migrate/20210421121431_add_case_insensitive_btree_index_to_tags.rb'
|
|
||||||
|
|
||||||
# Offense count: 5
|
# Offense count: 5
|
||||||
# This cop supports unsafe autocorrection (--autocorrect-all).
|
# This cop supports unsafe autocorrection (--autocorrect-all).
|
||||||
# Configuration parameters: Methods.
|
# Configuration parameters: Methods.
|
||||||
|
|
|
@ -16,7 +16,7 @@ class AddFixedLowercaseIndexToAccounts < ActiveRecord::Migration[5.2]
|
||||||
add_index :accounts, "lower (username), COALESCE(lower(domain), '')", name: 'index_accounts_on_username_and_domain_lower', unique: true, algorithm: :concurrently
|
add_index :accounts, "lower (username), COALESCE(lower(domain), '')", name: 'index_accounts_on_username_and_domain_lower', unique: true, algorithm: :concurrently
|
||||||
rescue ActiveRecord::RecordNotUnique
|
rescue ActiveRecord::RecordNotUnique
|
||||||
remove_index :accounts, name: 'index_accounts_on_username_and_domain_lower'
|
remove_index :accounts, name: 'index_accounts_on_username_and_domain_lower'
|
||||||
raise CorruptionError.new('index_accounts_on_username_and_domain_lower')
|
raise CorruptionError, 'index_accounts_on_username_and_domain_lower'
|
||||||
end
|
end
|
||||||
|
|
||||||
remove_index :accounts, name: 'old_index_accounts_on_username_and_domain_lower' if index_name_exists?(:accounts, 'old_index_accounts_on_username_and_domain_lower')
|
remove_index :accounts, name: 'old_index_accounts_on_username_and_domain_lower' if index_name_exists?(:accounts, 'old_index_accounts_on_username_and_domain_lower')
|
||||||
|
|
|
@ -10,7 +10,7 @@ class AddCaseInsensitiveBtreeIndexToTags < ActiveRecord::Migration[5.2]
|
||||||
safety_assured { execute 'CREATE UNIQUE INDEX CONCURRENTLY index_tags_on_name_lower_btree ON tags (lower(name) text_pattern_ops)' }
|
safety_assured { execute 'CREATE UNIQUE INDEX CONCURRENTLY index_tags_on_name_lower_btree ON tags (lower(name) text_pattern_ops)' }
|
||||||
rescue ActiveRecord::StatementInvalid => e
|
rescue ActiveRecord::StatementInvalid => e
|
||||||
remove_index :tags, name: 'index_tags_on_name_lower_btree'
|
remove_index :tags, name: 'index_tags_on_name_lower_btree'
|
||||||
raise CorruptionError.new('index_tags_on_name_lower_btree') if e.is_a?(ActiveRecord::RecordNotUnique)
|
raise CorruptionError, 'index_tags_on_name_lower_btree' if e.is_a?(ActiveRecord::RecordNotUnique)
|
||||||
raise e
|
raise e
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue