Re: [PATCH 1/3] u_boot_pylib: Support running coverage on selected functions

2024-10-17 Thread Simon Glass
At present run_test_coverage() assumes you want code coverage for the
entire code base. This is the normal situation, but sometimes it is
useful to see the coverage provided by just a single test. Add support
for this.

Signed-off-by: Simon Glass 
---

 tools/u_boot_pylib/test_util.py | 13 -
 1 file changed, 8 insertions(+), 5 deletions(-)

Applied to u-boot-dm, thanks!


[PATCH 1/3] u_boot_pylib: Support running coverage on selected functions

2024-09-30 Thread Simon Glass
At present run_test_coverage() assumes you want code coverage for the
entire code base. This is the normal situation, but sometimes it is
useful to see the coverage provided by just a single test. Add support
for this.

Signed-off-by: Simon Glass 
---

 tools/u_boot_pylib/test_util.py | 13 -
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/tools/u_boot_pylib/test_util.py b/tools/u_boot_pylib/test_util.py
index 857ce58c98c..dd671965263 100644
--- a/tools/u_boot_pylib/test_util.py
+++ b/tools/u_boot_pylib/test_util.py
@@ -23,8 +23,9 @@ except:
 use_concurrent = False
 
 
-def run_test_coverage(prog, filter_fname, exclude_list, build_dir, 
required=None,
-extra_args=None, single_thread='-P1'):
+def run_test_coverage(prog, filter_fname, exclude_list, build_dir,
+  required=None, extra_args=None, single_thread='-P1',
+  args=None):
 """Run tests and check that we get 100% coverage
 
 Args:
@@ -42,6 +43,7 @@ def run_test_coverage(prog, filter_fname, exclude_list, 
build_dir, required=None
 single_thread (str): Argument string to make the tests run
 single-threaded. This is necessary to get proper coverage results.
 The default is '-P0'
+args (list of str): List of tests to run, or None to run all
 
 Raises:
 ValueError if the code coverage is not 100%
@@ -66,9 +68,10 @@ def run_test_coverage(prog, filter_fname, exclude_list, 
build_dir, required=None
'coverage')
 
 cmd = ('%s%s run '
-   '--omit "%s" %s %s %s %s' % (prefix, covtool, ','.join(glob_list),
-prog, extra_args or '', test_cmd,
-single_thread or '-P1'))
+   '--omit "%s" %s %s %s %s %s' % (prefix, covtool, 
','.join(glob_list),
+   prog, extra_args or '', test_cmd,
+   single_thread or '-P1',
+   ' '.join(args) if args else ''))
 os.system(cmd)
 stdout = command.output(covtool, 'report')
 lines = stdout.splitlines()
-- 
2.34.1