From 10ddd913f0d13584e13b5e6bbcb381f1618c90c0 Mon Sep 17 00:00:00 2001 From: Thomas Hindoe Paaboel Andersen Date: Thu, 7 Feb 2013 00:33:58 +0100 Subject: [PATCH] tests: move strv_parse_nulstr to test-strv, and actually test it The test was originally added with this commit message: "ask-password: supported plymouth cached passwords" --- src/test/test-env-replace.c | 10 +--------- src/test/test-strv.c | 18 ++++++++++++++++++ 2 files changed, 19 insertions(+), 9 deletions(-) diff --git a/src/test/test-env-replace.c b/src/test/test-env-replace.c index ad5c99160..0bd42e329 100644 --- a/src/test/test-env-replace.c +++ b/src/test/test-env-replace.c @@ -47,15 +47,7 @@ int main(int argc, char *argv[]) { NULL }; - char **i, **r, *t, **a, **b; - const char nulstr[] = "fuck\0fuck2\0fuck3\0\0fuck5\0\0xxx"; - - a = strv_parse_nulstr(nulstr, sizeof(nulstr)-1); - - STRV_FOREACH(i, a) - printf("nulstr--%s\n", *i); - - strv_free(a); + char **r, *t, **a, **b; r = replace_env_argv((char**) line, (char**) env); strv_print(r); diff --git a/src/test/test-strv.c b/src/test/test-strv.c index 1df8157b5..07aac3a67 100644 --- a/src/test/test-strv.c +++ b/src/test/test-strv.c @@ -100,6 +100,23 @@ static void test_strv_join(void) { assert(streq(t, "")); } +static void test_strv_parse_nulstr(void) { + _cleanup_strv_free_ char **l = NULL; + const char nulstr[] = "fuck\0fuck2\0fuck3\0\0fuck5\0\0xxx"; + + l = strv_parse_nulstr(nulstr, sizeof(nulstr)-1); + puts("Parse nulstr:"); + strv_print(l); + + assert(streq(l[0], "fuck")); + assert(streq(l[1], "fuck2")); + assert(streq(l[2], "fuck3")); + assert(streq(l[3], "")); + assert(streq(l[4], "fuck5")); + assert(streq(l[5], "")); + assert(streq(l[6], "xxx")); +} + static void test_strv_overlap(void) { const char * const input_table[] = { "one", @@ -146,6 +163,7 @@ int main(int argc, char *argv[]) { test_strv_find(); test_strv_find_prefix(); test_strv_join(); + test_strv_parse_nulstr(); test_strv_overlap(); test_strv_sort(); -- 2.30.2