summaryrefslogtreecommitdiff
path: root/coverage
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
parentab861cc4535777fed86e6cb36b3608b4fc8c079d (diff)
downloadpython-coveragepy-git-7df1277770b99b7484e5e2687489f52a89327095.tar.gz
Clean up the --skip-covered pull request.
Diffstat (limited to 'coverage')
-rw-r--r--coverage/cmdline.py2
-rw-r--r--coverage/summary.py14
2 files changed, 12 insertions, 4 deletions
diff --git a/coverage/cmdline.py b/coverage/cmdline.py
index f776950a..3e3f7e44 100644
--- a/coverage/cmdline.py
+++ b/coverage/cmdline.py
@@ -63,7 +63,7 @@ class Opts(object):
"executed."
)
skip_covered = optparse.make_option(
- '-s', '--skip-covered', action='store_true',
+ '--skip-covered', action='store_true',
help="Skip files with 100% coverage."
)
omit = optparse.make_option(
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)