Coverage for Integration test on Package#93620
Coverage for Integration test on Package#93620hrishikesh-kadam wants to merge 1 commit intoflutter:masterfrom
Conversation
This will help to generate coverage for the Package when running Integration test on the Package's example like project. For example, ``` $ cd my-package/example $ flutter test integration_test --coverage --coverage-for-parent-package ``` Coverage will be stored at `my-package/coverage/lcov.info`.
flutter-dashboard
bot
added
the
tool
|
It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat. If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix? Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing. |
|
@jiahaog Requesting your review on this PR. |
|
@jiahaog do you think this is the right change to make? |
|
I'm sorry @hrishikesh-kadam, but I'm going to close this PR as I am not sure this is the correct approach. Feel free to discuss on your feature request: #93619 |
2 participants
This will help to generate coverage for the Package when running
Integration test on the Package's example like project.
For example,
Coverage will be stored at
my-package/coverage/lcov.info.List which issues are fixed by this PR. You must list at least one issue.
Use case Package repo - flutter_package_integration_testing_coverage
It would be appropriate that the Flutter Authors take this PR forward to achieve the above feature request.
Pre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.