Skip to content

Commit b2089c5

Browse files
author
MarcoFalke
committed
Merge #12438: [Tests] Fix trivial typo in test_runner.py causing error
ada1af6 Fix typo in test_runner.py causing error (MeshCollider) Pull request description: In the case that a test fails, the typo in run_tests() (introduced in #11858) will cause an error rather than printing out the combined logs, hiding the cause of the failure. Tree-SHA512: 7d7aa406d92750320ed20610cc5f174cdc94086f630af8c0c4db2003497132e0c56d59b94312fb42ad4507904a2fa858226a4a9337450930bf206183fc35c0a0
2 parents e782099 + ada1af6 commit b2089c5

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/functional/test_runner.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -342,7 +342,7 @@ def run_tests(test_list, src_dir, build_dir, exeext, tmpdir, jobs=1, enable_cove
342342
print('\n============')
343343
print('{}Combined log for {}:{}'.format(BOLD[1], testdir, BOLD[0]))
344344
print('============\n')
345-
combined_logs, _ = subprocess.Popen([sys.executble, os.path.join(tests_dir, 'combine_logs.py'), '-c', testdir], universal_newlines=True, stdout=subprocess.PIPE).communicate()
345+
combined_logs, _ = subprocess.Popen([sys.executable, os.path.join(tests_dir, 'combine_logs.py'), '-c', testdir], universal_newlines=True, stdout=subprocess.PIPE).communicate()
346346
print("\n".join(deque(combined_logs.splitlines(), combined_logs_len)))
347347

348348
print_results(test_results, max_len_name, (int(time.time() - time0)))

0 commit comments

Comments
 (0)