Fix avoid-late if initialized

lib/lints/avoid_late_keyword/avoid_late_keyword_rule.dart

@@ -1,10 +1,12 @@
+import 'package:analyzer/dart/ast/ast.dart';
 import 'package:analyzer/error/listener.dart';
 import 'package:custom_lint_builder/custom_lint_builder.dart';
+import 'package:solid_lints/lints/avoid_late_keyword/models/avoid_late_keyword_parameters.dart';
 import 'package:solid_lints/models/rule_config.dart';
 import 'package:solid_lints/models/solid_lint_rule.dart';
 
 /// A `late` keyword rule which forbids using it to avoid runtime exceptions.
-class AvoidLateKeywordRule extends SolidLintRule {
+class AvoidLateKeywordRule extends SolidLintRule<AvoidLateKeywordParameters> {
   /// The [LintCode] of this lint rule that represents
   /// the error whether we use `late` keyword.
   static const lintName = 'avoid_late_keyword';
@@ -17,6 +19,7 @@ class AvoidLateKeywordRule extends SolidLintRule {
     final rule = RuleConfig(
       configs: configs,
       name: lintName,
+      paramsParser: AvoidLateKeywordParameters.fromJson,
       problemMessage: (_) => 'Avoid using the "late" keyword. '
           'It may result in runtime exceptions.',
     );
@@ -31,9 +34,29 @@ class AvoidLateKeywordRule extends SolidLintRule {
     CustomLintContext context,
   ) {
     context.registry.addVariableDeclaration((node) {
-      if (node.declaredElement?.isLate ?? false) {
+      if (_shouldLint(node)) {
         reporter.reportErrorForNode(code, node);
       }
     });
   }
+
+  bool _shouldLint(VariableDeclaration node) {
+    final isLateDeclaration = node.declaredElement?.isLate ?? false;
+    if (!isLateDeclaration) return false;
+
+    final ignoredTypes = _hasIgnoredType(node);
+    if (ignoredTypes) return false;
+
+    final allowInitialized = config.parameters.allowInitialized;
+    if (!allowInitialized) return true;
+
+    final hasInitializer = node.initializer != null;
+    return !hasInitializer;
+  }
+
+  bool _hasIgnoredType(VariableDeclaration node) =>
+      config.parameters.ignoredTypes.contains(
+        // ignore: deprecated_member_use
+        node.declaredElement2?.type.getDisplayString(withNullability: false),
+      );
 }

lib/lints/avoid_late_keyword/models/_config_parser.dart

@@ -0,0 +1,15 @@
+part of 'avoid_late_keyword_parameters.dart';
+
+class _ConfigParser {
+  static const _allowInitializedConfig = 'allow_initialized';
+  static const _ignoredTypesConfig = 'ignored_types';
+
+  static bool parseAllowInitialized(Map<String, Object?> config) =>
+      config[_allowInitializedConfig] as bool? ?? false;
+
+  static Iterable<String> parseIgnoredTypes(Map<String, Object?> config) =>
+      config.containsKey(_ignoredTypesConfig) &&
+              config[_ignoredTypesConfig] is Iterable
+          ? List<String>.from(config[_ignoredTypesConfig] as Iterable)
+          : <String>['AnimationController'];
+}

lib/lints/avoid_late_keyword/models/avoid_late_keyword_parameters.dart

@@ -0,0 +1,24 @@
+part '_config_parser.dart';
+
+/// A data model class that represents the "avoid late keyword" input
+/// parameters.
+class AvoidLateKeywordParameters {
+  /// Allow to dynamically initialize
+  final bool allowInitialized;
+
+  /// Types that would be ignored by avoid-late rule
+  final Iterable<String> ignoredTypes;
+
+  /// Constructor for [AvoidLateKeywordParameters] model
+  const AvoidLateKeywordParameters({
+    this.allowInitialized = false,
+    this.ignoredTypes = const [],
+  });
+
+  /// Method for creating from json data
+  factory AvoidLateKeywordParameters.fromJson(Map<String, Object?> json) =>
+      AvoidLateKeywordParameters(
+        allowInitialized: _ConfigParser.parseAllowInitialized(json),
+        ignoredTypes: _ConfigParser.parseIgnoredTypes(json),
+      );
+}

lint_test/analysis_options.yaml

@@ -11,7 +11,11 @@ custom_lint:
     - function_lines_of_code:
       max_lines: 50
     - avoid_non_null_assertion
-    - avoid_late_keyword
+    - avoid_late_keyword:
+      allow_initialized: true
+      ignored_types:
+        - ColorTween
+        - AnimationController
     - avoid_global_state
     - avoid_returning_widgets
     - avoid_unnecessary_setstate

lint_test/avoid_late_keyword_test.dart

@@ -1,18 +1,26 @@
 // ignore_for_file: prefer_const_declarations, unused_local_variable, prefer_match_file_name
 // ignore_for_file: avoid_global_state
 
+import 'package:flutter/material.dart';
+
 /// Check "late" keyword fail
 ///
 /// `avoid_late_keyword`
 class AvoidLateKeyword {
-  /// expect_lint: avoid_late_keyword
+  late final ColorTween colorTween;
+
+  late final AnimationController controller1;
+
   late final field1 = 'string';
 
   /// expect_lint: avoid_late_keyword
   late String field2;
 
   void test() {
-    /// expect_lint: avoid_late_keyword
+    late final ColorTween colorTween;
+
+    late final AnimationController controller2;
+
     late final field3 = 'string';
 
     /// expect_lint: avoid_late_keyword