CI coverage: build coverage report
Test coverage is now measured separately for each test run and combined together after that. Resulting HTML pages can be downloaded from artifacts. Publication as Gitlab pages should theoretically work but needs furher setup outside of repo.
Merge request reports
Activity
@pspacek I think I've done what needs to be done from https://about.gitlab.com/2016/11/03/publish-code-coverage-report-with-gitlab-pages/ here. I don't know how to access the pages though, I guess you need to set up a CNAME (or wildcard under gitlab.labs.nic.cz to be able to see the pages), see: https://docs.gitlab.com/ee/user/project/pages/introduction.html#add-a-custom-domain-to-your-pages-website
This is suspicious:
genhtml: WARNING: cannot read /builds/knot/knot-resolver/.local/lib/kdns_modules/priming.lua! genhtml: WARNING: cannot read /builds/knot/knot-resolver/.local/lib/kdns_modules/predict.lua! genhtml: WARNING: cannot read /builds/knot/knot-resolver/.local/lib/kdns_modules/kres.lua!
@vcunat The tests run after install (
installcheck
), so it tracks coverage of the installed files. I'll have to rewrite this into source paths at some point, but it's good enough to track coverage for now.The gitlab pages is for browsing coverage report without the need to download artifacts. It needs to be configured though in gitlab, which I'm not sure it is. If you already use it, please let me know how.
@vavrusam Our Gitlab guru is looking into it but we are not there yet. I will test and merge this when it is ready.
added tests label
added 45 commits
- 16a85117...07ff6fda - 43 commits from branch
master
- ba220f11 - ci: build coverage report and publish to pages
- a923c488 - fix gitignore
- 16a85117...07ff6fda - 43 commits from branch
added 1 commit
- da41be1f - WIP: fix cannot read .local/lib/kdns_modules errors
added 2 commits
- 890d517b - ci: build coverage report and publish to pages
- 9327d9c5 - fix gitignore
added 2 commits
- f4e52230 - CI: build coverage report and publish to pages
- 4d90664f - fix gitignore
added 3 commits
- 73ae7073 - CI: build coverage report and publish to pages
- a9aa1ccb - fix gitignore
- c0f74f55 - CI: measure branch test coverage
added 2 commits
- 86480efe - CI coverage: handle Lua code coverage properly
- 0b38fe1b - CI coverage: add missing Lua files to the report
added 2 commits
- da4a79d9 - daemon: gather luacov statistics from early start
- f34fbcef - WIP: parallel code coverage
added 1 commit
- a5d4bd1e - CI: measure test code coverage for parallel runs
added 1 commit
- e648eefa - CI: measure test code coverage for parallel runs
added 1 commit
- 6938f347 - CI: measure test code coverage for parallel runs
added 1 commit
- cc40fd7d - CI: parallelize Lua code coverage processing
added 8 commits
- 84fb5efc - CI coverage: build coverage report and publish to pages
- 0c5e1466 - fix gitignore
- 5a75e722 - CI coverage: handle Lua code coverage properly
- b5a447f9 - CI coverage: add missing Lua files to the report
- 77af3465 - daemon: gather luacov statistics from early start
- b2497b13 - CI coverage: merge test coverage data from parallel runs
- 3adb3418 - CI coverage: parallelize Lua code coverage processing
- db0d42c9 - fixup! build: added
make coverage
to compute C and Lua code coverage
Toggle commit listadded 1 commit
- 02b2ec64 - fixup! CI coverage: parallelize Lua code coverage processing
mentioned in commit 81a77557