Skip to content

feat: add --file-reporter support to test commands#1640

Merged
marcossevilla merged 3 commits into
mainfrom
vgv-ai-bot/issue-331
Jul 7, 2026
Merged

feat: add --file-reporter support to test commands#1640
marcossevilla merged 3 commits into
mainfrom
vgv-ai-bot/issue-331

Conversation

@unicoderbot

@unicoderbot unicoderbot Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds support for the --file-reporter option to both very_good test (Flutter) and very_good dart test commands. This option enables writing test results to a file in various formats (e.g., JSON) without affecting the CLI's stdout output.

Changes

  • Added --file-reporter option to Flutter test command that accepts format in <name>:<path> form (e.g., json:reports/tests.json)
  • Added --file-reporter option to Dart test command with the same format
  • Updated both FlutterTestOptions and DartTestOptions classes to parse and store the file reporter value
  • Updated argument passing to flutterTest() and dartTest() to include the --file-reporter flag when provided
  • Added tests to verify the option is correctly passed through
  • Updated documentation in site/docs/commands/test.md

Motivation

The file reporter output can be converted to popular formats like JUnit XML reports using tools like junitreport, enabling better integration with CI/CD platforms and IDEs.

Closes #331

Closes #331

Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@unicoderbot unicoderbot Bot requested a review from a team as a code owner July 6, 2026 13:37
Comment thread lib/src/commands/dart/commands/dart_test_command.dart Outdated
Comment thread lib/src/commands/dart/commands/dart_test_command.dart Outdated
Comment thread lib/src/commands/test/test.dart Outdated
Comment thread site/docs/commands/test.md Outdated
Comment thread test/src/commands/dart/commands/dart_test_test.dart Outdated
Comment thread test/src/commands/test/test_test.dart Outdated
@marcossevilla

Copy link
Copy Markdown
Member

@vgvbot /address-review

@unicoderbot

unicoderbot Bot commented Jul 7, 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 20a3943 into main Jul 7, 2026
23 checks passed
@marcossevilla marcossevilla deleted the vgv-ai-bot/issue-331 branch July 7, 2026 15:28
@vgvbot vgvbot mentioned this pull request Jul 7, 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.

Support for --machine in very_good test

1 participant