michaelh/zephyr
1
0
Fork 0
Code Releases Activity

ztest: migrate error_hook tests to new API

Browse source 
Update the tests to use the new ztest API.

Signed-off-by: Yuval Peress <peress@google.com>
This commit is contained in:
Yuval Peress 2022-04-13 00:08:32 -06:00 • committed by Anas Nashif
commit fa13d57f01
3 changed files with 14 additions and 21 deletions
Download patch file Download diff file Expand all files Collapse all files

1
tests/ztest/error_hook/prj.conf
View file

@ -1,4 +1,5 @@
CONFIG_ZTEST=y CONFIG_ZTEST=y
CONFIG_ZTEST_NEW_API=y
CONFIG_TEST_USERSPACE=y CONFIG_TEST_USERSPACE=y
CONFIG_IRQ_OFFLOAD=y CONFIG_IRQ_OFFLOAD=y
CONFIG_ZTEST_FATAL_HOOK=y CONFIG_ZTEST_FATAL_HOOK=y

1
tests/ztest/error_hook/prj_no_userspace.conf
View file

@ -1,4 +1,5 @@
CONFIG_ZTEST=y CONFIG_ZTEST=y
CONFIG_ZTEST_NEW_API=y
CONFIG_IRQ_OFFLOAD=y CONFIG_IRQ_OFFLOAD=y
CONFIG_ZTEST_FATAL_HOOK=y CONFIG_ZTEST_FATAL_HOOK=y
CONFIG_ZTEST_ASSERT_HOOK=y CONFIG_ZTEST_ASSERT_HOOK=y

33
tests/ztest/error_hook/src/main.c
View file

@ -262,7 +262,7 @@ static int run_trigger_thread(int i)
* If the fatal error happened and the program enter assert_post_handler, * If the fatal error happened and the program enter assert_post_handler,
* that means fatal error triggered as expected. * that means fatal error triggered as expected.
*/ */
void test_catch_fatal_error(void) ZTEST_USER(error_hook_tests, test_catch_fatal_error)
{ {
#if defined(CONFIG_USERSPACE) #if defined(CONFIG_USERSPACE)
run_trigger_thread(ZTEST_CATCH_FATAL_ACCESS); run_trigger_thread(ZTEST_CATCH_FATAL_ACCESS);
@ -287,15 +287,20 @@ void test_catch_fatal_error(void)
* fail happened and the program enter assert_post_handler, that means * fail happened and the program enter assert_post_handler, that means
* assert works as expected. * assert works as expected.
*/ */
void test_catch_assert_fail(void) ZTEST_USER(error_hook_tests, test_catch_assert_fail)
{ {
case_type = ZTEST_CATCH_ASSERT_FAIL; case_type = ZTEST_CATCH_ASSERT_FAIL;
printk("1\n");
ztest_set_assert_valid(false); ztest_set_assert_valid(false);
printk("2\n");
ztest_set_assert_valid(true); ztest_set_assert_valid(true);
printk("3\n");
trigger_assert_fail(NULL); trigger_assert_fail(NULL);
printk("4\n");
ztest_test_fail(); ztest_test_fail();
} }
@ -313,7 +318,7 @@ static void tIsr_assert(const void *p)
* fail happened and the program enter assert_post_handler, that means * fail happened and the program enter assert_post_handler, that means
* assert works as expected. * assert works as expected.
*/ */
void test_catch_assert_in_isr(void) ZTEST(error_hook_tests, test_catch_assert_in_isr)
{ {
case_type = ZTEST_CATCH_ASSERT_IN_ISR; case_type = ZTEST_CATCH_ASSERT_IN_ISR;
irq_offload(tIsr_assert, NULL); irq_offload(tIsr_assert, NULL);
@ -337,7 +342,7 @@ static void trigger_z_oops(void)
* that means z_oops triggered as expected. This test only for * that means z_oops triggered as expected. This test only for
* userspace. * userspace.
*/ */
void test_catch_z_oops(void) ZTEST(error_hook_tests, test_catch_z_oops)
{ {
case_type = ZTEST_CATCH_USER_FATAL_Z_OOPS; case_type = ZTEST_CATCH_USER_FATAL_Z_OOPS;
@ -347,25 +352,11 @@ void test_catch_z_oops(void)
#endif #endif
void test_main(void) static void *error_hook_tests_setup(void)
{ {
#if defined(CONFIG_USERSPACE) #if defined(CONFIG_USERSPACE)
k_thread_access_grant(k_current_get(), &tdata, &tstack); k_thread_access_grant(k_current_get(), &tdata, &tstack);
ztest_test_suite(error_hook_tests,
ztest_user_unit_test(test_catch_assert_fail),
ztest_user_unit_test(test_catch_fatal_error),
ztest_unit_test(test_catch_z_oops),
ztest_unit_test(test_catch_assert_in_isr)
);
ztest_run_test_suite(error_hook_tests);
#else
ztest_test_suite(error_hook_tests,
ztest_unit_test(test_catch_fatal_error),
ztest_unit_test(test_catch_assert_fail),
ztest_unit_test(test_catch_assert_in_isr)
);
ztest_run_test_suite(error_hook_tests);
#endif #endif
return NULL;
} }
ZTEST_SUITE(error_hook_tests, NULL, error_hook_tests_setup, NULL, NULL, NULL);