Skip to content

Commit ca921fe

Browse files
committed
Added a switch to control generation of the code coverage html report, which now defaults to false. The report can be generated post-testing with "coverage html -d static/coverage/".
- Legacy-Id: 13821
1 parent 28f0efb commit ca921fe

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

ietf/utils/test_runner.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -352,7 +352,7 @@ def code_coverage_test(self):
352352
for pattern in settings.TEST_CODE_COVERAGE_EXCLUDE_LINES:
353353
checker.exclude(pattern)
354354
# Maybe output a html report
355-
if self.runner.run_full_test_suite:
355+
if self.runner.run_full_test_suite and self.runner.html_report:
356356
checker.html_report(directory=settings.TEST_CODE_COVERAGE_REPORT_DIR)
357357
# In any case, build a dictionary with per-file data for this run
358358
reporter = CoverageReporter(checker, checker.config)
@@ -444,14 +444,17 @@ def add_arguments(cls, parser):
444444
parser.add_argument('--skip-coverage',
445445
action='store_true', dest='skip_coverage', default=False,
446446
help='Skip test coverage measurements for code, templates, and URLs. ' )
447-
parser.add_argument('--save-version-coverage',
447+
parser.add_argument('--save-version-coverage', metavar='RELEASE_VERSION',
448448
action='store', dest='save_version_coverage', default=False,
449449
help='Save test coverage data under the given version label')
450450
parser.add_argument('--save-testresult',
451451
action='store_true', dest='save_testresult', default=False,
452452
help='Save short test result data in %s/.testresult' % os.path.dirname(os.path.dirname(settings.BASE_DIR))),
453+
parser.add_argument('--html-report',
454+
action='store_true', default=False,
455+
help='Generate a html code coverage report in %s' % settings.TEST_CODE_COVERAGE_REPORT_DIR)
453456

454-
def __init__(self, skip_coverage=False, save_version_coverage=None, **kwargs):
457+
def __init__(self, skip_coverage=False, save_version_coverage=None, html_report=None, **kwargs):
455458
#
456459
self.check_coverage = not skip_coverage
457460
self.save_version_coverage = save_version_coverage

0 commit comments

Comments
 (0)