diff options
| author | Simon Glass <sjg@chromium.org> | 2022-07-30 20:57:05 -0600 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2022-08-09 11:55:41 -0600 |
| commit | 25980791b1848df7df6c273dc5578837cc84f853 (patch) | |
| tree | e67662583365e50adb7bfb020d54684ceb5750b2 /tools/dtoc/test_fdt.py | |
| parent | 3dd4e916324efc825a7ee8e412f5cf1ded839021 (diff) | |
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) |