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>
This commit is contained in:
Simon Glass 2022-07-30 20:57:05 -06:00
parent 3dd4e91632
commit 25980791b1

View File

@ -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)