Code Coverage dashboard should be able to filter out testonly files |
|
Issue descriptionIt is not as critical for test code to have 100% coverage, so there should be a view of the coverage dashboard with testonly files removed. For instance, [1] is a class we use for faking Safe Browsing results. Many of the methods are not called on it because our code under test does not need all of the functionality of the interface. [1]: https://chromium-coverage.appspot.com/reports/561573/linux/chromium/src/components/subresource_filter/content/browser/fake_safe_browsing_database_manager.cc.html
,
Aug 30
Does anyone have an idea on how to distinguish testonly files automatically?
,
Aug 31
Hey Max, you might be able to use GN for this. I don't have the perfect command but: testonly refs: gn refs <out_dir> <file> --testonly=true --type=executable --all all refs: gn refs <out_dir> <file> --type=executable --all If the output of both is the same, this is only used for tests. |
|
►
Sign in to add a comment |
|
Comment 1 by csharrison@chromium.org
, May 25 2018