Always use [] with aryptn

Author: kddnewton

lib/syntax_tree/node.rb

@@ -1123,38 +1123,20 @@ def deconstruct_keys(_keys)
     end
 
     def format(q)
-      parts = [*requireds]
-      parts << RestFormatter.new(rest) if rest
-      parts += posts
-
-      if constant
-        q.group do
-          q.format(constant)
-          q.text("[")
-          q.indent do
-            q.breakable("")
-            q.seplist(parts) { |part| q.format(part) }
-          end
-          q.breakable("")
-          q.text("]")
-        end
-
-        return
-      end
-
-      parent = q.parent
-      if parts.length == 1 || PATTERNS.include?(parent.class)
+      q.group do
+        q.format(constant) if constant
         q.text("[")
         q.indent do
           q.breakable("")
+
+          parts = [*requireds]
+          parts << RestFormatter.new(rest) if rest
+          parts += posts
+
           q.seplist(parts) { |part| q.format(part) }
         end
         q.breakable("")
         q.text("]")
-      elsif parts.empty?
-        q.text("[]")
-      else
-        q.group { q.seplist(parts) { |part| q.format(part) } }
       end
     end
   end

test/fixtures/aryptn.rb

@@ -6,51 +6,92 @@
 case foo
 in _, _
 end
+-
+case foo
+in [_, _]
+end
 %
 case foo
 in bar, baz
 end
+-
+case foo
+in [bar, baz]
+end
 %
 case foo
 in [bar]
 end
 %
 case foo
-in [bar, baz]
+in [bar]
+in [baz]
 end
--
+%
 case foo
-in bar, baz
+in [bar, baz]
 end
 %
 case foo
 in bar, *baz
 end
+-
+case foo
+in [bar, *baz]
+end
 %
 case foo
 in *bar, baz
 end
+-
+case foo
+in [*bar, baz]
+end
 %
 case foo
 in bar, *, baz
 end
+-
+case foo
+in [bar, *, baz]
+end
 %
 case foo
 in *, bar, baz
 end
+-
+case foo
+in [*, bar, baz]
+end
 %
 case foo
 in Constant[bar]
 end
 %
 case foo
+in Constant(bar)
+end
+-
+case foo
+in Constant[bar]
+end
+%
+case foo
 in Constant[bar, baz]
 end
 %
 case foo
 in bar, [baz, _] => qux
 end
+-
+case foo
+in [bar, [baz, _] => qux]
+end
 %
 case foo
 in bar, baz if bar == baz
 end
+-
+case foo
+in [bar, baz] if bar == baz
+end

test/fixtures/hshptn.rb

@@ -71,3 +71,8 @@
 in bar, { baz:, **nil }
 in qux:
 end
+-
+case foo
+in [bar, { baz:, **nil }]
+in qux:
+end

test/fixtures/in.rb

@@ -14,8 +14,10 @@
 end
 -
 case foo
-in fooooooooooooooooooooooooooooooooooooo,
-   barrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrr
+in [
+     fooooooooooooooooooooooooooooooooooooo,
+     barrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrrr
+   ]
   baz
 end
 %