random-mirrors/smaeul-u-boot
1
0
Fork 0
mirror of https://github.com/smaeul/u-boot.git synced 2025-10-14 12:56:00 +01:00
Code Issues Packages Projects Releases Wiki Activity

dtoc: Tidy up fdt_tests RunTests()

Browse Source 
Pass the options args in rather than using the global variables. Use snake
case, fix up comments and use a ternary operator to make pylint happy.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2022-07-30 20:57:06 -06:00
parent 25980791b1
commit b26dd9648c
1 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
tools/dtoc/test_fdt.py
View File

@ -778,17 +778,20 @@ def run_test_coverage(build_dir):
['tools/patman/*.py', '*test_fdt.py'], build_dir) ['tools/patman/*.py', '*test_fdt.py'], build_dir)
def RunTests(args): def run_tests(args, processes):
"""Run all the test we have for the fdt model """Run all the test we have for the fdt model
Args: Args:
args: List of positional args provided to fdt. This can hold a test args (list or str): List of positional args provided. This can hold a
name to execute (as in 'fdt -t testFdt', for example) test name to execute (as in 'test_fdt -t testFdt', for example)
processes (int): Number of processes to use (None means as many as there
are CPUs on the system. This must be set to 1 when running under
the python3-coverage tool
Returns: Returns:
Return code, 0 on success int: Return code, 0 on success
""" """
test_name = args and args[0] or None test_name = args[0] if args else None
result = test_util.run_test_suites( result = test_util.run_test_suites(
'test_fdt', False, False, False, None, test_name, None, 'test_fdt', False, False, False, None, test_name, None,
[TestFdt, TestNode, TestProp, TestFdtUtil]) [TestFdt, TestNode, TestProp, TestFdtUtil])
@ -812,7 +815,7 @@ parser.add_option('-T', '--test-coverage', action='store_true',
# Run our meagre tests # Run our meagre tests
if options.test: if options.test:
ret_code = RunTests(args) ret_code = run_tests(args, options.processes)
sys.exit(ret_code) sys.exit(ret_code)
elif options.test_coverage: elif options.test_coverage:
run_test_coverage(options.build_dir) run_test_coverage(options.build_dir)