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 reporting #107

Merged
merged 2 commits into from
Jan 9, 2017
Merged

Conversation

cdecker
Copy link
Member

@cdecker cdecker commented Jan 9, 2017

Setting the environment variable COVERAGE=1 allows us to turn coverage measurements on. This produces a number of auxiliary gcno and gcda files which can then be converted into coverage reports. make coverage then allows us to generate HTML reports.

I know line coverage and branch coverage is not a perfect metric, but it might get us some more visibility into where more tests might be needed.

Setting the environment variable `COVERAGE=1` allows us to turn
coverage measurements on. This produces a number of auxiliary gcno and
gcda files which can then be converted into coverage reports.

I know line coverage and branch coverage is not a perfect metric, but
it might get us some more visibility into where more tests might be
needed.
Added coverage files to `clean` and `coverage` target to generate HTML
report of coverage.
@cdecker cdecker requested a review from rustyrussell January 9, 2017 10:21
@rustyrussell rustyrussell merged commit 07fbeee into ElementsProject:master Jan 9, 2017
@cdecker cdecker deleted the coverage branch January 18, 2017 13:35
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.

2 participants