Commit 88d826cf by Calen Pennington

Allow the diff_coverage command to share options with the coverage command

parent c3434d14
...@@ -267,7 +267,7 @@ def coverage(options): ...@@ -267,7 +267,7 @@ def coverage(options):
sh("coverage xml --rcfile={}".format(rcfile)) sh("coverage xml --rcfile={}".format(rcfile))
# Generate the coverage.py HTML report # Generate the coverage.py HTML report
sh("coverage html --rcfile={}".format(rcfile)) sh("coverage html --rcfile={}".format(rcfile))
call_task('diff_coverage', options=options.coverage) diff_coverage() # pylint: disable=no-value-for-parameter
@task @task
...@@ -275,12 +275,12 @@ def coverage(options): ...@@ -275,12 +275,12 @@ def coverage(options):
@cmdopts([ @cmdopts([
("compare-branch=", "b", "Branch to compare against, defaults to origin/master"), ("compare-branch=", "b", "Branch to compare against, defaults to origin/master"),
("compare_branch=", None, "deprecated in favor of compare-branch"), ("compare_branch=", None, "deprecated in favor of compare-branch"),
]) ], share_with=['coverage'])
def diff_coverage(options): def diff_coverage(options):
""" """
Build the diff coverage reports Build the diff coverage reports
""" """
compare_branch = options.diff_coverage.get('compare_branch', 'origin/master') compare_branch = options.get('compare_branch', 'origin/master')
# Find all coverage XML files (both Python and JavaScript) # Find all coverage XML files (both Python and JavaScript)
xml_reports = [] xml_reports = []
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment