Skip to content

feat(test): add very_good.yaml for config#1636

Merged
marcossevilla merged 19 commits into
mainfrom
vgv-ai-bot/issue-360
Jul 22, 2026
Merged

feat(test): add very_good.yaml for config#1636
marcossevilla merged 19 commits into
mainfrom
vgv-ai-bot/issue-360

Conversation

@unicoderbot

@unicoderbot unicoderbot Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

Part of #360.

Adds support for configuring CLI test parameters through a very_good.yaml configuration file at the project root. This allows developers to maintain consistent configuration across local and CI environments without manually repeating flags.

Changes

  • Introduces VeryGoodConfig loading from very_good.yaml files
  • Updates FlutterTestOptions.parse() to accept config with proper precedence (CLI args override config file values)
  • Adds comprehensive test coverage for config loading and precedence rules
  • Implements graceful error handling for malformed config files
  • Documents new feature in test command documentation

Configuration Example

# very_good.yaml
test:
  min-coverage: 100
  exclude-coverage: "**/*.g.dart"
  report-on:
    - lib/
  dart-define:
    - FLAVOR=development

Supported options include: concurrency, coverage, min-coverage, show-uncovered, exclude-tags, tags, exclude-coverage, collect-coverage-from, optimization, update-goldens, fail-fast, dart-define, dart-define-from-file, platform, report-on, run-skipped, flavor, and timeout.

Precedence

  • CLI arguments take precedence over config file values
  • Config file values are used as defaults when flags aren't explicitly passed

Closes #360

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

Copy link
Copy Markdown
Member

@vgvbot investigate and fix CI. additionally add an e2e test that provides a very_good.yaml file and verifies the file is parsed correctly and overrides the configuration based on it

@marcossevilla marcossevilla changed the title feat: support CLI parameter configuration via very_good.yaml feat: introduce very_good.yaml for param configuration Jul 2, 2026
@unicoderbot

unicoderbot Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

On it, I'll address the review comments and push the changes.

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Reviewing feedback
  • Working on changes
  • Pushing updates

I've pushed changes to address the feedback.

@marcossevilla marcossevilla changed the title feat: introduce very_good.yaml for param configuration feat(test): add very_good.yaml for config Jul 6, 2026
Comment thread e2e/test/commands/test/very_good_config/very_good_config_test.dart Outdated
Comment thread e2e/test/commands/test/very_good_config/very_good_config_test.dart Outdated
Comment thread test/src/commands/test/test_test.dart
Comment thread test/src/commands/test/test_test.dart Outdated
Comment thread test/src/very_good_config/very_good_config_test.dart Outdated
Comment thread .gitignore Outdated
@marcossevilla

Copy link
Copy Markdown
Member

@vgvbot /address-review

@unicoderbot

unicoderbot Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

On it, I'll address the review comments and push the changes.

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Reviewing feedback
  • Working on changes
  • Pushing updates

I've pushed changes to address the feedback.

Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@marcossevilla
marcossevilla marked this pull request as ready for review July 6, 2026 12:18
@marcossevilla
marcossevilla requested a review from a team as a code owner July 6, 2026 12:18
omartinma
omartinma previously approved these changes Jul 7, 2026
@marcossevilla

Copy link
Copy Markdown
Member

@vgvbot add test-reporter flag to very_good.yaml

@unicoderbot

unicoderbot Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

On it, I'll address the review comments and push the changes.

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Reviewing feedback
  • Working on changes
  • Pushing updates

I've pushed changes to address the feedback.

Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@marcossevilla
marcossevilla merged commit fed3074 into main Jul 22, 2026
24 checks passed
@marcossevilla
marcossevilla deleted the vgv-ai-bot/issue-360 branch July 22, 2026 13:58
@vgvbot vgvbot mentioned this pull request Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants