feat: introduce very_good.yaml for param configuration

.github/workflows/e2e.yaml

@@ -33,6 +33,7 @@ jobs:
           - test/commands/test/compilation_error/compilation_error_test.dart
           - test/commands/test/no_project/no_project_test.dart
           - test/commands/test/spaced_golden_file_name/spaced_golden_file_name_test.dart
+          - test/commands/test/very_good_config/very_good_config_test.dart
 
           # E2E tests for the create command
           - test/commands/create/flutter_app/core_test.dart

.github/workflows/very_good_cli.yaml

@@ -29,12 +29,14 @@ jobs:
         os: [ubuntu-latest, windows-latest]
     uses: VeryGoodOpenSource/very_good_workflows/.github/workflows/flutter_package.yml@v1
     with:
+      runs_on: ${{ matrix.os }}
       flutter_version: "3.44.x"
+      coverage_excludes: "**/*.{gen,g}.dart"
       concurrency: 1
-      coverage_excludes: "**/*.gen.dart"
-      run_bloc_lint: false
       run_skipped: true
-      runs_on: ${{ matrix.os }}
+      run_bloc_lint: false
+      # The CLI's own test suite is run directly rather than through the test optimizer,
+      # matching the CI behavior prior to adopting this shared workflow.
       test_optimization: false
 
   pana:

analysis_options.yaml

@@ -1,5 +1,6 @@
 include: package:very_good_analysis/analysis_options.yaml
 analyzer:
   exclude:
+    - "**/*.g.dart"
     - "**/version.dart"
     - "bricks/**/__brick__"

build.yaml

@@ -0,0 +1,8 @@
+targets:
+  $default:
+    sources:
+      - $package$
+      - lib/**
+      - bin/**
+      - test/**
+      - pubspec.yaml

e2e/test/commands/test/very_good_config/fixture/lib/uncovered.dart

@@ -0,0 +1 @@
+int untestedFunction(int value) => value * 2;

e2e/test/commands/test/very_good_config/fixture/pubspec.yaml

@@ -0,0 +1,10 @@
+name: very_good_config_fixture
+description: Fixture for testing very_good.yaml configuration.
+version: 0.1.0+1
+publish_to: none
+
+environment:
+  sdk: ^3.12.0
+
+dev_dependencies:
+  test: ^1.24.3

e2e/test/commands/test/very_good_config/fixture/test/covered_test.dart

@@ -0,0 +1,12 @@
+import 'package:test/test.dart';
+
+void main() {
+  group('very_good_config_fixture', () {
+    // This test exists only to produce a passing coverage run so the
+    // `min_coverage: 100` config can be evaluated against the deliberately
+    // uncovered `lib/uncovered.dart`.
+    test('trivially succeeds', () {
+      expect(true, isTrue);
+    });
+  });
+}

e2e/test/commands/test/very_good_config/fixture/very_good.yaml

@@ -0,0 +1,2 @@
+test:
+  min_coverage: 100

e2e/test/commands/test/very_good_config/very_good_config_test.dart

@@ -0,0 +1,127 @@
+import 'package:mason/mason.dart';
+import 'package:mocktail/mocktail.dart';
+import 'package:path/path.dart' as path;
+import 'package:test/test.dart';
+import 'package:universal_io/io.dart';
+
+import '../../../../helpers/helpers.dart';
+
+void main() {
+  group('very_good.yaml', () {
+    test(
+      'enforces min-coverage from very_good.yaml when the flag is not passed',
+      timeout: const Timeout(Duration(minutes: 2)),
+      withRunner((commandRunner, logger, updater, logs, progressLogs) async {
+        final tempDirectory = Directory.systemTemp.createTempSync(
+          'very_good_config',
+        );
+        addTearDown(() => tempDirectory.deleteSync(recursive: true));
+
+        final fixture = Directory(
+          path.join(
+            Directory.current.path,
+            'test/commands/test/very_good_config/fixture',
+          ),
+        );
+
+        await copyDirectory(fixture, tempDirectory);
+
+        await expectSuccessfulProcessResult('flutter', [
+          'pub',
+          'get',
+        ], workingDirectory: tempDirectory.path);
+
+        final cwd = Directory.current;
+        Directory.current = tempDirectory;
+        addTearDown(() => Directory.current = cwd);
+
+        final result = await commandRunner.run(['test', '--coverage']);
+
+        expect(result, equals(ExitCode.unavailable.code));
+        verify(
+          () => logger.err(any(that: contains('Expected coverage >= 100.00%'))),
+        ).called(1);
+      }),
+    );
+
+    test(
+      'CLI --min-coverage takes precedence over very_good.yaml',
+      timeout: const Timeout(Duration(minutes: 2)),
+      withRunner((commandRunner, logger, updater, logs, progressLogs) async {
+        final tempDirectory = Directory.systemTemp.createTempSync(
+          'very_good_config_override',
+        );
+        addTearDown(() => tempDirectory.deleteSync(recursive: true));
+
+        final fixture = Directory(
+          path.join(
+            Directory.current.path,
+            'test/commands/test/very_good_config/fixture',
+          ),
+        );
+
+        await copyDirectory(fixture, tempDirectory);
+
+        await expectSuccessfulProcessResult('flutter', [
+          'pub',
+          'get',
+        ], workingDirectory: tempDirectory.path);
+
+        final cwd = Directory.current;
+        Directory.current = tempDirectory;
+        addTearDown(() => Directory.current = cwd);
+
+        final result = await commandRunner.run([
+          'test',
+          '--coverage',
+          '--min-coverage',
+          '0',
+        ]);
+
+        expect(result, equals(ExitCode.success.code));
+      }),
+    );
+
+    test(
+      'fails with config exit code when very_good.yaml is malformed',
+      timeout: const Timeout(Duration(minutes: 2)),
+      withRunner((commandRunner, logger, updater, logs, progressLogs) async {
+        final tempDirectory = Directory.systemTemp.createTempSync(
+          'very_good_config_malformed',
+        );
+        addTearDown(() => tempDirectory.deleteSync(recursive: true));
+
+        File(path.join(tempDirectory.path, 'pubspec.yaml')).writeAsStringSync(
+          '''
+name: malformed_fixture
+description: Fixture for testing malformed very_good.yaml.
+version: 0.1.0+1
+publish_to: none
+
+environment:
+  sdk: ^3.12.0
+
+dev_dependencies:
+  test: ^1.24.3
+''',
+        );
+        File(
+          path.join(tempDirectory.path, 'very_good.yaml'),
+        ).writeAsStringSync('- not\n- a\n- map');
+
+        final cwd = Directory.current;
+        Directory.current = tempDirectory;
+        addTearDown(() => Directory.current = cwd);
+
+        final result = await commandRunner.run(['test']);
+
+        expect(result, equals(ExitCode.config.code));
+        verify(
+          () => logger.err(
+            any(that: contains('Could not read `very_good.yaml`')),
+          ),
+        ).called(1);
+      }),
+    );
+  });
+}