From af19e67961a398caa899a7585ffe40f4b44bc5c1 Mon Sep 17 00:00:00 2001 From: Paul Goins Date: Fri, 1 Mar 2019 11:02:20 +0100 Subject: [PATCH] Added tox environment for gathering coverage This technique was borrowed from the tox "cover" environment in openstack/nova's tox.ini. This leverages the fact that stestr lets you override the python executable via the PYTHON environment variable. Doing this allows us to easily generate coverage for our unit tests. An important caveat is that this does not provide any coverage for tests via zaza, amulet, etc. It is purely focused on the unit tests. Note that this replaces the previous .coveragerc; coverage configuration is instead pulled from tox.ini. Change-Id: I2a14d3a3b0215aa637bc18e780d1ad765c796daa --- .coveragerc | 6 ------ .zuul.yaml | 1 + tox.ini | 27 +++++++++++++++++++++++++++ 3 files changed, 28 insertions(+), 6 deletions(-) delete mode 100644 .coveragerc diff --git a/.coveragerc b/.coveragerc deleted file mode 100644 index 3608a21..0000000 --- a/.coveragerc +++ /dev/null @@ -1,6 +0,0 @@ -[report] -# Regexes for lines to exclude from consideration -exclude_lines = - if __name__ == .__main__.: -include= - hooks/cinder_* diff --git a/.zuul.yaml b/.zuul.yaml index 7051aee..dc27661 100644 --- a/.zuul.yaml +++ b/.zuul.yaml @@ -1,3 +1,4 @@ - project: templates: - python35-charm-jobs + - openstack-cover-jobs diff --git a/tox.ini b/tox.ini index 1457901..7041c83 100644 --- a/tox.ini +++ b/tox.ini @@ -39,6 +39,33 @@ deps = -r{toxinidir}/requirements.txt commands = flake8 {posargs} hooks unit_tests tests actions lib charm-proof +[testenv:cover] +# Technique based heavily upon +# https://github.com/openstack/nova/blob/master/tox.ini +basepython = python3 +deps = -r{toxinidir}/requirements.txt + -r{toxinidir}/test-requirements.txt +setenv = + {[testenv]setenv} + PYTHON=coverage run +commands = + coverage erase + ostestr {posargs} + coverage combine + coverage html -d cover + coverage xml -o cover/coverage.xml + coverage report + +[coverage:run] +branch = True +concurrency = multiprocessing +parallel = True +source = + . +omit = + .tox/* + */charmhelpers/* + [testenv:venv] basepython = python3 commands = {posargs}