diff options
| author |   Dmitry Valter <d-valter@yandex-team.ru> | 2022-03-26 15:43:14 +0000 |
|---|---|---|
| committer |   Damjan Marion <dmarion@me.com> | 2022-03-27 09:21:34 +0000 |
| commit | 3ace4d62e2dd23154f00dcc6eb7991477f7eb5c5 (patch) | |
| tree | feeb9e127a09b2e5f0dd349b671396f3004c52c4 /test/run_tests.py | |
| parent | 30bb344ab6c82d742d2e5a79f62f8d4868db16f1 (diff) | |
tests: fix core file message
Prevent crashing on nonexistent VPP binary path class member when creating
testsuite core message.
Type: fix
Fixes: b23ffd7ef216463c35b75c831e6a27e58971f4ec
Signed-off-by: Dmitry Valter <d-valter@yandex-team.ru>
Change-Id: Ib9b3dc8c69317e6561e5404bbdcbf672e417cbcd
Diffstat (limited to 'test/run_tests.py')
| -rw-r--r-- | test/run_tests.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/test/run_tests.py b/test/run_tests.py index 1d194ad96ca..7209ddfab5d 100644 --- a/test/run_tests.py +++ b/test/run_tests.py @@ -239,7 +239,7 @@ def stdouterr_reader_wrapper(unread_testcases, finished_unread_testcases, read_testcase = None -def handle_failed_suite(logger, last_test_temp_dir, vpp_pid): +def handle_failed_suite(logger, last_test_temp_dir, vpp_pid, vpp_binary): if last_test_temp_dir: # Need to create link in case of a timeout or core dump without failure lttd = os.path.basename(last_test_temp_dir) @@ -273,7 +273,7 @@ def handle_failed_suite(logger, last_test_temp_dir, vpp_pid): except Exception as e: logger.exception("Unexpected error running `file' utility " "on core-file") - logger.error(f"gdb {config.vpp_bin} {core_path}") + logger.error(f"gdb {vpp_binary} {core_path}") if vpp_pid: # Copy api post mortem @@ -319,7 +319,8 @@ def process_finished_testsuite(wrapped_testcase_suite, failed_wrapped_testcases.add(wrapped_testcase_suite) handle_failed_suite(wrapped_testcase_suite.logger, wrapped_testcase_suite.last_test_temp_dir, - wrapped_testcase_suite.vpp_pid) + wrapped_testcase_suite.vpp_pid, + wrapped_testcase_suite.last_test_vpp_binary,) return stop_run |