Add more test cases, do not advertise class support

Author: Yurii Prykhodko

lib/src/common/parameters/excluded_entities_list_parameter.dart

@@ -4,7 +4,6 @@ import 'package:analyzer/dart/ast/ast.dart';
 /// identifiers (classes, mixins, enums, extensions) to be ignored during
 /// analysis.
 /// Supported entities:
-///   - class
 ///   - mixin
 ///   - extension
 ///   - enum
@@ -28,6 +27,7 @@ class ExcludedEntitiesListParameter {
         excludedEntityNames: Set<String>.from(raw),
       );
     }
+
     return ExcludedEntitiesListParameter(
       excludedEntityNames: {},
     );
@@ -37,10 +37,7 @@ class ExcludedEntitiesListParameter {
   bool shouldIgnoreEntity(Declaration node) {
     if (excludedEntityNames.isEmpty) return false;
 
-    if (node is ClassDeclaration && excludedEntityNames.contains('class')) {
-      return true;
-    } else if (node is MixinDeclaration &&
-        excludedEntityNames.contains('mixin')) {
+    if (node is MixinDeclaration && excludedEntityNames.contains('mixin')) {
       return true;
     } else if (node is EnumDeclaration &&
         excludedEntityNames.contains('enum')) {

lint_test/prefer_match_file_name_ignore_entity/enum/analysis_options.yaml

@@ -0,0 +1,9 @@
+analyzer:
+  plugins:
+    - ../../../custom_lint
+
+custom_lint:
+  rules:
+    - prefer_match_file_name:
+      exclude_entity:
+        - enum

lint_test/prefer_match_file_name_ignore_entity/enum/test.dart

@@ -0,0 +1,14 @@
+// ignore_for_file: unused_element, unused_field
+
+/// `prefer_match_file_name` rule will be ignored by this enttiy because of
+/// exclude_entity:
+/// - enum
+/// in analysis_options.yaml
+enum Ignored { _ }
+enum IgnoredAgain { _ }
+
+// expect_lint: prefer_match_file_name
+abstract class WrongNamedClass {}
+
+/// Only first public element declaration is checked
+class PreferMatchFileNameIgnoreExtensions {}

…_ignore_extensions/analysis_options.yaml …e_entity/extension/analysis_options.yamllint_test/prefer_match_file_name_ignore_extensions/analysis_options.yaml renamed to lint_test/prefer_match_file_name_ignore_entity/extension/analysis_options.yaml lint_test/prefer_match_file_name_ignore_extensions/analysis_options.yaml renamed to lint_test/prefer_match_file_name_ignore_entity/extension/analysis_options.yaml

@@ -1,6 +1,6 @@
 analyzer:
   plugins:
-    - ../../custom_lint
+    - ../../../custom_lint
 
 custom_lint:
   rules:

…r_match_file_name_ignore_extensions.dart …e_name_ignore_entity/extension/test.dartlint_test/prefer_match_file_name_ignore_extensions/prefer_match_file_name_ignore_extensions.dart renamed to lint_test/prefer_match_file_name_ignore_entity/extension/test.dart lint_test/prefer_match_file_name_ignore_extensions/prefer_match_file_name_ignore_extensions.dart renamed to lint_test/prefer_match_file_name_ignore_entity/extension/test.dart

@@ -4,7 +4,8 @@
 /// exclude_entity:
 /// - extension
 /// in analysis_options.yaml
-extension DefaultExtension on String {}
+extension Ignored on String {}
+extension IgnoredAgain on String {}
 
 // expect_lint: prefer_match_file_name
 abstract class WrongNamedClass {}

lint_test/prefer_match_file_name_ignore_entity/mixin/analysis_options.yaml

@@ -0,0 +1,9 @@
+analyzer:
+  plugins:
+    - ../../../custom_lint
+
+custom_lint:
+  rules:
+    - prefer_match_file_name:
+      exclude_entity:
+        - mixin

lint_test/prefer_match_file_name_ignore_entity/mixin/test.dart

@@ -0,0 +1,13 @@
+// ignore_for_file: unused_element, unused_field
+
+/// `prefer_match_file_name` rule will be ignored by this entity because of
+/// exclude_entity:
+/// - mixin
+/// in analysis_options.yaml
+mixin IgnoredMixin {}
+
+// expect_lint: prefer_match_file_name
+abstract class WrongNamedClass {}
+
+/// Only first public element declaration is checked
+class PreferMatchFileNameIgnoreExtensions {}