ObjectDeclarations::getClassProperties(): allow for abstract final

PHPCSUtils/Utils/ObjectDeclarations.php

@@ -157,7 +157,6 @@ public static function getName(File $phpcsFile, $stackPtr)
      * - Bugs fixed:
      *   - Handling of PHPCS annotations.
      *   - Handling of unorthodox docblock placement.
-     *   - A class cannot both be abstract as well as final, so this utility should not allow for that.
      * - Defensive coding against incorrect calls to this method.
      *
      * @see \PHP_CodeSniffer\Files\File::getClassProperties()   Original source.
@@ -203,11 +202,11 @@ public static function getClassProperties(File $phpcsFile, $stackPtr)
             switch ($tokens[$i]['code']) {
                 case \T_ABSTRACT:
                     $properties['is_abstract'] = true;
-                    break 2;
+                    break;
 
                 case \T_FINAL:
                     $properties['is_final'] = true;
-                    break 2;
+                    break;
             }
         }
 

Tests/BackCompat/BCFile/GetClassPropertiesTest.inc

@@ -29,3 +29,10 @@ abstract
  * @phpcs:disable Standard.Cat.SniffName -- Just because.
  */
 class ClassWithDocblock {}
+
+/* testParseErrorAbstractFinal */
+final /* comment */
+
+    abstract // Intentional parse error, class cannot both be final and abstract.
+
+        class AbstractFinal {}

Tests/BackCompat/BCFile/GetClassPropertiesTest.php

@@ -138,6 +138,13 @@ public function dataGetClassProperties()
                     'is_final'    => false,
                 ],
             ],
+            'abstract-final-parse-error' => [
+                '/* testParseErrorAbstractFinal */',
+                [
+                    'is_abstract' => true,
+                    'is_final'    => true,
+                ],
+            ],
         ];
     }
 }

Tests/Utils/ObjectDeclarations/GetClassPropertiesDiffTest.inc

@@ -16,10 +16,3 @@ final
  * @phpcs:disable Standard.Cat.SniffName -- Just because.
  */
 class ClassWithPropertyBeforeDocblock {}
-
-/* testParseErrorAbstractFinal */
-final /* comment */
-
-    abstract // Intentional parse error, class cannot both be final and abstract.
-
-        class AbstractFinal {}

Tests/Utils/ObjectDeclarations/GetClassPropertiesDiffTest.php

@@ -81,13 +81,6 @@ public function dataGetClassProperties()
                     'is_final'    => true,
                 ],
             ],
-            'abstract-final-parse-error' => [
-                '/* testParseErrorAbstractFinal */',
-                [
-                    'is_abstract' => true,
-                    'is_final'    => false,
-                ],
-            ],
         ];
     }
 }