colin/u-boot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

binman: Support getting test-coverage on just one test

Browse Source 
Pass the arguments through to test_util so that a single test can be
used.

Update the docs and add some missing backquotes in the same section.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass
2024-09-30 12:51:37 -06:00
parent c95a3c8890
commit 3473e1779c
2 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
tools/binman/binman.rst
View File

@@ -2099,12 +2099,15 @@ Code coverage
------------- -------------
Binman is a critical tool and is designed to be very testable. Entry Binman is a critical tool and is designed to be very testable. Entry
implementations target 100% test coverage. Run 'binman test -T' to check this. implementations target 100% test coverage. Run ``binman test -T`` to check this.
To enable Python test coverage on Debian-type distributions (e.g. Ubuntu):: To enable Python test coverage on Debian-type distributions (e.g. Ubuntu)::
$ sudo apt-get install python-coverage python3-coverage python-pytest $ sudo apt-get install python-coverage python3-coverage python-pytest
You can also check the coverage provided by a single test, e.g.::
binman test -T testSimple
Exit status Exit status
----------- -----------

6
tools/binman/main.py
View File

@@ -85,7 +85,7 @@ def RunTests(debug, verbosity, processes, test_preserve_dirs, args, toolpath):
return (0 if result.wasSuccessful() else 1) return (0 if result.wasSuccessful() else 1)
def RunTestCoverage(toolpath, build_dir): def RunTestCoverage(toolpath, build_dir, args):
"""Run the tests and check that we get 100% coverage""" """Run the tests and check that we get 100% coverage"""
glob_list = control.GetEntryModules(False) glob_list = control.GetEntryModules(False)
all_set = set([os.path.splitext(os.path.basename(item))[0] all_set = set([os.path.splitext(os.path.basename(item))[0]
@@ -97,7 +97,7 @@ def RunTestCoverage(toolpath, build_dir):
test_util.run_test_coverage('tools/binman/binman', None, test_util.run_test_coverage('tools/binman/binman', None,
['*test*', '*main.py', 'tools/patman/*', 'tools/dtoc/*', ['*test*', '*main.py', 'tools/patman/*', 'tools/dtoc/*',
'tools/u_boot_pylib/*'], 'tools/u_boot_pylib/*'],
build_dir, all_set, extra_args or None) build_dir, all_set, extra_args or None, args=args)
def RunBinman(args): def RunBinman(args):
"""Main entry point to binman once arguments are parsed """Main entry point to binman once arguments are parsed
@@ -117,7 +117,7 @@ def RunBinman(args):
if args.cmd == 'test': if args.cmd == 'test':
if args.test_coverage: if args.test_coverage:
RunTestCoverage(args.toolpath, args.build_dir) RunTestCoverage(args.toolpath, args.build_dir, args.tests)
else: else:
ret_code = RunTests(args.debug, args.verbosity, args.processes, ret_code = RunTests(args.debug, args.verbosity, args.processes,
args.test_preserve_dirs, args.tests, args.test_preserve_dirs, args.tests,