feat(dart): add dart config to very_good.yaml

Closes #1655

Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>

Author: unicoderbot[bot]

lib/src/commands/dart/commands/dart_test_command.dart

@@ -7,6 +7,7 @@ import 'package:mason/mason.dart';
 import 'package:meta/meta.dart';
 import 'package:path/path.dart' as path;
 import 'package:very_good_cli/src/cli/cli.dart';
+import 'package:very_good_cli/src/very_good_config/very_good_config.dart';
 
 /// Options for configuring the Dart test command.
 class DartTestOptions {
@@ -32,18 +33,53 @@ class DartTestOptions {
   });
 
   /// Parses [ArgResults] into a [DartTestOptions] instance.
-  factory DartTestOptions.parse(ArgResults argResults) {
-    final concurrency = argResults['concurrency'] as String;
-    final collectCoverage = argResults['coverage'] as bool;
-    final minCoverage = double.tryParse(
-      argResults['min-coverage'] as String? ?? '',
+  ///
+  /// When [config] is provided, its values are used as defaults for any
+  /// option that was not explicitly parsed on the command line.
+  factory DartTestOptions.parse(
+    ArgResults argResults, {
+    VeryGoodConfig config = VeryGoodConfig.empty,
+  }) {
+    final testConfig = config.dart.test;
+
+    final concurrency = _resolveArg(
+      argResults,
+      'concurrency',
+      testConfig.concurrency,
+    );
+    final collectCoverage = _resolveArg(
+      argResults,
+      'coverage',
+      testConfig.coverage,
+    );
+    final minCoverageString = _resolveArg<String?>(
+      argResults,
+      'min-coverage',
+      testConfig.minCoverage,
+    );
+    final minCoverage = double.tryParse(minCoverageString ?? '');
+    final showUncovered = _resolveArg(
+      argResults,
+      'show-uncovered',
+      testConfig.showUncovered,
+    );
+    final excludeTags = _resolveArg<String?>(
+      argResults,
+      'exclude-tags',
+      testConfig.excludeTags,
+    );
+    final tags = _resolveArg<String?>(argResults, 'tags', testConfig.tags);
+    final excludeFromCoverage = _resolveArg<String?>(
+      argResults,
+      'exclude-coverage',
+      testConfig.excludeCoverage,
+    );
+    final collectCoverageFromString = _resolveArg<String>(
+      argResults,
+      'collect-coverage-from',
+      testConfig.collectCoverageFrom,
+      fallbackValue: 'imports',
     );
-    final showUncovered = argResults['show-uncovered'] as bool;
-    final excludeTags = argResults['exclude-tags'] as String?;
-    final tags = argResults['tags'] as String?;
-    final excludeFromCoverage = argResults['exclude-coverage'] as String?;
-    final collectCoverageFromString =
-        argResults['collect-coverage-from'] as String? ?? 'imports';
     final collectCoverageFrom = CoverageCollectionMode.fromString(
       collectCoverageFromString,
     );
@@ -52,17 +88,46 @@ class DartTestOptions {
     final randomSeed = randomOrderingSeed == 'random'
         ? Random().nextInt(4294967295).toString()
         : randomOrderingSeed;
-    final optimizePerformance = argResults['optimization'] as bool;
-    final failFast = argResults['fail-fast'] as bool;
+    final optimizePerformance = _resolveArg(
+      argResults,
+      'optimization',
+      testConfig.optimization,
+    );
+    final failFast = _resolveArg(
+      argResults,
+      'fail-fast',
+      testConfig.failFast,
+    );
     final forceAnsi = argResults['force-ansi'] as bool?;
-    final platform = argResults['platform'] as String?;
-    final reportOn = (argResults['report-on'] as List<String>)
+    final platform = _resolveArg<String?>(
+      argResults,
+      'platform',
+      testConfig.platform,
+    );
+    final reportOn = _resolveArg<List<String>>(
+      argResults,
+      'report-on',
+      testConfig.reportOn,
+    );
+    final effectiveReportOn = reportOn
         .expand((e) => e.split(RegExp(r'[,\s]+')))
         .where((e) => e.isNotEmpty)
         .toList();
-    final runSkipped = argResults['run-skipped'] as bool;
-    final checkIgnore = argResults['check-ignore'] as bool;
-    final fileReporter = argResults['file-reporter'] as String?;
+    final runSkipped = _resolveArg(
+      argResults,
+      'run-skipped',
+      testConfig.runSkipped,
+    );
+    final checkIgnore = _resolveArg(
+      argResults,
+      'check-ignore',
+      testConfig.checkIgnore,
+    );
+    final fileReporter = _resolveArg<String?>(
+      argResults,
+      'file-reporter',
+      testConfig.fileReporter,
+    );
     final rest = argResults.rest;
 
     return DartTestOptions._(
@@ -79,7 +144,7 @@ class DartTestOptions {
       failFast: failFast,
       forceAnsi: forceAnsi,
       platform: platform,
-      reportOn: reportOn,
+      reportOn: effectiveReportOn,
       runSkipped: runSkipped,
       checkIgnore: checkIgnore,
       fileReporter: fileReporter,
@@ -144,6 +209,27 @@ class DartTestOptions {
   final List<String> rest;
 }
 
+/// Resolves the value for the argument named [name] against a `very_good.yaml`
+/// configuration value.
+///
+/// Resolution follows a fixed precedence, from highest to lowest:
+///
+/// 1. A command line argument that was explicitly parsed.
+/// 2. [configValue], the corresponding value from the configuration file.
+/// 3. [fallbackValue], used when neither the command line nor the configuration
+///    provide a value (typically the argument's command line default).
+T _resolveArg<T>(
+  ArgResults argResults,
+  String name,
+  T? configValue, {
+  T? fallbackValue,
+}) {
+  final value = configValue != null && !argResults.wasParsed(name)
+      ? configValue
+      : argResults[name] as T?;
+  return (value ?? fallbackValue) as T;
+}
+
 /// Signature for the [Dart.installed] method.
 typedef DartInstalledCommand = Future<bool> Function({required Logger logger});
 
@@ -327,9 +413,20 @@ This command should be run from the root of your Dart project.''');
       return ExitCode.noInput.code;
     }
 
+    final VeryGoodConfig config;
+    try {
+      config = VeryGoodConfig.loadFromClosestAncestor(Directory(targetPath));
+    } on VeryGoodConfigParseException catch (e) {
+      _logger.err(
+        'Could not read `$veryGoodConfigFileName`.\n'
+        '${e.message}',
+      );
+      return ExitCode.config.code;
+    }
+
     final isDartInstalled = await _dartInstalled(logger: _logger);
 
-    final options = DartTestOptions.parse(_argResults);
+    final options = DartTestOptions.parse(_argResults, config: config);
 
     if (isDartInstalled) {
       try {

lib/src/very_good_config/very_good_config.dart

@@ -48,7 +48,10 @@ class VeryGoodConfigParseException implements Exception {
 )
 class VeryGoodConfig extends Equatable {
   /// {@macro very_good_config}
-  const VeryGoodConfig({this.test = const VeryGoodTestConfig()});
+  const VeryGoodConfig({
+    this.test = const VeryGoodTestConfig(),
+    this.dart = const VeryGoodDartConfig(),
+  });
 
   /// Creates a [VeryGoodConfig] from a decoded YAML/JSON [json] map.
   factory VeryGoodConfig.fromJson(Map<dynamic, dynamic> json) {
@@ -121,8 +124,11 @@ class VeryGoodConfig extends Equatable {
   /// Configuration values for the `very_good test` command.
   final VeryGoodTestConfig test;
 
+  /// Configuration values for the `very_good dart test` command.
+  final VeryGoodDartConfig dart;
+
   @override
-  List<Object?> get props => [test];
+  List<Object?> get props => [test, dart];
 }
 
 /// {@template very_good_test_config}
@@ -256,6 +262,145 @@ class VeryGoodTestConfig extends Equatable {
   ];
 }
 
+/// {@template very_good_dart_config}
+/// Configuration values that customize the defaults of the
+/// `very_good dart test` command.
+///
+/// Any field that is left as `null` retains its CLI default.
+/// {@endtemplate}
+@JsonSerializable(
+  anyMap: true,
+  checked: true,
+  createToJson: false,
+  disallowUnrecognizedKeys: true,
+  fieldRename: FieldRename.snake,
+)
+class VeryGoodDartConfig extends Equatable {
+  /// {@macro very_good_dart_config}
+  const VeryGoodDartConfig({
+    this.test = const VeryGoodDartTestConfig(),
+  });
+
+  /// Creates a [VeryGoodDartConfig] from a decoded YAML/JSON [json] map.
+  factory VeryGoodDartConfig.fromJson(Map<dynamic, dynamic> json) {
+    return _$VeryGoodDartConfigFromJson(json);
+  }
+
+  /// Configuration values for the `very_good dart test` subcommand.
+  final VeryGoodDartTestConfig test;
+
+  @override
+  List<Object?> get props => [test];
+}
+
+/// {@template very_good_dart_test_config}
+/// Configuration values that customize the defaults of the
+/// `very_good dart test` command.
+///
+/// Any field that is left as `null` retains its CLI default.
+/// {@endtemplate}
+@JsonSerializable(
+  anyMap: true,
+  checked: true,
+  createToJson: false,
+  disallowUnrecognizedKeys: true,
+  fieldRename: FieldRename.snake,
+)
+class VeryGoodDartTestConfig extends Equatable {
+  /// {@macro very_good_dart_test_config}
+  const VeryGoodDartTestConfig({
+    this.coverage,
+    this.optimization,
+    this.concurrency,
+    this.tags,
+    this.excludeCoverage,
+    this.excludeTags,
+    this.minCoverage,
+    this.showUncovered,
+    this.collectCoverageFrom,
+    this.failFast,
+    this.platform,
+    this.reportOn,
+    this.runSkipped,
+    this.checkIgnore,
+    this.fileReporter,
+  });
+
+  /// Creates a [VeryGoodDartTestConfig] from a decoded YAML/JSON [json] map.
+  factory VeryGoodDartTestConfig.fromJson(Map<dynamic, dynamic> json) {
+    return _$VeryGoodDartTestConfigFromJson(json);
+  }
+
+  /// Whether to collect coverage information.
+  final bool? coverage;
+
+  /// Whether to apply optimizations for test performance.
+  final bool? optimization;
+
+  /// The number of concurrent test suites run.
+  @JsonKey(fromJson: _concurrency)
+  final String? concurrency;
+
+  /// Run only tests associated with the specified tags.
+  final String? tags;
+
+  /// A glob which will be used to exclude files that match from the coverage.
+  final String? excludeCoverage;
+
+  /// Run only tests that do not have the specified tags.
+  final String? excludeTags;
+
+  /// The minimum coverage percentage enforced.
+  @JsonKey(fromJson: _minCoverage)
+  final String? minCoverage;
+
+  /// Whether to show uncovered lines when coverage is below 100%.
+  final bool? showUncovered;
+
+  /// Whether to collect coverage from imported files only or all files.
+  @JsonKey(fromJson: _collectCoverageFrom)
+  final String? collectCoverageFrom;
+
+  /// Whether to stop running tests after the first failure.
+  final bool? failFast;
+
+  /// The platform to run tests on (e.g. `chrome`, `vm`).
+  final String? platform;
+
+  /// Optional file paths to report coverage information to.
+  @JsonKey(fromJson: _stringList)
+  final List<String>? reportOn;
+
+  /// Whether to run skipped tests instead of skipping them.
+  final bool? runSkipped;
+
+  /// Whether to check for and respect coverage ignore comments.
+  final bool? checkIgnore;
+
+  /// Additional reporter that writes test results to a file, expressed as
+  /// `<name>:<path>` (e.g. `json:reports/tests.json`).
+  final String? fileReporter;
+
+  @override
+  List<Object?> get props => [
+    coverage,
+    optimization,
+    concurrency,
+    tags,
+    excludeCoverage,
+    excludeTags,
+    minCoverage,
+    showUncovered,
+    collectCoverageFrom,
+    failFast,
+    platform,
+    reportOn,
+    runSkipped,
+    checkIgnore,
+    fileReporter,
+  ];
+}
+
 // The coercers below intentionally validate more strictly than the CLI flag
 // parser. A value such as `min_coverage: 150` is rejected here at config load
 // time even though `--min-coverage 150` is accepted by the flag parser, so