diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2014-11-23 07:52:53 -0500 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2014-11-23 07:52:53 -0500 |
commit | 7df1277770b99b7484e5e2687489f52a89327095 (patch) | |
tree | dfb81f388d09346361fb140061cefcdc1dabe06d /coverage/summary.py | |
parent | ab861cc4535777fed86e6cb36b3608b4fc8c079d (diff) | |
download | python-coveragepy-git-7df1277770b99b7484e5e2687489f52a89327095.tar.gz |
Clean up the --skip-covered pull request.
Diffstat (limited to 'coverage/summary.py')
-rw-r--r-- | coverage/summary.py | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/coverage/summary.py b/coverage/summary.py index c9877658..33aa017f 100644 --- a/coverage/summary.py +++ b/coverage/summary.py @@ -54,9 +54,17 @@ class SummaryReporter(Reporter): try: analysis = self.coverage._analyze(cu) nums = analysis.numbers - if self.config.skip_covered and nums.n_missing == 0 and \ - (not self.branches or nums.n_partial_branches == 0): - continue + + if self.config.skip_covered: + # Don't report on 100% files. + no_missing_lines = (nums.n_missing == 0) + if self.branches: + no_missing_branches = (nums.n_partial_branches == 0) + else: + no_missing_branches = True + if no_missing_lines and no_missing_branches: + continue + args = (cu.name, nums.n_statements, nums.n_missing) if self.branches: args += (nums.n_branches, nums.n_partial_branches) |