rename cop to be more specific

wealthsimple · Jul 8, 2017 · a6257af · a6257af
1 parent c74646f
commit a6257af
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/lib/rubocop-migration.rb b/lib/rubocop-migration.rb
@@ -8,7 +8,7 @@
 
 require "rubocop/migration/version"
 require "rubocop/migration/strong_migrations_checker"
-require "rubocop/cop/migration/unsafe_operation"
+require "rubocop/cop/migration/unsafe_migration"
 
 module RuboCop
   module Migration

diff --git a/...rubocop/cop/migration/unsafe_operation.rb → ...rubocop/cop/migration/unsafe_migration.rb b/...rubocop/cop/migration/unsafe_operation.rb → ...rubocop/cop/migration/unsafe_migration.rb
@@ -1,15 +1,15 @@
 module RuboCop
   module Cop
     module Migration
-      class UnsafeOperation < Cop
+      class UnsafeMigration < Cop
         # List of all public methods that can be used within a migration method,
         # like `add_index`, `rename_table`, etc.
         SCHEMA_STATEMENTS = ActiveRecord::ConnectionAdapters::SchemaStatements
           .public_instance_methods
           .freeze
         SCHEMA_STATEMENTS_PATTERN = SCHEMA_STATEMENTS.map { |s| ":#{s}" }.join(" ")
 
-        ERROR_NOTICE = "To ignore this warning, add `# rubocop:disable UnsafeOperation` above and `# rubocop:enable UnsafeOperation` below your code."
+        ERROR_NOTICE = "To ignore this warning, add `# rubocop:disable UnsafeMigration` above and `# rubocop:enable UnsafeMigration` below your code."
 
         # Handle `ActiveRecord::Migration` and `ActiveRecord::Migration[5.0]`
         def_node_matcher :migration_class?, <<-PATTERN

diff --git a/...op/cop/migration/unsafe_operation_spec.rb → ...op/cop/migration/unsafe_migration_spec.rb b/...op/cop/migration/unsafe_operation_spec.rb → ...op/cop/migration/unsafe_migration_spec.rb
@@ -1,4 +1,4 @@
-describe RuboCop::Cop::Migration::UnsafeOperation do
+describe RuboCop::Cop::Migration::UnsafeMigration do
   subject(:cop) { described_class.new }
 
   before do