diff options
author | Simon Glass <sjg@chromium.org> | 2022-07-31 05:57:06 +0300 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2022-08-09 20:55:41 +0300 |
commit | b26dd9648c48d41c1486a623012363167c7a5fc7 (patch) | |
tree | 982bb99014d42f9e3022687da58486d9b3b2069c /tools/dtoc/test_fdt.py | |
parent | 25980791b1848df7df6c273dc5578837cc84f853 (diff) | |
download | u-boot-b26dd9648c48d41c1486a623012363167c7a5fc7.tar.xz |
dtoc: Tidy up fdt_tests RunTests()
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>
Diffstat (limited to 'tools/dtoc/test_fdt.py')
-rwxr-xr-x | tools/dtoc/test_fdt.py | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/tools/dtoc/test_fdt.py b/tools/dtoc/test_fdt.py index ec257552ef..b48819831d 100755 --- a/tools/dtoc/test_fdt.py +++ b/tools/dtoc/test_fdt.py @@ -778,17 +778,20 @@ def run_test_coverage(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 Args: - args: List of positional args provided to fdt. This can hold a test - name to execute (as in 'fdt -t testFdt', for example) + args (list or str): List of positional args provided. This can hold a + 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: - 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( 'test_fdt', False, False, False, None, test_name, None, [TestFdt, TestNode, TestProp, TestFdtUtil]) @@ -812,7 +815,7 @@ parser.add_option('-T', '--test-coverage', action='store_true', # Run our meagre tests if options.test: - ret_code = RunTests(args) + ret_code = run_tests(args, options.processes) sys.exit(ret_code) elif options.test_coverage: run_test_coverage(options.build_dir) |