Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add --coverage-package to cli #2491

Open
FritzMatthaeus opened this issue Jan 15, 2025 · 1 comment
Open

add --coverage-package to cli #2491

FritzMatthaeus opened this issue Jan 15, 2025 · 1 comment
Labels
feature New feature request

Comments

@FritzMatthaeus
Copy link
Contributor

Use case

i have an app structured as a monorepo (without Melos, just a dart monorepo as all packages are only local):

structure:

  • packages/foo
  • lib/myapp

Running patrol test --coverage only covers the app files (which are in /lib) but not the files in the packages which contain the major part of the UI. This might be a case of a badly structured monorepo but it is what it is. It would be great if foo would be covered too.

I have found that flutter test does support including packages into the coverage by running:
flutter test --coverage --coverage-package foo

currently this is not supported by the Patrol CLI. For my purpose this would be a valuable feature

Proposal

include --coverage-package [mypackage] to Patrol CLI like
patrol test --coverage --coverage-package foo

@zoskar zoskar added the feature New feature request label Jan 17, 2025
@pdenert
Copy link
Collaborator

pdenert commented Jan 29, 2025

@gogolon could you please take a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature request
Projects
None yet
Development

No branches or pull requests

3 participants