From f326d5d7ce7b87b3cad81f4866d6298a077b2720 Mon Sep 17 00:00:00 2001 From: Fu Hanxi Date: Fri, 25 Oct 2024 12:49:58 +0200 Subject: [PATCH] ci: print esp-coredump output when failed in panic tests --- .../system/panic/test_panic_util/panic_dut.py | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/tools/test_apps/system/panic/test_panic_util/panic_dut.py b/tools/test_apps/system/panic/test_panic_util/panic_dut.py index 2619008fe0..ff8004cb85 100644 --- a/tools/test_apps/system/panic/test_panic_util/panic_dut.py +++ b/tools/test_apps/system/panic/test_panic_util/panic_dut.py @@ -151,9 +151,15 @@ class PanicTestDut(IdfDut): logging.info('espcoredump output is written to %s', self.coredump_output.name) self.serial.close() - subprocess.check_call(espcoredump_args, stdout=self.coredump_output) - self.coredump_output.flush() - self.coredump_output.seek(0) + try: + subprocess.check_call(espcoredump_args, stdout=self.coredump_output, stderr=self.coredump_output) + except subprocess.CalledProcessError: + self.coredump_output.flush() + with open(output_file_name, 'r') as file: + logging.error('espcoredump failed with output: %s', file.read()) + raise + finally: + self.coredump_output.seek(0) def process_coredump_uart( self, expected: Optional[List[Union[str, re.Pattern]]] = None, wait_reboot: bool = True