From db1ca39c60a6a9eaf77e4f1eda4565c3ca5fecf4 Mon Sep 17 00:00:00 2001 From: Niels De Graef Date: Thu, 12 Nov 2020 19:05:25 +0100 Subject: gio, glib: Use g_assert_cmpstrv where it makes sense --- gio/tests/desktop-app-info.c | 7 ++----- glib/tests/gvariant.c | 24 ++++++------------------ glib/tests/strfuncs.c | 4 +--- 3 files changed, 9 insertions(+), 26 deletions(-) diff --git a/gio/tests/desktop-app-info.c b/gio/tests/desktop-app-info.c index f4e509a59..27af9934e 100644 --- a/gio/tests/desktop-app-info.c +++ b/gio/tests/desktop-app-info.c @@ -350,6 +350,7 @@ wait_for_file (const gchar *want_this, static void test_actions (void) { + const char *expected[] = { "frob", "tweak", "twiddle", "broken", NULL }; const gchar * const *actions; GDesktopAppInfo *appinfo; gchar *name; @@ -358,11 +359,7 @@ test_actions (void) g_assert_nonnull (appinfo); actions = g_desktop_app_info_list_actions (appinfo); - g_assert_cmpstr (actions[0], ==, "frob"); - g_assert_cmpstr (actions[1], ==, "tweak"); - g_assert_cmpstr (actions[2], ==, "twiddle"); - g_assert_cmpstr (actions[3], ==, "broken"); - g_assert_cmpstr (actions[4], ==, NULL); + g_assert_cmpstrv (actions, expected); name = g_desktop_app_info_get_action_name (appinfo, "frob"); g_assert_cmpstr (name, ==, "Frobnicate"); diff --git a/glib/tests/gvariant.c b/glib/tests/gvariant.c index 1e30d3673..35434cae9 100644 --- a/glib/tests/gvariant.c +++ b/glib/tests/gvariant.c @@ -2992,18 +2992,6 @@ test_varargs_empty_array (void) g_assert_not_reached (); } -static void -assert_cmpstrv (const gchar **strv1, const gchar **strv2) -{ - gsize i; - - for (i = 0; strv1[i] != NULL && strv2[i] != NULL; i++) - g_assert_cmpstr (strv1[i], ==, strv2[i]); - - g_assert_null (strv1[i]); - g_assert_null (strv2[i]); -} - static void test_varargs (void) { @@ -3153,8 +3141,8 @@ test_varargs (void) g_variant_iter_next (&tuple, "^a&s", &strv); g_variant_iter_next (&tuple, "^as", &my_strv); - assert_cmpstrv (strv, strvector); - assert_cmpstrv ((const char **)my_strv, strvector); + g_assert_cmpstrv (strv, strvector); + g_assert_cmpstrv (my_strv, strvector); g_variant_unref (value); g_strfreev (my_strv); @@ -3223,8 +3211,8 @@ test_varargs (void) g_variant_iter_next (&tuple, "^a&ay", &strv); g_variant_iter_next (&tuple, "^aay", &my_strv); - assert_cmpstrv (strv, strvector); - assert_cmpstrv ((const char **)my_strv, strvector); + g_assert_cmpstrv (strv, strvector); + g_assert_cmpstrv (my_strv, strvector); g_variant_unref (value); g_strfreev (my_strv); @@ -3272,8 +3260,8 @@ test_varargs (void) g_variant_iter_next (&tuple, "^a&o", &strv); g_variant_iter_next (&tuple, "^ao", &my_strv); - assert_cmpstrv (strv, strvector); - assert_cmpstrv ((const char **)my_strv, strvector); + g_assert_cmpstrv (strv, strvector); + g_assert_cmpstrv (my_strv, strvector); g_variant_unref (value); g_strfreev (my_strv); diff --git a/glib/tests/strfuncs.c b/glib/tests/strfuncs.c index 3947fdf9a..37cbc5a8a 100644 --- a/glib/tests/strfuncs.c +++ b/glib/tests/strfuncs.c @@ -525,9 +525,7 @@ test_strdupv (void) copy = g_strdupv (vec); g_assert_nonnull (copy); - g_assert_cmpstr (copy[0], ==, "Foo"); - g_assert_cmpstr (copy[1], ==, "Bar"); - g_assert_null (copy[2]); + g_assert_cmpstrv (copy, vec); g_strfreev (copy); } -- cgit v1.2.1