Mirrors/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2026-10-02 11:10:54 +03:00
Code Issues Packages Projects Releases Wiki Activity

test(panic): fix rtc_fast_reg2 skip marker on no-PMS targets (backport v5.5)

Browse Source 
The skipif marker on test_rtc_fast_reg2_execute_violation evaluates
config.getvalue("target"), but in CI pytest is invoked by parametrized
node-id with no --target option, so the predicate is never true and the
test is not skipped on esp32c5/esp32c6/esp32h2/esp32p4. These targets have
no PMS peripheral case for executing from RTC-fast memory, so they raise a
generic "Illegal instruction" panic instead of the "Memory protection
fault" the test expects, and fail on every run.

Replace skipif with the temp_skip marker (already done on master in
288fc561b7). The idf_pytest plugin evaluates temp_skip against the
parametrized target, so these targets are skipped correctly. esp32c3 and
esp32s2 keep running, as they do raise a real memory protection fault.

Closes IDFCI-8848
Closes IDFCI-10459

Co-authored-by: Hrushikesh Bhosale <hrushikesh.bhosale@espressif.com>
This commit is contained in:
Aditya Patwardhan
2026-08-17 14:13:16 +02:00
committed by Fu Hanxi
co-authored by Hrushikesh Bhosale
parent 97ba24fe55
commit 498784008d
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tools/test_apps/system/panic/pytest_panic.py
+3 -3
View File
@@ -1873,9 +1873,9 @@ def test_rtc_fast_reg1_execute_violation(dut: PanicTestDut, test_func_name: str)
@pytest.mark.generic
@pytest.mark.skipif(
'config.getvalue("target") in ["esp32c5", "esp32c6", "esp32h2", "esp32p4"]',
reason='Not a violation condition, no PMS peripheral case',
@pytest.mark.temp_skip(
targets=['esp32c5', 'esp32c6', 'esp32h2', 'esp32p4'],
reason='Not a violation condition, no PMS peripheral cases',
)
@idf_parametrize(
'config,target',