[Fix rubocop#2331] Do not register an offense for count with an argum…

…ent in Performance/Size

CHANGELOG.md

@@ -36,6 +36,7 @@
 * [#2324](https://github.com/bbatsov/rubocop/issues/2324): Handle `--only Lint/Syntax` and `--except Lint/Syntax` correctly. ([@jonas054][])
 * [#2317](https://github.com/bbatsov/rubocop/issues/2317): Handle `case` as an argument correctly in `Lint/EndAlignment`. ([@lumeet][])
 * [#2287](https://github.com/bbatsov/rubocop/issues/2287): Fix auto-correct of lines with only whitespace in `Style/IndentationWidth`. ([@lumeet][])
+* [#2331](https://github.com/bbatsov/rubocop/issues/2331): Do not register an offense in `Performance/Size` for `count` with an argument. ([@rrosenblum][])
 
 ## 0.34.2 (21/09/2015)
 

lib/rubocop/cop/performance/size.rb

@@ -33,7 +33,7 @@ def on_send(node)
           return unless method == :count
           return unless array?(receiver) || hash?(receiver)
           return if node.parent && node.parent.block_type?
-          return if args && args.block_pass_type?
+          return if args
 
           add_offense(node, node.loc.selector)
         end

spec/rubocop/cop/performance/size_spec.rb

@@ -62,6 +62,12 @@
       expect(cop.offenses).to be_empty
     end
 
+    it 'does not register an offense when calling count with an argument' do
+      inspect_source(cop, '[1, 2, 3].count(1)')
+
+      expect(cop.offenses).to be_empty
+    end
+
     it 'corrects count to size' do
       new_source = autocorrect_source(cop, '[1, 2, 3].count')
 
@@ -124,6 +130,12 @@
       expect(cop.offenses).to be_empty
     end
 
+    it 'does not register an offense when calling count with an argument' do
+      inspect_source(cop, '{a: 1, b: 2, c: 3}.count(1)')
+
+      expect(cop.offenses).to be_empty
+    end
+
     it 'corrects count to size' do
       new_source = autocorrect_source(cop, '{a: 1, b: 2, c: 3}.count')