michaelh/zephyr
1
0
Fork 0
Code Releases Activity

tests: arch: arm: Correctly report skipped tests as skipped

Browse source 
test_arm_user_interrupt and test_arm_null_pointer_exception currently
only print "Skipped" but reports as PASS in twister. This fixes that
misleading report.

Signed-off-by: Wilfried Chauveau <wilfried.chauveau@arm.com>
This commit is contained in:
Wilfried Chauveau 2025-01-30 10:06:59 +00:00 • committed by Benjamin Cabé
commit 1ad33e2d71
1 changed files with 3 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

12
tests/arch/arm/arm_interrupt/src/arm_interrupt.c
View file

@ -455,16 +455,17 @@ ZTEST_USER(arm_interrupt, test_arm_user_interrupt)
#else #else
ZTEST_USER(arm_interrupt, test_arm_user_interrupt) ZTEST_USER(arm_interrupt, test_arm_user_interrupt)
{ {
TC_PRINT("Skipped\n"); ztest_test_skip();
} }
#endif /* CONFIG_USERSPACE */ #endif /* CONFIG_USERSPACE */
#if defined(CONFIG_CORTEX_M_NULL_POINTER_EXCEPTION)
#pragma GCC push_options #pragma GCC push_options
#pragma GCC optimize("O0") #pragma GCC optimize("O0")
/* Avoid compiler optimizing null pointer de-referencing. */ /* Avoid compiler optimizing null pointer de-referencing. */
ZTEST(arm_interrupt, test_arm_null_pointer_exception) ZTEST(arm_interrupt, test_arm_null_pointer_exception)
{ {
Z_TEST_SKIP_IFNDEF(CONFIG_CORTEX_M_NULL_POINTER_EXCEPTION);
int reason; int reason;
struct test_struct { struct test_struct {
@ -483,13 +484,6 @@ ZTEST(arm_interrupt, test_arm_null_pointer_exception)
"expected_reason has not been reset (%d)\n", reason); "expected_reason has not been reset (%d)\n", reason);
} }
#pragma GCC pop_options #pragma GCC pop_options
#else
ZTEST(arm_interrupt, test_arm_null_pointer_exception)
{
TC_PRINT("Skipped\n");
}
#endif /* CONFIG_CORTEX_M_NULL_POINTER_EXCEPTION */
/** /**
* @} * @}