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
commit59365bacfdddddb1cfec48084ca08528f2bf7d2a (patch)
tree2debafe565ec961a7373742be4b07b51490fceee /coverage/summary.py
parent737f8ffa024563fb8315f94cdf1484aa7bdb6eaa (diff)
downloadpython-coveragepy-59365bacfdddddb1cfec48084ca08528f2bf7d2a.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 c987765..33aa017 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)