From 98c4f4dbeae22536b6f5fd83c6cec815fa2fa42f Mon Sep 17 00:00:00 2001 From: Clark Boylan Date: Wed, 27 Jul 2016 14:11:06 -0700 Subject: [PATCH] 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 --- .coveragerc | 3 +++ tox.ini | 2 -- 2 files changed, 3 insertions(+), 2 deletions(-) create mode 100644 .coveragerc diff --git a/.coveragerc b/.coveragerc new file mode 100644 index 000000000..f45d51a87 --- /dev/null +++ b/.coveragerc @@ -0,0 +1,3 @@ +[report] +include = ironicclient/* +omit = ironicclient/tests/functional/* diff --git a/tox.ini b/tox.ini index 1d3a0ecd4..3bde7cbb3 100644 --- a/tox.ini +++ b/tox.ini @@ -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}