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 fa6bb47..892e2d5 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 63532d9..534080e 100644 --- a/level_0/f_memory/tests/c/test-memory-adjust.h +++ b/level_0/f_memory/tests/c/test-memory-adjust.h @@ -11,6 +11,15 @@ #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. * * @see f_memory_new() 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 f2e538c..85ddd6a 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 c78a5b1..fe9626c 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 cc6732f..75c3046 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 4a57ce6..561aec2 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 c52f195..257500e 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 08a2807..a3ab867 100644 --- a/level_0/f_memory/tests/c/test-memory-new.h +++ b/level_0/f_memory/tests/c/test-memory-new.h @@ -11,6 +11,15 @@ #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. * * @see f_memory_new() 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 7849823..859727b 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 a819b5a..9b520f6 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 @@ -11,6 +11,15 @@ #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. * * @see f_memory_new_aligned() 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 a0d77dc..8070bf0 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 fc91c53..58efe23 100644 --- a/level_0/f_memory/tests/c/test-memory-resize.h +++ b/level_0/f_memory/tests/c/test-memory-resize.h @@ -11,6 +11,15 @@ #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. * * @see f_memory_new() diff --git a/level_0/f_memory/tests/c/test-memory.c b/level_0/f_memory/tests/c/test-memory.c index 9d868c6..e45960e 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);