diff options
author | garar <devnull@localhost> | 2015-08-14 12:44:23 +0000 |
---|---|---|
committer | garar <devnull@localhost> | 2015-08-14 12:44:23 +0000 |
commit | 6648c52f8563e7e29715e7d6d5bb8045e0a89601 (patch) | |
tree | 9cd3b42b1fd65e4513cd421c7f2d22e1141acfc4 /coverage/summary.py | |
parent | d51330ce8d1ccc566fc1e01aa660788f68ff07b8 (diff) | |
download | python-coveragepy-6648c52f8563e7e29715e7d6d5bb8045e0a89601.tar.gz |
Update skip_covered handling, add info when files are skipped. Tests.
Diffstat (limited to 'coverage/summary.py')
-rw-r--r-- | coverage/summary.py | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/coverage/summary.py b/coverage/summary.py index 2bf097b..9b8b233 100644 --- a/coverage/summary.py +++ b/coverage/summary.py @@ -29,6 +29,9 @@ class SummaryReporter(Reporter): max_name = max([len(fr.relative_filename()) for fr in self.file_reporters] + [5]) fmt_name = "%%- %ds " % max_name fmt_err = "%s %s: %s\n" + fmt_skip_covered = ("\n%s file%s skipped because of 'skip covered' " + "option.\n") + header = (fmt_name % "Name") + " Stmts Miss" fmt_coverage = fmt_name + "%6d %6d" if self.branches: @@ -52,6 +55,7 @@ class SummaryReporter(Reporter): outfile.write(rule) total = Numbers() + skipped_count = 0 for fr in self.file_reporters: try: @@ -66,6 +70,7 @@ class SummaryReporter(Reporter): else: no_missing_branches = True if no_missing_lines and no_missing_branches: + skipped_count += 1 continue args = (fr.relative_filename(), nums.n_statements, nums.n_missing) @@ -103,6 +108,10 @@ class SummaryReporter(Reporter): if self.config.show_missing: args += ("",) outfile.write(fmt_coverage % args) - if not total.n_files and not self.config.skip_covered: + if not total.n_files: raise CoverageException("No data to report.") + if self.config.skip_covered and skipped_count: + outfile.write( + fmt_skip_covered % (skipped_count, + 's' if skipped_count > 1 else '')) return total.n_statements and total.pc_covered |