chore: update test command to use dart test with coverage flag (#51)

* fix: update test command to use dart test with coverage flag

* fix: update code coverage reporting action to v4

* fix: improve code coverage reporting by adding coverage tool installation and LCOV conversion

* Add AST validation utilities for router generation and enhance tests

- Introduced `router_ast_validators.dart` with specialized validators for router class structure, routes field, pages map, and getter methods.
- Enhanced `ast_comparison_helper.dart` with methods to find constructors and methods in extensions.
- Updated `route_class_generator_test.dart` to utilize new validation methods for various routing scenarios, including empty routes, nested routers, and aliased imports.
- Refactored existing tests to improve structure and validation logic.

* fix: update import for AST validation helpers in router class builder test

* refactor: streamline AST validation utilities and enhance documentation for clarity

* Add comprehensive tests for logging and routing generators

- Introduced `logger_builder_simple_test.dart` to validate basic logger generation, structure, error handling, stacktrace support, method chaining, and determinism.
- Updated `logger_class_generator_test.dart` to utilize AST-based validation for logger generation instead of string comparison.
- Modified `simple_log_printer_test.dart` to correct the log event structure for error handling.
- Enhanced `arguments_class_test.dart` to validate the generation of argument classes with basic AST checks.
- Refactored `navigation_extension_class_test.dart` and `router_extension_class_test.dart` to implement AST validation for navigation extensions.
- Improved `router_class_builder_helper_test.dart` and `routes_class_builder_test.dart` to include basic AST validation for generated routes and getters.
- Added new tests for Router 2.0 in `auto_route_generator_test.dart` and `integration_test/router_2_integration_test.dart` to cover complete library generation, error scenarios, and code quality checks.
- Created `library_builder_test.dart` and `router_2_generator_test.dart` to validate library generation and router generator functionality.

* fix: update SDK version constraint in pubspec.yaml

Author: Eimen2018

.github/workflows/test.yml

@@ -20,13 +20,17 @@ jobs:
       - run: dart pub get
       - run: dart format .
       - run: dart analyze
-      - run: flutter test --coverage 
+      - run: dart test --coverage=coverage
+      - name: Install coverage tools
+        run: dart pub global activate coverage
+      - name: Convert coverage to LCOV format
+        run: dart pub global run coverage:format_coverage --lcov --in=coverage --out=coverage/lcov.info --packages=.dart_tool/package_config.json --report-on=lib
       - name: Setup LCOV
         uses: hrishikesh-kadam/setup-lcov@v1
       - name: Report code coverage
-        uses: zgosalvez/github-actions-report-lcov@v3
+        uses: zgosalvez/github-actions-report-lcov@v4
         with:
-          coverage-files: coverage/lcov*.info
+          coverage-files: coverage/lcov.info
           artifact-name: code-coverage-report
           github-token: ${{ secrets.GITHUB_TOKEN }}
           update-comment: true

lib/src/generators/extensions/routes_extension.dart

@@ -7,11 +7,14 @@ extension RoutesExtension on RouterConfig {
     performAction(this);
 
     for (var route in routes) {
-      RouterConfig(
-        routes: route.children,
-        routerClassName: '${route.name}Router'.capitalize,
-        routesClassName: '${route.name}Routes'.capitalize,
-      ).traverseRoutes(performAction);
+      // Only create and traverse child router config if the route has children
+      if (route.children.isNotEmpty) {
+        RouterConfig(
+          routes: route.children,
+          routerClassName: '${route.name}Router'.capitalize,
+          routesClassName: '${route.name}Routes'.capitalize,
+        ).traverseRoutes(performAction);
+      }
     }
   }
 }

lib/src/generators/logging/logger_class_content.dart

@@ -42,7 +42,7 @@ class SimpleLogPrinter extends LogPrinter {
 
     var methodNameSection =
         printCallingFunctionName && methodName != null ? ' | \$methodName' : '';
-    var stackLog = event.stackTrace.toString();
+    var stackLog = event.stackTrace?.toString() ?? '';
     var output =
         '\$emoji \$className\$methodNameSection - \${event.message}\${event.error != null ? '\\nERROR: \${event.error}\\n' : ''}\${printCallStack ? '\\nSTACKTRACE:\\n\$stackLog' : ''}';
 
@@ -61,7 +61,7 @@ class SimpleLogPrinter extends LogPrinter {
         if (kReleaseMode) {
           return match.group(0)!;
         } else {
-          return color!(match.group(0)!);
+          return color != null ? color(match.group(0)!) : match.group(0)!;
         }
       }));
     }

lib/src/generators/router/generator/navigate_extension_class/navigate_extension_class_builder_helper.dart

@@ -90,8 +90,8 @@ mixin NavigateExtensionClassBuilderHelper {
   ///
   /// int? routerId,
   /// bool preventDuplicates = true,
-  /// Map<String, String>? parameters,
-  /// Widget Function(BuildContext, Animation<double>, Animation<double>, Widget)? transition,
+  /// Map`<String, String>?` parameters,
+  /// Widget Function(BuildContext, Animation`<double>`, Animation`<double>`, Widget)? transition,
   List<Parameter> get _constParameters => [
         Parameter((b) => b
           ..name = 'routerId'

lib/src/generators/router/generator/router_class/router_class_builder_helper.dart

@@ -36,7 +36,7 @@ mixin RouterClassBuilderHelper {
   /// Example
   ///
   /// @override
-  /// Map<Type, _i1.StackedRouteFactory> get pagesMap => _pagesMap;
+  /// Map`<Type, _i1.StackedRouteFactory>` get pagesMap => _pagesMap;
   Method get pagesMapGetter => Method((b) => b
     ..name = 'pagesMap'
     ..annotations.add(const Reference('override'))
@@ -74,7 +74,7 @@ mixin RouterClassBuilderHelper {
 
   /// Example
   ///
-  ///  final _pagesMap = <Type, _i1.StackedRouteFactory>{};
+  ///  final _pagesMap = `<Type, _i1.StackedRouteFactory>`{};
   Field mapOfPages(List<RouteConfig> routes) => Field(
         (b) => b
           ..name = '_pagesMap'
@@ -142,7 +142,7 @@ mixin RouterClassBuilderHelper {
 
   /// Example
   ///
-  /// return MaterialPageRoute<dynamic>(
+  /// return MaterialPageRoute`<dynamic>`(
   ///   builder: (context) => HomeView(
   ///     key: args.key,
   ///     title: args.title,

lib/src/generators/router/generator/routes_class/routes_class_builder.dart

@@ -15,7 +15,7 @@ class RoutesClassBuilder {
   /// static const String loginView = 'pathNamaw';
   /// static const String _homeView = '/family/:fid';
   /// static String homeView({@required dynamic fid}) => '/family/\$fid';
-  /// static const all = <String>{
+  /// static const all = `<String>`{
   /// loginView,_homeView,};}
   ///
   Class buildRoutesClass() {

lib/src/generators/router/route_config/material_route_config.dart

@@ -16,6 +16,7 @@ class MaterialRouteConfig extends RouteConfig {
     super.children,
     super.imports,
     super.parentClassName,
+    super.pageType,
   });
 
   Code registerRoute() {

lib/src/helpers/file_helper.dart

@@ -99,7 +99,7 @@ class FileHelper {
   ///   filePath (String): The path to the file to read.
   ///
   /// Returns:
-  ///   A Future<List<String>>
+  ///   A Future`<List<String>>`
   Future<List<String>> readFileAsLines({
     required String filePath,
   }) {

pubspec.yaml

@@ -5,10 +5,9 @@ version: 2.0.0
 homepage: https://github.com/FilledStacks/stacked/tree/master/packages/stacked_generator
 
 environment:
-  sdk: ">=2.17.0 <4.0.0"
+  sdk: ">=2.19.0 <4.0.0"
 
 dependencies:
-
   build: ^3.0.0
   path: ^1.9.1
   recase: ^4.1.0

test/bottomsheets/bottomsheet_class_generator_test.dart

@@ -2,36 +2,59 @@ import 'package:stacked_generator/src/generators/bottomsheets/bottomsheet_config
 import 'package:stacked_generator/src/generators/bottomsheets/generate/bottomsheet_class_generator.dart';
 import 'package:test/test.dart';
 
-import '../helpers/test_constants/bottomsheets_constants.dart';
+import '../helpers/ast/bottomsheet_ast_validators.dart';
 
 void main() {
   group('BottomsheetClassGeneratorTest -', () {
     group('generate -', () {
       test('When empty', () async {
         final generator = BottomsheetClassGenerator([]);
-        expect(await generator.generate(), kBottomsheetsEmpty);
+        final result = await generator.generate();
+        
+        BottomsheetClassGeneratorAstValidator.validateEmptyBottomsheetSetup(
+          result,
+          expectedLocatorCall: 'locator',
+        );
       });
       test('When change locator name', () async {
         final generator =
             BottomsheetClassGenerator([], locatorName: 'customLocator');
-        expect(await generator.generate(), kBottomsheetsWithCustomNamedLocator);
+        final result = await generator.generate();
+        
+        BottomsheetClassGeneratorAstValidator.validateEmptyBottomsheetSetup(
+          result,
+          expectedLocatorCall: 'customLocator',
+        );
       });
       test('One bottomsheet', () async {
-        final generator = BottomsheetClassGenerator([
+        final bottomsheets = [
           const BottomsheetConfig(
               import: 'one.dart', bottomsheetClassName: 'BasicBottomsheet')
-        ]);
-        expect(await generator.generate(), kOneBottomsheet);
+        ];
+        final generator = BottomsheetClassGenerator(bottomsheets);
+        final result = await generator.generate();
+        
+        BottomsheetClassGeneratorAstValidator.validateBottomsheetSetup(
+          result,
+          expectedBottomsheets: bottomsheets,
+          expectedLocatorCall: 'locator',
+        );
       });
       test('Two bottomsheets', () async {
-        final generator = BottomsheetClassGenerator([
+        final bottomsheets = [
           const BottomsheetConfig(
               import: 'one.dart', bottomsheetClassName: 'BasicBottomsheet'),
           const BottomsheetConfig(
               import: 'two.dart', bottomsheetClassName: 'ComplexBottomsheet')
-        ]);
+        ];
+        final generator = BottomsheetClassGenerator(bottomsheets);
+        final result = await generator.generate();
 
-        expect(await generator.generate(), kTwoBottomsheets);
+        BottomsheetClassGeneratorAstValidator.validateBottomsheetSetup(
+          result,
+          expectedBottomsheets: bottomsheets,
+          expectedLocatorCall: 'locator',
+        );
       });
     });
   });