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

We should add a coverage test to stratis-min testing #3299

Open
mulkieran opened this issue Mar 29, 2023 · 2 comments
Open

We should add a coverage test to stratis-min testing #3299

mulkieran opened this issue Mar 29, 2023 · 2 comments
Assignees

Comments

@mulkieran
Copy link
Member

It should be easy to get complete coverage information for the stratis-min chunk of code.

@mulkieran mulkieran added this to To do in 2024April via automation Apr 11, 2024
@mulkieran mulkieran moved this from To do to In progress (long term) in 2024April Apr 11, 2024
@mulkieran
Copy link
Member Author

The first step is getting the complete coverage. As ever, we will have to exclude a few things, but llvm-cov tool does not have a convenient way of annotating lines, unlike the standard Python coverage tools. This means that there is more of a benefit to using, e..,g., codecov because we can get changes in the coverage induced by a PR, which is an informative kind of information. See example: rust-netlink/netlink-packet-route#59 (comment)

@mulkieran mulkieran removed this from In progress (long term) in 2024April Apr 29, 2024
@mulkieran mulkieran added this to To do in 2024May via automation Apr 29, 2024
@mulkieran mulkieran moved this from To do to In progress (long term) in 2024May Apr 29, 2024
@mulkieran
Copy link
Member Author

Coverage at 64.8%.

@mulkieran mulkieran removed this from In progress (long term) in 2024May May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In progress (long term)
Development

No branches or pull requests

2 participants