From: Brian Warner <warner@allmydata.com>
Date: Tue, 1 May 2007 18:07:28 +0000 (-0700)
Subject: figleaf_htmlizer: when all counts are zero, claim 0% instead of 100%, since it probab... 
X-Git-Tag: allmydata-tahoe-0.2.0~22
X-Git-Url: https://git.rkrishnan.org/specifications/banana.xhtml?a=commitdiff_plain;h=0650e5543f283dc0cd6aea9a56849bc717642466;p=tahoe-lafs%2Ftahoe-lafs.git

figleaf_htmlizer: when all counts are zero, claim 0% instead of 100%, since it probably means that coverage checking has failed completely
---

diff --git a/src/allmydata/util/figleaf_htmlizer.py b/src/allmydata/util/figleaf_htmlizer.py
index 15c21cfa..579ab5a1 100644
--- a/src/allmydata/util/figleaf_htmlizer.py
+++ b/src/allmydata/util/figleaf_htmlizer.py
@@ -108,7 +108,7 @@ def report_as_html(coverage, directory, exclude_patterns=[], root=None):
 		try:
 			pcnt = n_covered * 100. / n_lines
 		except ZeroDivisionError:
-			pcnt = 100
+			pcnt = 0
 		info_dict[k] = (n_lines, n_covered, pcnt, display_filename)
 
 		html_outfile = make_html_filename(display_filename)
@@ -139,7 +139,7 @@ def report_as_html(coverage, directory, exclude_patterns=[], root=None):
 	summary_lines = sum([ v[0] for (k, v) in info_dict_items])
 	summary_cover = sum([ v[1] for (k, v) in info_dict_items])
 
-	summary_pcnt = 100
+	summary_pcnt = 0
 	if summary_lines:
 		summary_pcnt = float(summary_cover) * 100. / float(summary_lines)