[PATCH 1/7] dtoc: Tidy up fdt_tests RunTestCoverage() args
Simon Glass
sjg at chromium.org
Sun Jul 31 04:57:05 CEST 2022
Pass the options args in rather than using the global various. Use snake
case and fix up comments to make pylint happy.
Signed-off-by: Simon Glass <sjg at chromium.org>
---
tools/dtoc/test_fdt.py | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/tools/dtoc/test_fdt.py b/tools/dtoc/test_fdt.py
index 3baf4437cdd..ec257552efc 100755
--- a/tools/dtoc/test_fdt.py
+++ b/tools/dtoc/test_fdt.py
@@ -768,10 +768,14 @@ class TestFdtUtil(unittest.TestCase):
tools.outdir= old_outdir
-def RunTestCoverage():
- """Run the tests and check that we get 100% coverage"""
+def run_test_coverage(build_dir):
+ """Run the tests and check that we get 100% coverage
+
+ Args:
+ build_dir (str): Directory containing the build output
+ """
test_util.run_test_coverage('tools/dtoc/test_fdt.py', None,
- ['tools/patman/*.py', '*test_fdt.py'], options.build_dir)
+ ['tools/patman/*.py', '*test_fdt.py'], build_dir)
def RunTests(args):
@@ -811,4 +815,4 @@ if options.test:
ret_code = RunTests(args)
sys.exit(ret_code)
elif options.test_coverage:
- RunTestCoverage()
+ run_test_coverage(options.build_dir)
--
2.37.1.455.g008518b4e5-goog
More information about the U-Boot
mailing list