openstack/placement
Code Issues Proposed changes

Use both unit and functional for coverage testing

Browse Source 
Placement has always relied heavily on functional testing for both API
and database testing, so not including those tests in 'coverage' leads
to misleading results.

This change adjusts the cover job in tox to use a test path of
placement/tests, resulting in all the tests running.

In the process .coveragerc is updated to be placement oriented
and make sure we do not ignore branches.

This is in preparation for turning on a check job for coverage.
See this email for more on that:

http://lists.openstack.org/pipermail/openstack-dev/2018-September/134385.html

A followup patch will turn on a check job for it.

Change-Id: I6c122facc9c9f6e0b2a1d69c6aaf5faeee7cf253
This commit is contained in:
Chris Dent 2018-09-06 19:25:33 +01:00
parent 868e443b09
commit 09905baab2
2 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
.coveragerc
View File

@ -1,7 +1,4 @@
[run] [run]
branch = True branch = True
source = nova source = placement
omit = nova/tests/* omit = placement/tests/*
[report]
ignore_errors = True

2
tox.ini
View File

@ -121,7 +121,7 @@ setenv =
commands = commands =
{[testenv]commands} {[testenv]commands}
coverage erase coverage erase
stestr run {posargs} stestr --test-path=./placement/tests --group_regex=placement\.tests\.functional\.test_api(?:\.|_)([^_]+) run {posargs}
coverage combine coverage combine
coverage html -d cover coverage html -d cover
coverage xml -o cover/coverage.xml coverage xml -o cover/coverage.xml