summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorAnna Karas <aka@semihalf.com>2020-05-27 21:14:29 +0200
committerJulius Werner <jwerner@chromium.org>2020-07-06 23:47:27 +0000
commit907ddc3e07240b17e19e2ab83c4a68e0eca9a32f (patch)
tree753761c93f25430ed2c1f43d6602c68a536793f6 /tests
parent9ce5bee8c00fc285774af8d8d0ef4d623f9e1473 (diff)
downloadcoreboot-907ddc3e07240b17e19e2ab83c4a68e0eca9a32f.tar.xz
tests: Complete lib/string-test test case
Implement unit tests for remaining string library functions. Fix memory leak in test_strdup(). Signed-off-by: Anna Karas <aka@semihalf.com> Change-Id: I8ac6a6b2413d9077dc9ea81f638a2b0acd5c8862 Reviewed-on: https://review.coreboot.org/c/coreboot/+/42311 Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org> Reviewed-by: Julius Werner <jwerner@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'tests')
-rw-r--r--tests/lib/string-test.c253
1 files changed, 235 insertions, 18 deletions
diff --git a/tests/lib/string-test.c b/tests/lib/string-test.c
index 08f4177ef6..859f749618 100644
--- a/tests/lib/string-test.c
+++ b/tests/lib/string-test.c
@@ -1,50 +1,267 @@
/* SPDX-License-Identifier: GPL-2.0-only */
#include <string.h>
+#include <stdlib.h>
+#include <stdint.h>
+#include <stddef.h>
#include <tests/test.h>
+#include <stdio.h>
/*
* Important note: In every particular test, don't use any string-related
* functions other than function under test. We are linking against
* src/lib/string.c not the standard library. This is important for proper test
- * isolation. One can use __builtin_xxx for many of the most simple str*()
- * functions, when non-coreboot one is required.
+ * isolation.
*/
-struct strings_t {
+struct string_pairs_t {
+ char *dst;
+ char *src;
+} string_pairs[] = {
+ {"Hello ", "world!"},
+ {"He\0llo ", "world"},
+ {"", "world!"},
+ {"", ""},
+};
+
+const char *strings[] = {
+ "coreboot",
+ "is\0very",
+ "nice\n"
+};
+
+/* Used to test atol */
+struct str_with_l_val_t {
char *str;
- size_t size;
-} strings[] = {
- {"coreboot", 8},
- {"is\0very", 2}, /* strlen should be 2 because of the embedded \0 */
- {"nice\n", 5}
+ long value;
+} str_with_l_val[] = {
+ {"42", 42},
+ {"four42", 0},
+ {"42five5", 42},
+ {"4\02", 4},
+ {"+42", 42},
+ {"-42", -42},
+ {"\t\n\r\f\v-42", -42},
};
-static void test_strlen_strings(void **state)
+/* Used to test skip_atoi */
+struct str_with_u_val_t {
+ char *str;
+ uint32_t value;
+} str_with_u_val[] = {
+ {"42aa", 42},
+ {"a", 0},
+ {"0", 0},
+ {"4a2", 4},
+};
+
+static void test_strdup(void **state)
+{
+ char str[] = "Hello coreboot\n";
+ char *duplicate = strdup(str);
+
+ /* There is a more suitable Cmocka's function 'assert_string_equal()', but it
+ is using strcmp() internally. */
+ assert_int_equal(0, memcmp(str, duplicate, __builtin_strlen(str)));
+
+ free(duplicate);
+}
+
+static void test_strconcat(void **state)
+{
+ int i;
+ size_t str_len, str2_len, res_len;
+ char *result;
+
+ for (i = 0; i < ARRAY_SIZE(string_pairs); i++) {
+ str_len = __builtin_strlen(string_pairs[i].dst);
+ str2_len = __builtin_strlen(string_pairs[i].src);
+
+ result = strconcat(string_pairs[i].dst, string_pairs[i].src);
+ res_len = __builtin_strlen(result);
+
+ assert_int_equal(res_len, str_len + str2_len);
+ assert_int_equal(0, memcmp(string_pairs[i].dst, result, str_len));
+ assert_int_equal(0, memcmp(string_pairs[i].src, result + str_len, str2_len));
+
+ free(result);
+ }
+}
+
+static void test_strnlen(void **state)
+{
+ int i, n = 5;
+ size_t str_len, limited_len;
+
+ for (i = 0; i < ARRAY_SIZE(strings); i++) {
+ str_len = __builtin_strlen(strings[i]);
+ limited_len = MIN(n, str_len);
+ assert_int_equal(limited_len, strnlen(strings[i], n));
+ }
+}
+
+static void test_strlen(void **state)
{
int i;
for (i = 0; i < ARRAY_SIZE(strings); i++)
- assert_int_equal(strings[i].size, strlen(strings[i].str));
+ assert_int_equal(__builtin_strlen(strings[i]), strlen(strings[i]));
}
-static void test_strdup(void **state)
+static void test_strchr(void **state)
{
- char str[] = "Hello coreboot\n";
- char *duplicate;
+ char str[] = "Abracadabra!\n";
- duplicate = strdup(str);
+ assert_ptr_equal(str, strchr(str, 'A'));
+ assert_ptr_equal(str + 3, strchr(str, 'a'));
+ assert_ptr_equal(str + 12, strchr(str, '\n'));
- /* There is a more suitable Cmocka's function 'assert_string_equal()', but it
- is using strcmp() internally. */
- assert_int_equal(0, memcmp(str, duplicate, __builtin_strlen(str)));
+ assert_null(strchr(str, 'z'));
+}
+
+
+static void test_strrchr(void **state)
+{
+ char str[] = "Abracadabra!\n";
+
+ assert_ptr_equal(str, strrchr(str, 'A'));
+ assert_ptr_equal(str + 9, strrchr(str, 'r'));
+ assert_ptr_equal(str + 12, strrchr(str, '\n'));
+
+ assert_null(strrchr(str, 'z'));
+}
+
+static void test_strncpy(void **state)
+{
+ int i;
+ int n1 = 2, n2 = 8;
+ char src[] = "Hello";
+ char dst[sizeof(src) + 5];
+ size_t src_len = __builtin_strlen(src);
+ size_t dst_len = sizeof(dst);
+
+ /* n1 case */
+
+ /* Needed for ensuring that characters behind the limit
+ are not overwritten */
+ memset(dst, 'x', dst_len);
+
+ strncpy(dst, src, n1);
+
+ assert_int_equal(0, memcmp(dst, src, n1));
+
+ for (i = n1; i < dst_len; i++)
+ assert_true(dst[i] == 'x');
+
+ /* n2 case: */
+
+ memset(dst, 'x', dst_len);
+
+ strncpy(dst, src, n2);
+
+ assert_int_equal(0, memcmp(dst, src, src_len));
+
+ for (i = src_len; i < n2; i++)
+ assert_true(dst[i] == '\0');
+
+ for (i = n2; i < dst_len; i++)
+ assert_true(dst[i] == 'x');
+}
+
+static void test_strcpy(void **state)
+{
+ char src[] = "Hello coreboot\n";
+ char dst[sizeof(src)];
+
+ /* Make sure that strcpy() sets '\0' by initializing a whole
+ dst array to fixed, non-'\0' value */
+ memset(dst, 'x', sizeof(dst));
+
+ strcpy(dst, src);
+
+ assert_int_equal(0, memcmp(dst, src, __builtin_strlen(src) + 1));
+}
+
+static void test_strcmp(void **state)
+{
+ char str[] = "Banana";
+ char str2[] = "Banana";
+ char str3[] = "Bananas";
+
+ assert_true(strcmp(str, str3) < 0);
+ assert_int_equal(0, strcmp(str, str2));
+ assert_true(strcmp(str3, str2) > 0);
+}
+
+static void test_strncmp(void **state)
+{
+ char str[] = "Banana";
+ char str2[] = "Bananas";
+
+ size_t str2_len = __builtin_strlen(str2);
+
+ assert_true(strncmp(str, str2, str2_len) < 0);
+ assert_int_equal(0, strncmp(str, str2, str2_len - 1));
+}
+
+static void test_skip_atoi(void **state)
+{
+ int i;
+ char *ptr;
+
+ for (i = 0; i < ARRAY_SIZE(str_with_u_val); i++) {
+ ptr = str_with_u_val[i].str;
+ assert_true(str_with_u_val[i].value == skip_atoi(&ptr));
+ }
+}
+
+static void test_strspn(void **state)
+{
+ char str[] = "4213401234";
+ char str2[] = "01234";
+ char str3[] = "1234";
+
+ assert_int_equal(5, strspn(str, str3));
+ assert_int_equal(0, strspn(str2, str3));
+}
+
+static void test_strcspn(void **state)
+{
+ char str[] = "12340000";
+ char str2[] = "00001234";
+ char str3[] = "1234";
+
+ assert_int_equal(0, strcspn(str, str3));
+ assert_int_equal(4, strcspn(str2, str3));
+}
+
+/* Please bear in mind that `atol()` uses `strspn()` internally, so the result
+ of `test_atol` is dependent on the result of `test_strspn`. */
+static void test_atol(void **state)
+{
+ int i;
+
+ for (i = 0; i < ARRAY_SIZE(str_with_l_val); i++)
+ assert_int_equal(str_with_l_val[i].value, atol(str_with_l_val[i].str));
}
int main(void)
{
const struct CMUnitTest tests[] = {
- cmocka_unit_test(test_strlen_strings),
cmocka_unit_test(test_strdup),
+ cmocka_unit_test(test_strconcat),
+ cmocka_unit_test(test_strnlen),
+ cmocka_unit_test(test_strlen),
+ cmocka_unit_test(test_strchr),
+ cmocka_unit_test(test_strrchr),
+ cmocka_unit_test(test_strncpy),
+ cmocka_unit_test(test_strcpy),
+ cmocka_unit_test(test_strcmp),
+ cmocka_unit_test(test_strncmp),
+ cmocka_unit_test(test_skip_atoi),
+ cmocka_unit_test(test_strspn),
+ cmocka_unit_test(test_strcspn),
+ cmocka_unit_test(test_atol),
};
return cmocka_run_group_tests(tests, NULL, NULL);