Squiz.Formatting.OperatorBracket no longer reports false positives in match() structures

Author: gsherwood

package.xml

@@ -30,6 +30,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
     -- Thanks to Gary Jones (@GaryJones) for the patch
   - Generic.PHP.LowerCaseType sniff now correctly examines types inside arrow functions
     -- Thanks to Juliette Reinders Folmer (@jrfnl) for the patch
+  - Squiz.Formatting.OperatorBracket no longer reports false positives in match() structures
   - Fixed bug #3616 : Squiz.PHP.DisallowComparisonAssignment false positive for PHP 8 match expression
     -- Thanks to Juliette Reinders Folmer (@jrfnl) for the patch
   - Fixed bug #3618 : Generic.WhiteSpace.ArbitraryParenthesesSpacing false positive for return new parent()

src/Standards/Squiz/Sniffs/Formatting/OperatorBracketSniff.php

@@ -109,6 +109,7 @@ public function process(File $phpcsFile, $stackPtr)
                 T_OPEN_SHORT_ARRAY    => true,
                 T_CASE                => true,
                 T_EXIT                => true,
+                T_MATCH_ARROW         => true,
             ];
 
             if (isset($invalidTokens[$tokens[$previousToken]['code']]) === true) {

src/Standards/Squiz/Tests/Formatting/OperatorBracketUnitTest.inc

@@ -193,3 +193,9 @@ $expr = match (true) {
 if ($pos === count(value: $this->tokens) - 1) {
     $file = '...'.substr(string: $file, offset: $padding * -1 + 3);
 }
+
+match ($a) {
+    'a' => -1,
+    'b', 'c', 'd' => -2,
+    default => -3,
+};

src/Standards/Squiz/Tests/Formatting/OperatorBracketUnitTest.inc.fixed

@@ -193,3 +193,9 @@ $expr = match (true) {
 if ($pos === (count(value: $this->tokens) - 1)) {
     $file = '...'.substr(string: $file, offset: ($padding * -1 + 3));
 }
+
+match ($a) {
+    'a' => -1,
+    'b', 'c', 'd' => -2,
+    default => -3,
+};