From 9a69a43b43f89408fe7afcea76954cbdcf1c09b9 Mon Sep 17 00:00:00 2001 From: Ned Batchelder Date: Sun, 10 Jul 2016 17:15:43 -0400 Subject: Cover a few more lines in cmdline.py --- coverage/cmdline.py | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) (limited to 'coverage/cmdline.py') diff --git a/coverage/cmdline.py b/coverage/cmdline.py index 1b54e863..09e82323 100644 --- a/coverage/cmdline.py +++ b/coverage/cmdline.py @@ -216,7 +216,7 @@ class GlobalOptionParser(CoverageOptionParser): class CmdOptionParser(CoverageOptionParser): """Parse one of the new-style commands for coverage.py.""" - def __init__(self, action, options=None, defaults=None, usage=None, description=None): + def __init__(self, action, options, defaults=None, usage=None, description=None): """Create an OptionParser for a coverage.py command. `action` is the slug to put into `options.action`. @@ -233,8 +233,7 @@ class CmdOptionParser(CoverageOptionParser): description=description, ) self.set_defaults(action=action, **(defaults or {})) - if options: - self.add_options(options) + self.add_options(options) self.cmd = action def __eq__(self, other): @@ -277,7 +276,7 @@ CMDS = { [ Opts.append, ] + GLOBAL_ARGS, - usage=" ... ", + usage="[options] ... ", description=( "Combine data from multiple coverage files collected " "with 'run -p'. The combined results are written to a single " @@ -301,7 +300,6 @@ CMDS = { 'erase': CmdOptionParser( "erase", GLOBAL_ARGS, - usage=" ", description="Erase previously collected coverage data.", ), @@ -522,10 +520,9 @@ class CoverageScript(object): self.coverage.set_option("report:fail_under", options.fail_under) if self.coverage.get_option("report:fail_under"): - # Total needs to be rounded, but be careful of 0 and 100. - if 0 < total < 1: - total = 1 - elif 99 < total < 100: + # Total needs to be rounded, but don't want to report 100 + # unless it is really 100. + if 99 < total < 100: total = 99 else: total = round(total) -- cgit v1.2.1