From 5a3f53a2b5f9c9ce65817986a28887a47daef131 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 b19b32d539..ba104f4af8 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 @@ -147,9 +147,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