From: Kevin Day Date: Sun, 16 Jan 2022 22:49:43 +0000 (-0600) Subject: Update: Add parameter checking to unit tests and fix ordering. X-Git-Tag: 0.5.8~106 X-Git-Url: https://git.kevux.org/?a=commitdiff_plain;h=7daa0a5e405efe0ba056079e2d77975e0a17d6a0;p=fll Update: Add parameter checking to unit tests and fix ordering. --- diff --git a/level_0/f_memory/tests/c/test-memory-adjust.c b/level_0/f_memory/tests/c/test-memory-adjust.c index fa6bb47ad..892e2d5d0 100644 --- a/level_0/f_memory/tests/c/test-memory-adjust.c +++ b/level_0/f_memory/tests/c/test-memory-adjust.c @@ -5,6 +5,23 @@ extern "C" { #endif +#ifndef _di_level_0_parameter_checking_ + void test__f_memory_adjust__parameter_checking(void **state) { + + { + const f_status_t status = f_memory_adjust(1, 4, 0, 0); + + assert_int_equal(F_status_set_fine(status), F_parameter); + } + + { + const f_status_t status = f_memory_adjust(1, 4, 1, 0); + + assert_int_equal(F_status_set_fine(status), F_parameter); + } + } +#endif // _di_level_0_parameter_checking_ + void test__f_memory_adjust__works(void **state) { uint16_t *data = 0; diff --git a/level_0/f_memory/tests/c/test-memory-adjust.h b/level_0/f_memory/tests/c/test-memory-adjust.h index 63532d953..534080e82 100644 --- a/level_0/f_memory/tests/c/test-memory-adjust.h +++ b/level_0/f_memory/tests/c/test-memory-adjust.h @@ -10,6 +10,15 @@ #ifndef _TEST__F_memory__adjust #define _TEST__F_memory__adjust +/** + * Test that parameter checking works as expected. + * + * @see f_memory_adjust() + */ +#ifndef _di_level_0_parameter_checking_ + extern void test__f_memory_adjust__parameter_checking(void **state); +#endif // _di_level_0_parameter_checking_ + /** * Test that reallocation works. * diff --git a/level_0/f_memory/tests/c/test-memory-delete.c b/level_0/f_memory/tests/c/test-memory-delete.c index f2e538cc3..85ddd6aa5 100644 --- a/level_0/f_memory/tests/c/test-memory-delete.c +++ b/level_0/f_memory/tests/c/test-memory-delete.c @@ -95,6 +95,17 @@ void test__f_memory_delete__frees_resized_memory(void **state) { } } +#ifndef _di_level_0_parameter_checking_ + void test__f_memory_delete__parameter_checking(void **state) { + + { + const f_status_t status = f_memory_delete(4, 0, 0); + + assert_int_equal(F_status_set_fine(status), F_parameter); + } + } +#endif // _di_level_0_parameter_checking_ + #ifdef __cplusplus } // extern "C" #endif diff --git a/level_0/f_memory/tests/c/test-memory-delete.h b/level_0/f_memory/tests/c/test-memory-delete.h index c78a5b19f..fe9626ca6 100644 --- a/level_0/f_memory/tests/c/test-memory-delete.h +++ b/level_0/f_memory/tests/c/test-memory-delete.h @@ -44,4 +44,13 @@ extern void test__f_memory_delete__frees_memory(void **state); */ extern void test__f_memory_delete__frees_resized_memory(void **state); +/** + * Test that parameter checking works as expected. + * + * @see f_memory_delete() + */ +#ifndef _di_level_0_parameter_checking_ + extern void test__f_memory_delete__parameter_checking(void **state); +#endif // _di_level_0_parameter_checking_ + #endif // _TEST__F_memory__delete diff --git a/level_0/f_memory/tests/c/test-memory-destroy.c b/level_0/f_memory/tests/c/test-memory-destroy.c index cc6732fdf..75c30460e 100644 --- a/level_0/f_memory/tests/c/test-memory-destroy.c +++ b/level_0/f_memory/tests/c/test-memory-destroy.c @@ -95,6 +95,17 @@ void test__f_memory_destroy__frees_resized_memory(void **state) { } } +#ifndef _di_level_0_parameter_checking_ + void test__f_memory_destroy__parameter_checking(void **state) { + + { + const f_status_t status = f_memory_destroy(4, 0, 0); + + assert_int_equal(F_status_set_fine(status), F_parameter); + } + } +#endif // _di_level_0_parameter_checking_ + #ifdef __cplusplus } // extern "C" #endif diff --git a/level_0/f_memory/tests/c/test-memory-destroy.h b/level_0/f_memory/tests/c/test-memory-destroy.h index 4a57ce6df..561aec2f7 100644 --- a/level_0/f_memory/tests/c/test-memory-destroy.h +++ b/level_0/f_memory/tests/c/test-memory-destroy.h @@ -44,4 +44,13 @@ extern void test__f_memory_destroy__frees_memory(void **state); */ extern void test__f_memory_destroy__frees_resized_memory(void **state); +/** + * Test that parameter checking works as expected. + * + * @see f_memory_destroy() + */ +#ifndef _di_level_0_parameter_checking_ + extern void test__f_memory_destroy__parameter_checking(void **state); +#endif // _di_level_0_parameter_checking_ + #endif // _TEST__F_memory__destroy diff --git a/level_0/f_memory/tests/c/test-memory-new.c b/level_0/f_memory/tests/c/test-memory-new.c index c52f195a0..257500e16 100644 --- a/level_0/f_memory/tests/c/test-memory-new.c +++ b/level_0/f_memory/tests/c/test-memory-new.c @@ -5,6 +5,17 @@ extern "C" { #endif +#ifndef _di_level_0_parameter_checking_ + void test__f_memory_new__parameter_checking(void **state) { + + { + const f_status_t status = f_memory_new(1, 0, 0); + + assert_int_equal(F_status_set_fine(status), F_parameter); + } + } +#endif // _di_level_0_parameter_checking_ + void test__f_memory_new__works(void **state) { uint16_t *data = 0; diff --git a/level_0/f_memory/tests/c/test-memory-new.h b/level_0/f_memory/tests/c/test-memory-new.h index 08a2807e4..a3ab86709 100644 --- a/level_0/f_memory/tests/c/test-memory-new.h +++ b/level_0/f_memory/tests/c/test-memory-new.h @@ -10,6 +10,15 @@ #ifndef _TEST__F_memory__new #define _TEST__F_memory__new +/** + * Test that parameter checking works as expected. + * + * @see f_memory_new() + */ +#ifndef _di_level_0_parameter_checking_ + extern void test__f_memory_new__parameter_checking(void **state); +#endif // _di_level_0_parameter_checking_ + /** * Test that allocation works. * diff --git a/level_0/f_memory/tests/c/test-memory-new_aligned.c b/level_0/f_memory/tests/c/test-memory-new_aligned.c index 784982387..859727bf9 100644 --- a/level_0/f_memory/tests/c/test-memory-new_aligned.c +++ b/level_0/f_memory/tests/c/test-memory-new_aligned.c @@ -5,6 +5,17 @@ extern "C" { #endif +#ifndef _di_level_0_parameter_checking_ + void test__f_memory_new_aligned__parameter_checking(void **state) { + + { + const f_status_t status = f_memory_new_aligned(1, 0, 0); + + assert_int_equal(F_status_set_fine(status), F_parameter); + } + } +#endif // _di_level_0_parameter_checking_ + void test__f_memory_new_aligned__works(void **state) { uint16_t *data = 0; diff --git a/level_0/f_memory/tests/c/test-memory-new_aligned.h b/level_0/f_memory/tests/c/test-memory-new_aligned.h index a819b5a49..9b520f69a 100644 --- a/level_0/f_memory/tests/c/test-memory-new_aligned.h +++ b/level_0/f_memory/tests/c/test-memory-new_aligned.h @@ -10,6 +10,15 @@ #ifndef _TEST__F_memory__new_aligned #define _TEST__F_memory__new_aligned +/** + * Test that parameter checking works as expected. + * + * @see f_memory_new_aligned() + */ +#ifndef _di_level_0_parameter_checking_ + extern void test__f_memory_new_aligned__parameter_checking(void **state); +#endif // _di_level_0_parameter_checking_ + /** * Test that allocation works. * diff --git a/level_0/f_memory/tests/c/test-memory-resize.c b/level_0/f_memory/tests/c/test-memory-resize.c index a0d77dc37..8070bf0bc 100644 --- a/level_0/f_memory/tests/c/test-memory-resize.c +++ b/level_0/f_memory/tests/c/test-memory-resize.c @@ -5,6 +5,23 @@ extern "C" { #endif +#ifndef _di_level_0_parameter_checking_ + void test__f_memory_resize__parameter_checking(void **state) { + + { + const f_status_t status = f_memory_resize(1, 4, 0, 0); + + assert_int_equal(F_status_set_fine(status), F_parameter); + } + + { + const f_status_t status = f_memory_resize(1, 4, 1, 0); + + assert_int_equal(F_status_set_fine(status), F_parameter); + } + } +#endif // _di_level_0_parameter_checking_ + void test__f_memory_resize__works(void **state) { uint16_t *data = 0; diff --git a/level_0/f_memory/tests/c/test-memory-resize.h b/level_0/f_memory/tests/c/test-memory-resize.h index fc91c53e3..58efe233d 100644 --- a/level_0/f_memory/tests/c/test-memory-resize.h +++ b/level_0/f_memory/tests/c/test-memory-resize.h @@ -10,6 +10,15 @@ #ifndef _TEST__F_memory__resize #define _TEST__F_memory__resize +/** + * Test that parameter checking works as expected. + * + * @see f_memory_resize() + */ +#ifndef _di_level_0_parameter_checking_ + extern void test__f_memory_resize__parameter_checking(void **state); +#endif // _di_level_0_parameter_checking_ + /** * Test that reallocation works. * diff --git a/level_0/f_memory/tests/c/test-memory.c b/level_0/f_memory/tests/c/test-memory.c index 9d868c63e..e45960ed1 100644 --- a/level_0/f_memory/tests/c/test-memory.c +++ b/level_0/f_memory/tests/c/test-memory.c @@ -33,6 +33,14 @@ int main(void) { cmocka_unit_test(test__f_memory_new_aligned__works), cmocka_unit_test(test__f_memory_resize__works), + + #ifndef _di_level_0_parameter_checking_ + cmocka_unit_test(test__f_memory_adjust__parameter_checking), + cmocka_unit_test(test__f_memory_delete__parameter_checking), + cmocka_unit_test(test__f_memory_destroy__parameter_checking), + cmocka_unit_test(test__f_memory_new__parameter_checking), + cmocka_unit_test(test__f_memory_resize__parameter_checking), + #endif // _di_level_0_parameter_checking_ }; return cmocka_run_group_tests(tests, setup, setdown);