From: "Philippe Mathieu-Daudé" <philmd@linaro.org>
To: Markus Armbruster <armbru@redhat.com>,
qemu-devel@nongnu.org, Steve Sistare <steven.sistare@oracle.com>
Cc: "Dr. David Alan Gilbert" <dave@treblig.org>,
"Jason Wang" <jasowang@redhat.com>,
"Michael Roth" <michael.roth@amd.com>,
"Peter Xu" <peterx@redhat.com>, "Fabiano Rosas" <farosas@suse.de>,
"Marc-André Lureau" <marcandre.lureau@redhat.com>
Subject: [PATCH v6 4/5] util: strList unit tests
Date: Mon, 26 Feb 2024 15:11:06 +0100 [thread overview]
Message-ID: <20240226141108.73664-5-philmd@linaro.org> (raw)
In-Reply-To: <20240226141108.73664-1-philmd@linaro.org>
From: Steve Sistare <steven.sistare@oracle.com>
Signed-off-by: Steve Sistare <steven.sistare@oracle.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Message-ID: <1708638470-114846-5-git-send-email-steven.sistare@oracle.com>
---
tests/unit/test-strList.c | 80 +++++++++++++++++++++++++++++++++++++++
tests/unit/meson.build | 1 +
2 files changed, 81 insertions(+)
create mode 100644 tests/unit/test-strList.c
diff --git a/tests/unit/test-strList.c b/tests/unit/test-strList.c
new file mode 100644
index 0000000000..40af6b2ad1
--- /dev/null
+++ b/tests/unit/test-strList.c
@@ -0,0 +1,80 @@
+/*
+ * Copyright (c) 2022 - 2024 Oracle and/or its affiliates.
+ *
+ * This work is licensed under the terms of the GNU GPL, version 2 or later.
+ * See the COPYING file in the top-level directory.
+ */
+
+#include "qemu/osdep.h"
+#include "qemu/strList.h"
+
+static strList *make_list(int length)
+{
+ strList *head = 0, *list, **prev = &head;
+
+ while (length--) {
+ list = *prev = g_new0(strList, 1);
+ list->value = g_strdup("aaa");
+ prev = &list->next;
+ }
+ return head;
+}
+
+static void test_length(void)
+{
+ strList *list;
+ int i;
+
+ for (i = 0; i < 5; i++) {
+ list = make_list(i);
+ g_assert_cmpint(i, ==, QAPI_LIST_LENGTH(list));
+ qapi_free_strList(list);
+ }
+}
+
+struct {
+ const char *string;
+ const char *delim;
+ const char *argv[5];
+} list_data[] = {
+ { NULL, ",", { NULL } },
+ { "", ",", { NULL } },
+ { "a", ",", { "a", NULL } },
+ { "a,b", ",", { "a", "b", NULL } },
+ { "a,b,c", ",", { "a", "b", "c", NULL } },
+ { "first last", " ", { "first", "last", NULL } },
+ { "a:", ":", { "a", "", NULL } },
+ { "a::b", ":", { "a", "", "b", NULL } },
+ { ":", ":", { "", "", NULL } },
+ { ":a", ":", { "", "a", NULL } },
+ { "::a", ":", { "", "", "a", NULL } },
+};
+
+static void test_strv(void)
+{
+ int i, j;
+ const char **expect;
+ strList *list;
+ char **argv;
+
+ for (i = 0; i < ARRAY_SIZE(list_data); i++) {
+ expect = list_data[i].argv;
+ list = str_split(list_data[i].string, list_data[i].delim);
+ argv = strv_from_strList(list);
+ qapi_free_strList(list);
+ for (j = 0; expect[j] && argv[j]; j++) {
+ g_assert_cmpstr(expect[j], ==, argv[j]);
+ }
+ g_assert_null(expect[j]);
+ g_assert_null(argv[j]);
+ g_strfreev(argv);
+ }
+}
+
+int main(int argc, char **argv)
+{
+ g_test_init(&argc, &argv, NULL);
+ g_test_add_func("/test-string/length", test_length);
+ g_test_add_func("/test-string/strv", test_strv);
+ return g_test_run();
+}
diff --git a/tests/unit/meson.build b/tests/unit/meson.build
index cae925c132..9984860f0f 100644
--- a/tests/unit/meson.build
+++ b/tests/unit/meson.build
@@ -35,6 +35,7 @@ tests = {
'test-rcu-simpleq': [],
'test-rcu-tailq': [],
'test-rcu-slist': [],
+ 'test-strList': [],
'test-qdist': [],
'test-qht': [],
'test-qtree': [],
--
2.41.0
next prev parent reply other threads:[~2024-02-26 14:12 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-26 14:11 [PATCH v6 0/5] string list functions Philippe Mathieu-Daudé
2024-02-26 14:11 ` [PATCH v6 1/5] util: str_split Philippe Mathieu-Daudé
2024-02-26 14:11 ` [PATCH v6 2/5] qapi: QAPI_LIST_LENGTH Philippe Mathieu-Daudé
2024-02-26 14:11 ` [PATCH v6 3/5] util: strv_from_strList Philippe Mathieu-Daudé
2024-02-26 14:11 ` Philippe Mathieu-Daudé [this message]
2024-02-26 14:11 ` [PATCH v6 5/5] migration: simplify exec migration functions Philippe Mathieu-Daudé
2024-02-26 14:47 ` [PATCH v6 0/5] string list functions Steven Sistare
2024-02-27 15:28 ` Markus Armbruster
2024-02-27 15:37 ` Steven Sistare
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240226141108.73664-5-philmd@linaro.org \
--to=philmd@linaro.org \
--cc=armbru@redhat.com \
--cc=dave@treblig.org \
--cc=farosas@suse.de \
--cc=jasowang@redhat.com \
--cc=marcandre.lureau@redhat.com \
--cc=michael.roth@amd.com \
--cc=peterx@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=steven.sistare@oracle.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).