Change in vdsm[master]: automation: Limit coverage report to vdsm code

nsoffer at redhat.com nsoffer at redhat.com
Thu Nov 26 21:33:51 UTC 2015


Nir Soffer has uploaded a new change for review.

Change subject: automation: Limit coverage report to vdsm code
......................................................................

automation: Limit coverage report to vdsm code

Current coverage report includes the tests, test helpers, 3rd party
libraries and maybe more.

This patch limit the coverage report to files under vdsm and lib
directories.

This change also fixes bogus results seen in vdsm library modules. I
don't know why the results were wrong, but limiting the output seems to
eliminate them.

Change-Id: I1d62578e0541ee82b70e064417b09c44252dd9b5
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
---
M automation/check-patch.sh
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/80/49280/1

diff --git a/automation/check-patch.sh b/automation/check-patch.sh
index 1f3ae0e..4d61269 100755
--- a/automation/check-patch.sh
+++ b/automation/check-patch.sh
@@ -7,7 +7,7 @@
 set -xe
 
 ./autogen.sh --system --enable-hooks
-make check NOSE_WITH_COVERAGE=1
+make check NOSE_WITH_COVERAGE=1 NOSE_COVER_PACKAGE="$PWD/vdsm,$PWD/lib"
 
 ./automation/build-artifacts.sh
 
@@ -22,4 +22,4 @@
 pushd "$TESTS_PATH"
 coverage html
 popd
-mv "$COVERAGE_REPORT" "$EXPORT_PATH/"
\ No newline at end of file
+mv "$COVERAGE_REPORT" "$EXPORT_PATH/"


-- 
To view, visit https://gerrit.ovirt.org/49280
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1d62578e0541ee82b70e064417b09c44252dd9b5
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>


More information about the vdsm-patches mailing list