Skip to content

Commit 2279047

Browse files
committed
fix(panic): fix reset checking in panic tests
1 parent e892415 commit 2279047

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

tools/test_apps/system/panic/pytest_panic.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -113,13 +113,15 @@ def get_default_backtrace(config: str) -> List[str]:
113113
return [config, 'app_main', 'main_task', 'vPortTaskWrapper']
114114

115115

116-
def expect_coredump_flash_write_logs(dut: PanicTestDut, config: str) -> None:
116+
def expect_coredump_flash_write_logs(dut: PanicTestDut, config: str, check_cpu_reset: Optional[bool] = True) -> None:
117117
dut.expect_exact('Save core dump to flash...')
118118
if 'extram_stack' in config:
119119
dut.expect_exact('Backing up stack @')
120120
dut.expect_exact('Restoring stack')
121121
dut.expect_exact('Core dump has been saved to flash.')
122122
dut.expect(dut.REBOOT)
123+
if check_cpu_reset:
124+
dut.expect_cpu_reset()
123125

124126

125127
def expect_coredump_uart_write_logs(dut: PanicTestDut, check_cpu_reset: Optional[bool] = True) -> Any:
@@ -173,8 +175,9 @@ def common_test(
173175
dut.process_coredump_uart(coredump_base64, expected_coredump)
174176
check_cpu_reset = False # CPU reset is already checked in expect_coredump_uart_write_logs
175177
elif 'flash' in config:
176-
expect_coredump_flash_write_logs(dut, config)
178+
expect_coredump_flash_write_logs(dut, config, check_cpu_reset)
177179
dut.process_coredump_flash(expected_coredump)
180+
check_cpu_reset = False # CPU reset is already checked in expect_coredump_flash_write_logs
178181
elif 'panic' in config:
179182
dut.expect(dut.REBOOT, timeout=60)
180183

0 commit comments

Comments
 (0)