Merge lp:~allanlesage/coverage-builder/test-fifty-percent-coverage into lp:coverage-builder
Proposed by
Allan LeSage
Status: | Merged |
---|---|
Merged at revision: | 275 |
Proposed branch: | lp:~allanlesage/coverage-builder/test-fifty-percent-coverage |
Merge into: | lp:coverage-builder |
Prerequisite: | lp:~allanlesage/coverage-builder/build-test |
Diff against target: |
38 lines (+24/-0) 1 file modified
test/test_test.py (+24/-0) |
To merge this branch: | bzr merge lp:~allanlesage/coverage-builder/test-fifty-percent-coverage |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Martin Pitt (community) | Needs Fixing | ||
Review via email: mp+234754@code.launchpad.net |
Description of the change
Quickie test for 50% coverage.
Martin you earlier suggested marking up the code with tags to extract--is just substituting the file adequate?
To post a comment you must log in.
Sure, replacing the entire file works too. I just meant that replacing a macro comment might be easier than always rewriting entire files for some tests.
The assertion needs to be fixed, otherwise LGTM. (Please just merge after fixing)