Browse Source

Fix coverage target

Previously the coverage target ended up deleting the coverage data when
the `coverage combine` command ran. This then caused the `coverage html`
command to fail due to lack of data to report on.

Thankfully the fix is simple because pbr is actually handlign all of
this for us behind the scenes. All we need to do is run with the
--coverage flag and make sure the .coveragerc reflects the desired
inclusions and ommisions.

Change-Id: Ide6c7639a19d961e4af27c2c59b036395e2af3ef
changes/43/348043/1
Clark Boylan 5 years ago
parent
commit
98c4f4dbea
2 changed files with 3 additions and 2 deletions
  1. +3
    -0
      .coveragerc
  2. +0
    -2
      tox.ini

+ 3
- 0
.coveragerc View File

@ -0,0 +1,3 @@
[report]
include = ironicclient/*
omit = ironicclient/tests/functional/*

+ 0
- 2
tox.ini View File

@ -30,8 +30,6 @@ setenv = VIRTUAL_ENV={envdir}
commands =
coverage erase
python setup.py testr --coverage --testr-args='{posargs}'
coverage combine
coverage html --include='ironicclient/*' --omit='ironicclient/tests/functional/*' -d covhtml -i
[testenv:venv]
commands = {posargs}


Loading…
Cancel
Save