diff options
| author | Robert Collins <rbtcollins@hp.com> | 2013-01-22 11:07:22 +1300 |
|---|---|---|
| committer | Robert Collins <rbtcollins@hp.com> | 2013-01-22 11:08:38 +1300 |
| commit | abd8957650641338e473db61cfb12f738a0d5d09 (patch) | |
| tree | 3a3472568bc0617ca7701a845e4ab37bbe833bc8 | |
| parent | fce9219eedf678b3f1bddc624c3233c105331b64 (diff) | |
| download | nova-abd8957650641338e473db61cfb12f738a0d5d09.tar.gz nova-abd8957650641338e473db61cfb12f738a0d5d09.tar.xz nova-abd8957650641338e473db61cfb12f738a0d5d09.zip | |
Generate coverage even if tests failed.
A test failing doesn't make coverage useless, so we should always
generate it.
Change-Id: I3ed9e54213c7fb6756906553920d19c5075e1adb
| -rwxr-xr-x | run_tests.sh | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/run_tests.sh b/run_tests.sh index 39176d78b..6c04abb5a 100755 --- a/run_tests.sh +++ b/run_tests.sh @@ -95,6 +95,13 @@ function run_tests { copy_subunit_log + if [ $coverage -eq 1 ]; then + echo "Generating coverage report in covhtml/" + # Don't compute coverage for common code, which is tested elsewhere + ${wrapper} coverage combine + ${wrapper} coverage html --include='nova/*' --omit='nova/openstack/common/*' -d covhtml -i + fi + return $RESULT } @@ -197,10 +204,3 @@ if [ -z "$testrargs" ]; then run_pep8 fi fi - -if [ $coverage -eq 1 ]; then - echo "Generating coverage report in covhtml/" - # Don't compute coverage for common code, which is tested elsewhere - ${wrapper} coverage combine - ${wrapper} coverage html --include='nova/*' --omit='nova/openstack/common/*' -d covhtml -i -fi |