Merge lp:~jpakkane/unity-lens-shopping/coverage into lp:unity-lens-shopping
Proposed by
Jussi Pakkanen
Status: | Merged |
---|---|
Approved by: | Paweł Stołowski |
Approved revision: | 39 |
Merged at revision: | 38 |
Proposed branch: | lp:~jpakkane/unity-lens-shopping/coverage |
Merge into: | lp:unity-lens-shopping |
Diff against target: |
254 lines (+173/-2) 6 files modified
Makefile.am (+1/-0) Makefile.am.coverage (+48/-0) configure.ac (+14/-0) m4/gcov.m4 (+86/-0) src/Makefile.am (+3/-0) tests/unit/Makefile.am (+21/-2) |
To merge this branch: | bzr merge lp:~jpakkane/unity-lens-shopping/coverage |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Paweł Stołowski (community) | Approve | ||
PS Jenkins bot | continuous-integration | Pending | |
Review via email: mp+148688@code.launchpad.net |
Commit message
Added coverage support and targets.
Description of the change
Added coverage support and targets.
To post a comment you must log in.
Could you also update test_markup_ cleaner_ SOURCES in tests/unit/ Makefile. am to include all vala files from src/ (except for config.vala and main.vala)? That way coverage report will be more accurate, as for now it's based on the only class that is unit-tested, and this results in unrealistic 100% function coverage...).
Apart from that it works fine.