[Fix #8617] Style/HashAsLastArrayItem no longer flags hashes if all elements in an array are hashes

CHANGELOG.md

@@ -10,6 +10,7 @@
 ### Changes
 
 * [#8470](https://github.com/rubocop-hq/rubocop/issues/8470): Do not autocorrect `Style/StringConcatenation` when parts of the expression are too complex. ([@dvandersluis][])
+* [#8617](https://github.com/rubocop-hq/rubocop/issues/8617): Fix `Style/HashAsLastArrayItem` to not register an offense when all items in an array are hashes. ([@dvandersluis][])
 
 ## 0.90.0 (2020-09-01)
 

docs/modules/ROOT/pages/cops_style.adoc

@@ -3691,6 +3691,9 @@ ok
 Checks for presence or absence of braces around hash literal as a last
 array item depending on configuration.
 
+NOTE: This cop will ignore arrays where all items are hashes, regardless of
+EnforcedStyle.
+
 === Examples
 
 ==== EnforcedStyle: braces (default)
@@ -3702,6 +3705,9 @@ array item depending on configuration.
 
 # good
 [1, 2, { one: 1, two: 2 }]
+
+# good
+[{ one: 1 }, { two: 2 }]
 ----
 
 ==== EnforcedStyle: no_braces
@@ -3713,6 +3719,9 @@ array item depending on configuration.
 
 # good
 [1, 2, one: 1, two: 2]
+
+# good
+[{ one: 1 }, { two: 2 }]
 ----
 
 === Configurable attributes

lib/rubocop/cop/style/hash_as_last_array_item.rb

@@ -6,20 +6,28 @@ module Style
       # Checks for presence or absence of braces around hash literal as a last
       # array item depending on configuration.
       #
+      # NOTE: This cop will ignore arrays where all items are hashes, regardless of
+      # EnforcedStyle.
+      #
       # @example EnforcedStyle: braces (default)
       #   # bad
       #   [1, 2, one: 1, two: 2]
       #
       #   # good
       #   [1, 2, { one: 1, two: 2 }]
       #
+      #   # good
+      #   [{ one: 1 }, { two: 2 }]
+      #
       # @example EnforcedStyle: no_braces
       #   # bad
       #   [1, 2, { one: 1, two: 2 }]
       #
       #   # good
       #   [1, 2, one: 1, two: 2]
       #
+      #   # good
+      #   [{ one: 1 }, { two: 2 }]
       class HashAsLastArrayItem < Base
         include ConfigurableEnforcedStyle
         extend AutoCorrector
@@ -38,9 +46,10 @@ def on_hash(node)
 
         def last_array_item?(node)
           parent = node.parent
-          return false unless parent
+          return false unless parent&.array_type?
+          return false if parent.child_nodes.all?(&:hash_type?)
 
-          parent.array_type? && parent.children.last.equal?(node)
+          parent.children.last.equal?(node)
         end
 
         def check_braces(node)

spec/rubocop/cop/style/hash_as_last_array_item_spec.rb

@@ -30,6 +30,12 @@
         foo(one: 1, two: 2)
       RUBY
     end
+
+    it 'does not register an offense when the array is all hashes' do
+      expect_no_offenses(<<~RUBY)
+        [{ one: 1 }, { two: 2 }]
+      RUBY
+    end
   end
 
   context 'when EnforcedStyle is no_braces' do
@@ -59,5 +65,11 @@
         foo({ one: 1, two: 2 })
       RUBY
     end
+
+    it 'does not register an offense when the array is all hashes' do
+      expect_no_offenses(<<~RUBY)
+        [{ one: 1 }, { two: 2 }]
+      RUBY
+    end
   end
 end