diff options
author | Simon Glass <sjg@chromium.org> | 2022-07-31 05:57:05 +0300 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2022-08-09 20:55:41 +0300 |
commit | 25980791b1848df7df6c273dc5578837cc84f853 (patch) | |
tree | e67662583365e50adb7bfb020d54684ceb5750b2 /tools/dtoc/test_fdt.py | |
parent | 3dd4e916324efc825a7ee8e412f5cf1ded839021 (diff) | |
download | u-boot-25980791b1848df7df6c273dc5578837cc84f853.tar.xz |
dtoc: Tidy up fdt_tests RunTestCoverage() args
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@chromium.org>
Diffstat (limited to 'tools/dtoc/test_fdt.py')
-rwxr-xr-x | tools/dtoc/test_fdt.py | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/tools/dtoc/test_fdt.py b/tools/dtoc/test_fdt.py index 3baf4437cd..ec257552ef 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) |