From e6aaa5f189a948fe6c74cac21ed28bba31b820b9 Mon Sep 17 00:00:00 2001
From: Denys Kurets <denys281@gmail.com>
Date: Tue, 10 Sep 2019 15:45:53 +0300
Subject: [PATCH] Fix #7320 support Naming/MethodName for attr_reader /
 attr_writer / attr_accessor

---
 CHANGELOG.md                                |   2 +
 lib/rubocop/cop/naming/method_name.rb       |  25 +++++
 spec/rubocop/cop/naming/method_name_spec.rb | 114 ++++++++++++++++++++
 3 files changed, 141 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 4fafc510f0b..2b88c5a081f 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -86,6 +86,7 @@
 * [#7091](https://github.com/rubocop-hq/rubocop/issues/7091): `Style/FormatStringToken` now detects format sequences with flags and modifiers. ([@buehmann][])
 * [#7319](https://github.com/rubocop-hq/rubocop/pull/7319): Rename `IgnoredMethodPatterns` option to `IgnoredPatterns` option for `Style/MethodCallWithArgsParentheses`. ([@koic][])
 * [#7345](https://github.com/rubocop-hq/rubocop/issues/7345): Mark unsafe for `Style/YodaCondition`. ([@koic][])
+* [#7320](https://github.com/rubocop-hq/rubocop/issues/7320): Naming/MethodName doesn't flag attr_reader / attr_writer / attr_accessor. ([@denys281][])
 
 ## 0.74.0 (2019-07-31)
 
@@ -4248,3 +4249,4 @@
 [@jdkaplan]: https://github.com/jdkaplan
 [@cstyles]: https://github.com/cstyles
 [@avmnu-sng]: https://github.com/avmnu-sng
+[@denys281]: https://github.com/denys281
diff --git a/lib/rubocop/cop/naming/method_name.rb b/lib/rubocop/cop/naming/method_name.rb
index e5be2dcdf70..eda3e5d6ea2 100644
--- a/lib/rubocop/cop/naming/method_name.rb
+++ b/lib/rubocop/cop/naming/method_name.rb
@@ -31,9 +31,27 @@ module Naming
       class MethodName < Cop
         include ConfigurableNaming
         include IgnoredPattern
+        include RangeHelp
 
         MSG = 'Use %<style>s for method names.'
 
+        def_node_matcher :attr?, <<~PATTERN
+          (send nil? ${:attr_reader :attr_writer :attr_accessor :attr} $...)
+        PATTERN
+
+        def_node_matcher :sym_name, '(sym $_name)'
+
+        def on_send(node)
+          return unless (attrs = attr?(node))
+
+          attrs.last.each do |name_item|
+            name = sym_name(name_item)
+            next if !name || matches_ignored_pattern?(name)
+
+            check_name(node, name, range_position(node))
+          end
+        end
+
         def on_def(node)
           return if node.operator_method? ||
                     matches_ignored_pattern?(node.method_name)
@@ -44,6 +62,13 @@ def on_def(node)
 
         private
 
+        def range_position(node)
+          selector_end_pos = node.loc.selector.end_pos + 1
+          expr_end_pos = node.loc.expression.end_pos
+
+          range_between(selector_end_pos, expr_end_pos)
+        end
+
         def message(style)
           format(MSG, style: style)
         end
diff --git a/spec/rubocop/cop/naming/method_name_spec.rb b/spec/rubocop/cop/naming/method_name_spec.rb
index 10e6b7ca2ae..2f4f32843a1 100644
--- a/spec/rubocop/cop/naming/method_name_spec.rb
+++ b/spec/rubocop/cop/naming/method_name_spec.rb
@@ -4,6 +4,29 @@
   subject(:cop) { described_class.new(config) }
 
   shared_examples 'never accepted' do |enforced_style|
+    it 'registers an offense for mixed snake case and camel case in attr.' do
+      expect_offense(<<~RUBY)
+        attr :visit_Arel_Nodes_SelectStatement
+             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Use #{enforced_style} for method names.
+
+        attr_reader :visit_Arel_Nodes_SelectStatement
+                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Use #{enforced_style} for method names.
+
+        attr_accessor :visit_Arel_Nodes_SelectStatement
+                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Use #{enforced_style} for method names.
+
+        attr_writer :visit_Arel_Nodes_SelectStatement
+                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Use #{enforced_style} for method names.
+      RUBY
+    end
+
+    it 'registers an offense for mixed snake case and camel case in attr.' do
+      expect_offense(<<~RUBY)
+        attr_reader :visit_Arel_Nodes_SelectStatement
+                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Use #{enforced_style} for method names.
+      RUBY
+    end
+
     it 'registers an offense for mixed snake case and camel case' do
       expect_offense(<<~RUBY)
         def visit_Arel_Nodes_SelectStatement
@@ -12,6 +35,22 @@ def visit_Arel_Nodes_SelectStatement
       RUBY
     end
 
+    it 'registers an offense for capitalized camel case name in attr.' do
+      expect_offense(<<~RUBY)
+        attr :MyMethod
+             ^^^^^^^^^ Use #{enforced_style} for method names.
+
+        attr_reader :MyMethod
+                    ^^^^^^^^^ Use #{enforced_style} for method names.
+
+        attr_accessor :MyMethod
+                      ^^^^^^^^^ Use #{enforced_style} for method names.
+
+        attr_writer :MyMethod
+                    ^^^^^^^^^ Use #{enforced_style} for method names.
+      RUBY
+    end
+
     it 'registers an offense for capitalized camel case' do
       expect_offense(<<~RUBY)
         class MyClass
@@ -100,6 +139,14 @@ def base.Start(aws_env, *args)
         }
       end
 
+      it 'does not register an offense for camel case method name in attr.' do
+        expect_no_offenses(<<~RUBY)
+          attr_reader :onSelectionBulkChange
+          attr_accessor :onSelectionBulkChange
+          attr_writer :onSelectionBulkChange
+        RUBY
+      end
+
       it 'does not register an offense for camel case method name ' \
          ' matching `IgnoredPatterns`' do
         expect_no_offenses(<<~RUBY)
@@ -108,6 +155,14 @@ def onSelectionBulkChange(arg)
         RUBY
       end
 
+      it 'does not register an offense for snake case method name in attr.' do
+        expect_no_offenses(<<~RUBY)
+          attr_reader :on_selection_cleared
+          attr_accessor :on_selection_cleared
+          attr_writer :on_selection_cleared
+        RUBY
+      end
+
       it 'does not register an offense for snake case method name ' \
          ' matching `IgnoredPatterns`' do
         expect_no_offenses(<<~RUBY)
@@ -118,9 +173,37 @@ def on_selection_cleared(arg)
     end
   end
 
+  shared_examples 'multiple attr methods' do |enforced_style|
+    it 'registers an offense for camel case methods names in attr.' do
+       expect_offense(<<~RUBY)
+        attr :my_method, :myMethod
+             ^^^^^^^^^^^^^^^^^^^^^ Use #{enforced_style} for method names.
+
+        attr_reader :my_method, :myMethod
+                    ^^^^^^^^^^^^^^^^^^^^^ Use #{enforced_style} for method names.
+
+        attr_accessor :myMethod, :my_method
+                      ^^^^^^^^^^^^^^^^^^^^^ Use #{enforced_style} for method names.
+      RUBY
+    end
+  end
+
   context 'when configured for snake_case' do
     let(:cop_config) { { 'EnforcedStyle' => 'snake_case' } }
 
+    it 'registers an offense for camel case method names in attr.' do
+      expect_offense(<<~RUBY)
+        attr_reader :myMethod
+                    ^^^^^^^^^ Use snake_case for method names.
+
+        attr_accessor :myMethod
+                      ^^^^^^^^^ Use snake_case for method names.
+
+        attr_writer :myMethod
+                    ^^^^^^^^^ Use snake_case for method names.
+      RUBY
+    end
+
     it 'registers an offense for camel case in instance method name' do
       expect_offense(<<~RUBY)
         def myMethod
@@ -149,6 +232,14 @@ def self.myMethod
       RUBY
     end
 
+    it 'accepts snake case in attr.' do
+      expect_no_offenses(<<~RUBY)
+        attr_reader :my_method
+        attr_accessor :my_method
+        attr_writer :my_method
+      RUBY
+    end
+
     it 'accepts snake case in names' do
       expect_no_offenses(<<~RUBY)
         def my_method
@@ -168,11 +259,20 @@ def self.fooBar
 
     include_examples 'never accepted',  'snake_case'
     include_examples 'always accepted', 'snake_case'
+    include_examples 'multiple attr methods', 'snake_case'
   end
 
   context 'when configured for camelCase' do
     let(:cop_config) { { 'EnforcedStyle' => 'camelCase' } }
 
+    it 'accepts camel case names in attr.' do
+      expect_no_offenses(<<~RUBY)
+        attr_reader :myMethod
+        attr_accessor :myMethod
+        attr_writer :myMethod
+      RUBY
+    end
+
     it 'accepts camel case in instance method name' do
       expect_no_offenses(<<~RUBY)
         def myMethod
@@ -189,6 +289,19 @@ def self.myMethod
       RUBY
     end
 
+    it 'registers an offense for snake case name in attr.' do
+      expect_offense(<<~RUBY)
+        attr_reader :my_method
+                    ^^^^^^^^^^ Use camelCase for method names.
+
+        attr_accessor :my_method
+                      ^^^^^^^^^^ Use camelCase for method names.
+
+        attr_writer :my_method
+                    ^^^^^^^^^^ Use camelCase for method names.
+      RUBY
+    end
+
     it 'registers an offense for snake case in names' do
       expect_offense(<<~RUBY)
         def my_method
@@ -219,5 +332,6 @@ def self.foo_bar
 
     include_examples 'always accepted', 'camelCase'
     include_examples 'never accepted',  'camelCase'
+    include_examples 'multiple attr methods', 'camelCase'
   end
 end