summaryrefslogtreecommitdiff
path: root/coverage/summary.py
diff options
context:
space:
mode:
authorNed Batchelder <ned@nedbatchelder.com>2014-11-23 07:52:53 -0500
committerNed Batchelder <ned@nedbatchelder.com>2014-11-23 07:52:53 -0500
commit7df1277770b99b7484e5e2687489f52a89327095 (patch)
treedfb81f388d09346361fb140061cefcdc1dabe06d /coverage/summary.py
parentab861cc4535777fed86e6cb36b3608b4fc8c079d (diff)
downloadpython-coveragepy-git-7df1277770b99b7484e5e2687489f52a89327095.tar.gz
Clean up the --skip-covered pull request.
Diffstat (limited to 'coverage/summary.py')
-rw-r--r--coverage/summary.py14
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)