Add select! and reject! to the list InverseMethods will check for

rubocop · Feb 24, 2017 · 905adb4 · 905adb4
1 parent 0c9ed7c
commit 905adb4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/config/default.yml b/config/default.yml
@@ -721,6 +721,7 @@ Style/InverseMethods:
   # of the block that is passed to the method
   InverseBlocks:
     :select: :reject
+    :select!: :reject!
 
 Style/Lambda:
   EnforcedStyle: line_count_dependent

diff --git a/spec/rubocop/cop/style/inverse_methods_spec.rb b/spec/rubocop/cop/style/inverse_methods_spec.rb
@@ -17,7 +17,8 @@
           :> => :<=
         },
         'InverseBlocks' => {
-          select: :reject
+          select: :reject,
+          select!: :reject!
         }
       }
     )
@@ -147,7 +148,9 @@
 
   context 'inverse blocks' do
     { select: :reject,
-      reject: :select }.each do |method, inverse|
+      reject: :select,
+      select!: :reject!,
+      reject!: :select! }.each do |method, inverse|
       it "registers an offense for foo.#{method} { |e| !e }" do
         inspect_source(cop, "foo.#{method} { |e| !e }")