fix: temporary remove enums support (#849)

* fix: temporary remove enums support

* test: update tests

Co-authored-by: Dmitry Krutskikh <dmitry.krutskikh@gmail.com>

Author: incendial

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 * fix: [`avoid-border-all`](https://dartcodemetrics.dev/docs/rules/flutter/avoid-border-all) is triggered even when it is not a const.
 * fix: remove duplicated and ignore void function calls for [`prefer-moving-to-variable`](https://dartcodemetrics.dev/docs/rules/common/prefer-moving-to-variable).
+* fix: temporary remove enums support for [`prefer-trailing-comma`](https://dartcodemetrics.dev/docs/rules/common/prefer-trailing-comma).
 
 ## 4.15.1
 

lib/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/visitor.dart

@@ -28,13 +28,6 @@ class _Visitor extends RecursiveAstVisitor<void> {
     );
   }
 
-  @override
-  void visitEnumDeclaration(EnumDeclaration node) {
-    super.visitEnumDeclaration(node);
-
-    _visitNodeList(node.constants, node.leftBracket, node.rightBracket);
-  }
-
   @override
   void visitListLiteral(ListLiteral node) {
     super.visitListLiteral(node);

test/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/examples/incorrect_example.dart

@@ -35,7 +35,7 @@ enum FirstEnum {
   thirdItem,
   forthItem,
   fifthItem,
-  sixthItem // LINT
+  sixthItem
 }
 
 class FirstClass {

test/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/prefer_trailing_comma_rule_test.dart

@@ -30,16 +30,15 @@ void main() {
 
       RuleTestHelper.verifyIssues(
         issues: issues,
-        startLines: [3, 9, 13, 18, 24, 28, 38, 49, 58, 64, 70],
-        startColumns: [50, 7, 5, 52, 9, 8, 3, 59, 3, 3, 3],
+        startLines: [3, 9, 13, 18, 24, 28, 49, 58, 64, 70],
+        startColumns: [50, 7, 5, 52, 9, 8, 59, 3, 3, 3],
         locationTexts: [
           'String thirdArgument',
           "'and another string for length exceed'",
           'String arg3',
           'String thirdArgument',
           "'and another string for length exceed'",
           "'some other string'",
-          'sixthItem',
           'this.forthField',
           "'and another string for length exceed'",
           "'and another string for length exceed'",
@@ -56,7 +55,6 @@ void main() {
           'Prefer trailing comma.',
           'Prefer trailing comma.',
           'Prefer trailing comma.',
-          'Prefer trailing comma.',
         ],
         replacementComments: [
           'Add trailing comma.',
@@ -69,7 +67,6 @@ void main() {
           'Add trailing comma.',
           'Add trailing comma.',
           'Add trailing comma.',
-          'Add trailing comma.',
         ],
         replacements: [
           'String thirdArgument,',
@@ -78,7 +75,6 @@ void main() {
           'String thirdArgument,',
           "'and another string for length exceed',",
           "'some other string',",
-          'sixthItem,',
           'this.forthField,',
           "'and another string for length exceed',",
           "'and another string for length exceed',",
@@ -102,8 +98,8 @@ void main() {
 
       RuleTestHelper.verifyIssues(
         issues: issues,
-        startLines: [9, 17, 19, 37, 41, 75, 91, 99, 109, 119],
-        startColumns: [21, 33, 20, 23, 19, 18, 43, 21, 19, 19],
+        startLines: [9, 17, 19, 37, 41, 91, 99, 109, 119],
+        startColumns: [21, 33, 20, 23, 19, 43, 21, 19, 19],
         locationTexts: [
           'String arg1',
           'void Function() callback',
@@ -114,7 +110,6 @@ void main() {
           '() {\n'
               '      return;\n'
               '    }',
-          'firstItem',
           '0',
           '\'some string\'',
           '\'some string\'',
@@ -130,7 +125,6 @@ void main() {
           'Prefer trailing comma.',
           'Prefer trailing comma.',
           'Prefer trailing comma.',
-          'Prefer trailing comma.',
         ],
       );
     });