rubocop-nueca 2.1.0 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 81918c1ee3c1d72d3bfdf6914398dab599378c4a37ae66419afb23792c35386e
-  data.tar.gz: a4348608537bc0876333bb78e4757770f91859623e83c3353b013e4d7233f1a2
+  metadata.gz: 9e2e8263941b4c5f0934ad7fb9f8df991e90583476f10fc3290a3c7dd2f917ff
+  data.tar.gz: 0e503986f5bfd5003b40beee3844cfeb9ae5cdc179b2d4576d139e1d244ba18f
 SHA512:
-  metadata.gz: 3e41c08ebed4ab33a3b41227cc97d1506d3c289aedf7468f8d8925d8b0ef792f8780d5a510aade458dfd08624bdb36aee4975d63c9689c68f8f9283a0dc7e679
-  data.tar.gz: a0c05dfcf12646d2c881bfa80f87d70bddfed41916b317f2544823b47b93a71233be42331b7926c9a41be39211dfed96cfaa9f07d7b723df18faee1cd41a87a9
+  metadata.gz: dc56766f7ecaa64cb93001e8ee3cd4eca0250bbf4b509d41f2fb6bcbe36d3d30e1d34dba4bdcdbcd26a4f8cd143600000364c5b9ad184793b99bc24f3992373e
+  data.tar.gz: 1e89c75c3f19fea9dc29979dd38af0695f4ea4e77f126e7d3d0bc407853f2a3e61baef1f9862016101ee280775bf4ca0c1a24f5cac2f1a48c4418d8cb4c03f05

data/README.md

@@ -18,7 +18,7 @@ Add to your `.rubocop.yml` the following code.
 
 ```yaml
 plugins:
-    - rubocop-nueca
+  - rubocop-nueca
 ```
 
 ## License

data/lib/rubocop/cop/rails/route_unnecessary_array_notation.rb

@@ -0,0 +1,72 @@
+# frozen_string_literal: true
+
+require_relative '../shared/route_helper'
+
+module RuboCop
+  module Cop
+    module Rails
+      class RouteUnnecessaryArrayNotation < RuboCop::Cop::Base
+        include RouteHelper
+        extend AutoCorrector
+
+        MSG = 'Unnecessary array notation for single element. ' \
+              'Use `%<key>s: %<value>s` instead of `%<key>s: [%<value>s]`.'
+
+        def on_send(node)
+          return unless route_file?
+          return unless route_method?(node)
+
+          node.arguments.each do |arg|
+            next unless arg.hash_type?
+
+            check_hash_pairs(arg)
+          end
+        end
+
+        private
+
+        def route_method?(node)
+          return false unless node.send_type?
+
+          route_methods = [
+            :get, :post, :put, :patch, :delete, :head, :options, :match, :root,
+            :resource, :resources,
+            :namespace, :scope, :concern, :member, :collection,
+            :draw
+          ]
+
+          route_methods.include?(node.method_name)
+        end
+
+        def check_hash_pairs(hash_node)
+          hash_node.pairs.each do |pair|
+            check_single_element_array(pair)
+          end
+        end
+
+        def check_single_element_array(pair)
+          value = pair.value
+          return unless value.array_type?
+          return unless value.children.size == 1
+
+          element = value.children.first
+          return unless element.sym_type? || element.str_type?
+
+          key = pair.key
+          key_source = key.source
+          element_source = element.source
+
+          message = format(
+            MSG,
+            key: key_source,
+            value: element_source
+          )
+
+          add_offense(value, message: message) do |corrector|
+            corrector.replace(value, element_source)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/nueca/version.rb

@@ -2,6 +2,6 @@
 
 module RuboCop
   module Nueca
-    VERSION = '2.1.0'
+    VERSION = '2.2.0'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-nueca
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.2.0
 platform: ruby
 authors:
 - Tien
@@ -152,6 +152,7 @@ files:
 - lib/rubocop/cop/rails/route_root_position.rb
 - lib/rubocop/cop/rails/route_separation.rb
 - lib/rubocop/cop/rails/route_sorting.rb
+- lib/rubocop/cop/rails/route_unnecessary_array_notation.rb
 - lib/rubocop/cop/rails/time_zone_today.rb
 - lib/rubocop/cop/rswag/scattered_setup.rb
 - lib/rubocop/cop/shared/collection_context.rb