From: Brian Warner 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/?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)