diff options
author | Marcus Cobden <mercurial@marcuscobden.co.uk> | 2012-10-31 13:43:08 +0000 |
---|---|---|
committer | Marcus Cobden <mercurial@marcuscobden.co.uk> | 2012-10-31 13:43:08 +0000 |
commit | 5d779a0469879d7f9ecd0053a451473dc26eafc5 (patch) | |
tree | e161740931f88a7b7518ad20ddd1c9657a6f323f /coverage/xmlreport.py | |
parent | 33022cba9ed2d8efe8245e4a7d9aba66075d6811 (diff) | |
download | python-coveragepy-5d779a0469879d7f9ecd0053a451473dc26eafc5.tar.gz |
Improve file and package names in xml output
Diffstat (limited to 'coverage/xmlreport.py')
-rw-r--r-- | coverage/xmlreport.py | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/coverage/xmlreport.py b/coverage/xmlreport.py index 03f910c..82db821 100644 --- a/coverage/xmlreport.py +++ b/coverage/xmlreport.py @@ -89,9 +89,11 @@ class XmlReporter(Reporter): # Create the 'lines' and 'package' XML elements, which # are populated later. Note that a package == a directory. - dirname, fname = os.path.split(cu.name) - dirname = dirname or '.' - package = self.packages.setdefault(dirname, [ {}, 0, 0, 0, 0 ]) + package_name, _ = ('.' + cu.name).rsplit('.', 1) + package_name = package_name[1:] + className = cu.name + + package = self.packages.setdefault(package_name, [{}, 0, 0, 0, 0]) xclass = self.xml_out.createElement("class") @@ -99,10 +101,9 @@ class XmlReporter(Reporter): xlines = self.xml_out.createElement("lines") xclass.appendChild(xlines) - className = fname.replace('.', '_') + xclass.setAttribute("name", className) - ext = os.path.splitext(cu.filename)[1] - xclass.setAttribute("filename", cu.name + ext) + xclass.setAttribute("filename", cu.file_locator.relative_filename(cu.filename)) xclass.setAttribute("complexity", "0") branch_stats = analysis.branch_stats() |