openstack/aodh
Code Issues Proposed changes

add coverage config file to control module coverage report

Browse Source 
Ceilometer copies oslo common code into its code stream. When run
unit tests with coverage option, the overall test code coverage
percentage is low due to the enclosure of the common libraries.
Adding the coverage config file will exclude these modules in
the code coverage report so that developers can easily see if
new test cases has increased or decreased the code test coverage.

to see the code coverage report, following the following steps:

1. In ceilometer root directory, execute the following command
   ./run-tests --coverage
2. Once the test run completed successfully, open up the follwoing
   file in your browser and scroll down to the bottom to see the
   total number. (this patch makes that number to be 90%)
   <ceilometer_root_dir>/cover/index.html

To keep up with this effort, whenever you add new code, please
run the test code with coverage option and make sure your new
code and test won't decrease the total percentage. If it did,
that means you may have new code un-tested :-)

Change-Id: I277c243b6aadb0e8524915452326ed875e14ce9c
This commit is contained in:
Tong Li 2013-09-25 17:40:27 -04:00 committed by Gerrit Code Review
parent c535527048
commit 50a7dcf421
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
.coveragerc Normal file
View File

@ -0,0 +1,5 @@
[report]
# code copied from oslo openstack common code, we do not have
# test cases particularly for these modules, excluding them
# from coverage report
omit = ceilometer/openstack/common/*.py