From 5b4be27f4a4faabb1fddc4b9bd2fb435331fb67d Mon Sep 17 00:00:00 2001 From: Dan Williams Date: Tue, 1 Jul 2014 08:59:15 -0500 Subject: [PATCH] dhcp: consolidate and simplify DHCP options tests Previously the tests required that a NMDHCPClient object was created, but all that's really being tested is the DHCP option parsing and that doesn't need a client object now that the option parsing has been split out. Also, the options test wasn't in src/dhcp-manager/, so put it there. Lastly, the test ran for both dhclient and dhcpcd, but the code executed for both cases was the same, so there's no reason to keep running the test for both clients. --- src/dhcp-manager/nm-dhcp-manager.c | 29 -- src/dhcp-manager/nm-dhcp-manager.h | 5 - src/dhcp-manager/tests/Makefile.am | 19 +- .../tests/test-dhcp-options.c | 264 +++++------------- src/tests/Makefile.am | 14 - 5 files changed, 84 insertions(+), 247 deletions(-) rename src/{ => dhcp-manager}/tests/test-dhcp-options.c (75%) diff --git a/src/dhcp-manager/nm-dhcp-manager.c b/src/dhcp-manager/nm-dhcp-manager.c index 423d2d1a3a..3a2e52a5db 100644 --- a/src/dhcp-manager/nm-dhcp-manager.c +++ b/src/dhcp-manager/nm-dhcp-manager.c @@ -517,35 +517,6 @@ nm_dhcp_manager_get_lease_ip_configs (NMDHCPManager *self, return NULL; } -NMIP4Config * -nm_dhcp_manager_test_ip4_options_to_config (const char *dhcp_client, - const char *iface, - GHashTable *options, - const char *reason) -{ - NMDHCPClient *client; - NMIP4Config *config; - GType client_type; - GError *error = NULL; - - client_type = get_client_type (dhcp_client, &error); - if (!client_type) { - nm_log_err (LOGD_DHCP4, "error: %s", error ? error->message : "(unknown)"); - g_clear_error (&error); - return NULL; - } - - client = (NMDHCPClient *) g_object_new (client_type, - NM_DHCP_CLIENT_INTERFACE, iface, - NULL); - g_return_val_if_fail (client != NULL, NULL); - nm_dhcp_client_new_options (client, options, reason); - config = nm_dhcp_client_get_ip4_config (client, TRUE); - g_object_unref (client); - - return config; -} - /***************************************************/ NMDHCPManager * diff --git a/src/dhcp-manager/nm-dhcp-manager.h b/src/dhcp-manager/nm-dhcp-manager.h index fd1287138e..37f2ccdff3 100644 --- a/src/dhcp-manager/nm-dhcp-manager.h +++ b/src/dhcp-manager/nm-dhcp-manager.h @@ -91,11 +91,6 @@ GSList * nm_dhcp_manager_get_lease_ip_configs (NMDHCPManager *self, gboolean ipv6); /* For testing only */ -NMIP4Config *nm_dhcp_manager_test_ip4_options_to_config (const char *dhcp_client, - const char *iface, - GHashTable *options, - const char *reason); - extern const char* nm_dhcp_helper_path; #endif /* NM_DHCP_MANAGER_H */ diff --git a/src/dhcp-manager/tests/Makefile.am b/src/dhcp-manager/tests/Makefile.am index 4701b3f1b9..e9f9a61013 100644 --- a/src/dhcp-manager/tests/Makefile.am +++ b/src/dhcp-manager/tests/Makefile.am @@ -3,6 +3,7 @@ AM_CPPFLAGS = \ -I${top_srcdir}/libnm-util \ -I${top_builddir}/libnm-util \ -I$(top_srcdir)/src/dhcp-manager \ + -I$(top_srcdir)/src/logging \ -I$(top_srcdir)/src \ -I$(top_srcdir)/src/platform \ -DG_LOG_DOMAIN=\""NetworkManager"\" \ @@ -10,9 +11,11 @@ AM_CPPFLAGS = \ $(GLIB_CFLAGS) \ -DTESTDIR="\"$(abs_srcdir)\"" -noinst_PROGRAMS = test-dhcp-dhclient +noinst_PROGRAMS = \ + test-dhcp-dhclient \ + test-dhcp-options -####### policy /etc/hosts test ####### +####### dhclient leases test ####### test_dhcp_dhclient_SOURCES = \ test-dhcp-dhclient.c @@ -20,7 +23,17 @@ test_dhcp_dhclient_SOURCES = \ test_dhcp_dhclient_LDADD = \ $(top_builddir)/src/libNetworkManager.la -TESTS = test-dhcp-dhclient +####### DHCP options test ####### + +test_dhcp_options_SOURCES = \ + test-dhcp-options.c + +test_dhcp_options_LDADD = \ + $(top_builddir)/src/libNetworkManager.la + +################################# + +TESTS = test-dhcp-dhclient test-dhcp-options EXTRA_DIST = \ test-dhclient-duid.leases \ diff --git a/src/tests/test-dhcp-options.c b/src/dhcp-manager/tests/test-dhcp-options.c similarity index 75% rename from src/tests/test-dhcp-options.c rename to src/dhcp-manager/tests/test-dhcp-options.c index 5df17e587c..0fa82da17c 100644 --- a/src/tests/test-dhcp-options.c +++ b/src/dhcp-manager/tests/test-dhcp-options.c @@ -1,7 +1,5 @@ /* -*- Mode: C; tab-width: 4; indent-tabs-mode: t; c-basic-offset: 4 -*- */ -/* nm-dhcp-manager.c - Handle the DHCP daemon for NetworkManager - * - * This program is free software; you can redistribute it and/or modify +/* This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by * the Free Software Foundation; either version 2, or (at your option) * any later version. @@ -15,19 +13,18 @@ * with this program; if not, write to the Free Software Foundation, Inc., * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. * - * Copyright (C) 2008 - 2011 Red Hat, Inc. + * Copyright (C) 2008 - 2014 Red Hat, Inc. * */ #include -#include #include #include #include #include -#include "nm-dhcp-manager.h" +#include "nm-dhcp-utils.h" #include "nm-logging.h" #include "nm-test-utils.h" @@ -37,43 +34,15 @@ typedef struct { const char *value; } Option; -static void -destroy_gvalue (gpointer data) -{ - GValue *value = (GValue *) data; - - g_value_unset (value); - g_slice_free (GValue, value); -} - -static GValue * -string_to_byte_array_gvalue (const char *str) -{ - GByteArray *array; - GValue *val; - - array = g_byte_array_sized_new (strlen (str)); - g_byte_array_append (array, (const guint8 *) str, strlen (str)); - - val = g_slice_new0 (GValue); - g_value_init (val, DBUS_TYPE_G_UCHAR_ARRAY); - g_value_take_boxed (val, array); - - return val; -} - static GHashTable * fill_table (Option *test_options, GHashTable *table) { Option *opt; if (!table) - table = g_hash_table_new_full (g_str_hash, g_str_equal, NULL, destroy_gvalue); - for (opt = test_options; opt->name; opt++) { - g_hash_table_insert (table, - (gpointer) opt->name, - string_to_byte_array_gvalue (opt->value)); - } + table = g_hash_table_new_full (g_str_hash, g_str_equal, NULL, NULL); + for (opt = test_options; opt->name; opt++) + g_hash_table_insert (table, (gpointer) opt->name, (gpointer) opt->value); return table; } @@ -99,9 +68,8 @@ static Option generic_options[] = { }; static void -test_generic_options (gconstpointer test_data) +test_generic_options (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const NMPlatformIP4Address *address; @@ -119,7 +87,7 @@ test_generic_options (gconstpointer test_data) const char *expected_route2_gw = "10.1.1.1"; options = fill_table (generic_options, NULL); - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-generic", "failed to parse DHCP4 options"); @@ -217,9 +185,8 @@ static Option wins_options[] = { }; static void -test_wins_options (gconstpointer test_data) +test_wins_options (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const NMPlatformIP4Address *address; @@ -229,8 +196,7 @@ test_wins_options (gconstpointer test_data) options = fill_table (generic_options, NULL); options = fill_table (wins_options, options); - - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-wins", "failed to parse DHCP4 options"); @@ -299,9 +265,8 @@ ip4_test_gateway (const char *test, } static void -test_classless_static_routes_1 (gconstpointer test_data) +test_classless_static_routes_1 (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const char *expected_route1_dest = "192.168.10.0"; @@ -316,8 +281,7 @@ test_classless_static_routes_1 (gconstpointer test_data) options = fill_table (generic_options, NULL); options = fill_table (data, options); - - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-classless-1", "failed to parse DHCP4 options"); @@ -333,9 +297,8 @@ test_classless_static_routes_1 (gconstpointer test_data) } static void -test_classless_static_routes_2 (gconstpointer test_data) +test_classless_static_routes_2 (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const char *expected_route1_dest = "192.168.10.0"; @@ -350,8 +313,7 @@ test_classless_static_routes_2 (gconstpointer test_data) options = fill_table (generic_options, NULL); options = fill_table (data, options); - - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-classless-2", "failed to parse DHCP4 options"); @@ -367,9 +329,8 @@ test_classless_static_routes_2 (gconstpointer test_data) } static void -test_fedora_dhclient_classless_static_routes (gconstpointer test_data) +test_fedora_dhclient_classless_static_routes (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const char *expected_route1_dest = "129.210.177.128"; @@ -385,8 +346,7 @@ test_fedora_dhclient_classless_static_routes (gconstpointer test_data) options = fill_table (generic_options, NULL); options = fill_table (data, options); - - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-fedora-dhclient-classless", "failed to parse DHCP4 options"); @@ -405,9 +365,8 @@ test_fedora_dhclient_classless_static_routes (gconstpointer test_data) } static void -test_dhclient_invalid_classless_routes_1 (gconstpointer test_data) +test_dhclient_invalid_classless_routes_1 (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const char *expected_route1_dest = "192.168.10.0"; @@ -423,7 +382,7 @@ test_dhclient_invalid_classless_routes_1 (gconstpointer test_data) g_test_expect_message ("NetworkManager", G_LOG_LEVEL_WARNING, "*ignoring invalid classless static routes*"); - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-dhclient-classless-invalid-1", "failed to parse DHCP4 options"); g_test_assert_expected_messages (); @@ -439,9 +398,8 @@ test_dhclient_invalid_classless_routes_1 (gconstpointer test_data) } static void -test_dhcpcd_invalid_classless_routes_1 (gconstpointer test_data) +test_dhcpcd_invalid_classless_routes_1 (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const char *expected_route1_dest = "10.1.1.5"; @@ -459,7 +417,7 @@ test_dhcpcd_invalid_classless_routes_1 (gconstpointer test_data) g_test_expect_message ("NetworkManager", G_LOG_LEVEL_WARNING, "*ignoring invalid classless static routes*"); - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-dhcpcd-classless-invalid-1", "failed to parse DHCP4 options"); g_test_assert_expected_messages (); @@ -478,9 +436,8 @@ test_dhcpcd_invalid_classless_routes_1 (gconstpointer test_data) } static void -test_dhclient_invalid_classless_routes_2 (gconstpointer test_data) +test_dhclient_invalid_classless_routes_2 (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const char *expected_route1_dest = "10.1.1.5"; @@ -497,7 +454,7 @@ test_dhclient_invalid_classless_routes_2 (gconstpointer test_data) g_test_expect_message ("NetworkManager", G_LOG_LEVEL_WARNING, "*ignoring invalid classless static routes*"); - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-dhclient-classless-invalid-2", "failed to parse DHCP4 options"); g_test_assert_expected_messages (); @@ -516,9 +473,8 @@ test_dhclient_invalid_classless_routes_2 (gconstpointer test_data) } static void -test_dhcpcd_invalid_classless_routes_2 (gconstpointer test_data) +test_dhcpcd_invalid_classless_routes_2 (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const char *expected_route1_dest = "10.1.1.5"; @@ -535,7 +491,7 @@ test_dhcpcd_invalid_classless_routes_2 (gconstpointer test_data) g_test_expect_message ("NetworkManager", G_LOG_LEVEL_WARNING, "*ignoring invalid classless static routes*"); - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-dhcpcd-classless-invalid-2", "failed to parse DHCP4 options"); g_test_assert_expected_messages (); @@ -556,9 +512,8 @@ test_dhcpcd_invalid_classless_routes_2 (gconstpointer test_data) } static void -test_dhclient_invalid_classless_routes_3 (gconstpointer test_data) +test_dhclient_invalid_classless_routes_3 (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const char *expected_route1_dest = "192.168.10.0"; @@ -573,7 +528,7 @@ test_dhclient_invalid_classless_routes_3 (gconstpointer test_data) g_test_expect_message ("NetworkManager", G_LOG_LEVEL_WARNING, "*ignoring invalid classless static routes*"); - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-dhclient-classless-invalid-3", "failed to parse DHCP4 options"); g_test_assert_expected_messages (); @@ -588,9 +543,8 @@ test_dhclient_invalid_classless_routes_3 (gconstpointer test_data) } static void -test_dhcpcd_invalid_classless_routes_3 (gconstpointer test_data) +test_dhcpcd_invalid_classless_routes_3 (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const char *expected_route1_dest = "192.168.10.0"; @@ -605,7 +559,7 @@ test_dhcpcd_invalid_classless_routes_3 (gconstpointer test_data) g_test_expect_message ("NetworkManager", G_LOG_LEVEL_WARNING, "*DHCP provided invalid classless static route*"); - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-dhcpcd-classless-invalid-3", "failed to parse DHCP4 options"); g_test_assert_expected_messages (); @@ -620,9 +574,8 @@ test_dhcpcd_invalid_classless_routes_3 (gconstpointer test_data) } static void -test_dhclient_gw_in_classless_routes (gconstpointer test_data) +test_dhclient_gw_in_classless_routes (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const char *expected_route1_dest = "192.168.10.0"; @@ -635,8 +588,7 @@ test_dhclient_gw_in_classless_routes (gconstpointer test_data) options = fill_table (generic_options, NULL); options = fill_table (data, options); - - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-dhclient-classless-gateway", "failed to parse DHCP4 options"); @@ -653,9 +605,8 @@ test_dhclient_gw_in_classless_routes (gconstpointer test_data) } static void -test_dhcpcd_gw_in_classless_routes (gconstpointer test_data) +test_dhcpcd_gw_in_classless_routes (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const char *expected_route1_dest = "192.168.10.0"; @@ -668,8 +619,7 @@ test_dhcpcd_gw_in_classless_routes (gconstpointer test_data) options = fill_table (generic_options, NULL); options = fill_table (data, options); - - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-dhcpcd-classless-gateway", "failed to parse DHCP4 options"); @@ -691,9 +641,8 @@ static Option escaped_searches_options[] = { }; static void -test_escaped_domain_searches (gconstpointer test_data) +test_escaped_domain_searches (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const char *expected_search0 = "host1"; @@ -702,8 +651,7 @@ test_escaped_domain_searches (gconstpointer test_data) options = fill_table (generic_options, NULL); options = fill_table (escaped_searches_options, options); - - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-escaped-domain-searches", "failed to parse DHCP4 options"); @@ -726,9 +674,8 @@ static Option invalid_escaped_searches_options[] = { }; static void -test_invalid_escaped_domain_searches (gconstpointer test_data) +test_invalid_escaped_domain_searches (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; @@ -737,7 +684,7 @@ test_invalid_escaped_domain_searches (gconstpointer test_data) g_test_expect_message ("NetworkManager", G_LOG_LEVEL_WARNING, "*invalid domain search*"); - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-invalid-escaped-domain-searches", "failed to parse DHCP4 options"); g_test_assert_expected_messages (); @@ -750,17 +697,17 @@ test_invalid_escaped_domain_searches (gconstpointer test_data) } static void -test_ip4_missing_prefix (const char *client, const char *ip, guint32 expected_prefix) +test_ip4_missing_prefix (const char *ip, guint32 expected_prefix) { GHashTable *options; NMIP4Config *ip4_config; const NMPlatformIP4Address *address; options = fill_table (generic_options, NULL); - g_hash_table_insert (options, "new_ip_address", string_to_byte_array_gvalue (ip)); + g_hash_table_insert (options, "new_ip_address", (gpointer) ip); g_hash_table_remove (options, "new_subnet_mask"); - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-ip4-missing-prefix", "failed to parse DHCP4 options"); @@ -779,33 +726,26 @@ test_ip4_missing_prefix (const char *client, const char *ip, guint32 expected_pr } static void -test_ip4_missing_prefix_24 (gconstpointer test_data) +test_ip4_missing_prefix_24 (void) { - const char *client = test_data; - - test_ip4_missing_prefix (client, "192.168.1.10", 24); + test_ip4_missing_prefix ("192.168.1.10", 24); } static void -test_ip4_missing_prefix_16 (gconstpointer test_data) +test_ip4_missing_prefix_16 (void) { - const char *client = test_data; - - test_ip4_missing_prefix (client, "172.16.54.50", 16); + test_ip4_missing_prefix ("172.16.54.50", 16); } static void -test_ip4_missing_prefix_8 (gconstpointer test_data) +test_ip4_missing_prefix_8 (void) { - const char *client = test_data; - - test_ip4_missing_prefix (client, "10.1.2.3", 8); + test_ip4_missing_prefix ("10.1.2.3", 8); } static void -test_ip4_prefix_classless (gconstpointer test_data) +test_ip4_prefix_classless (void) { - const char *client = test_data; GHashTable *options; NMIP4Config *ip4_config; const NMPlatformIP4Address *address; @@ -816,10 +756,10 @@ test_ip4_prefix_classless (gconstpointer test_data) */ options = fill_table (generic_options, NULL); - g_hash_table_insert (options, "new_ip_address", string_to_byte_array_gvalue ("172.16.54.22")); - g_hash_table_insert (options, "new_subnet_mask", string_to_byte_array_gvalue ("255.255.252.0")); + g_hash_table_insert (options, "new_ip_address", "172.16.54.22"); + g_hash_table_insert (options, "new_subnet_mask", "255.255.252.0"); - ip4_config = nm_dhcp_manager_test_ip4_options_to_config (client, "eth0", options, "rebind"); + ip4_config = nm_dhcp_utils_ip4_config_from_options ("eth0", options, 0); ASSERT (ip4_config != NULL, "dhcp-ip4-prefix-classless", "failed to parse DHCP4 options"); @@ -841,96 +781,28 @@ NMTST_DEFINE (); int main (int argc, char **argv) { - char *path; - const char *clients[2][2] = { {DHCLIENT_PATH, "dhclient"}, {DHCPCD_PATH, "dhcpcd"} }; - guint32 i; - nmtst_init_assert_logging (&argc, &argv); nm_logging_setup ("WARN", "DEFAULT", NULL, NULL); - for (i = 0; i < 2; i++) { - const char *client_path = clients[i][0]; - const char *client = clients[i][1]; - - if (!client_path || !strlen (client_path)) - continue; - - path = g_strdup_printf ("/dhcp/%s/generic-options", client); - g_test_add_data_func (path, client, test_generic_options); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/wins-options", client); - g_test_add_data_func (path, client, test_wins_options); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/classless-static-routes-1", client); - g_test_add_data_func (path, client, test_classless_static_routes_1); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/classless-static-routes-2", client); - g_test_add_data_func (path, client, test_classless_static_routes_2); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/fedora-dhclient-classless-static-routes", client); - g_test_add_data_func (path, client, test_fedora_dhclient_classless_static_routes); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/dhclient-invalid-classless-routes-1", client); - g_test_add_data_func (path, client, test_dhclient_invalid_classless_routes_1); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/dhcpcd-invalid-classless-routes-1", client); - g_test_add_data_func (path, client, test_dhcpcd_invalid_classless_routes_1); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/dhclient-invalid-classless-routes-2", client); - g_test_add_data_func (path, client, test_dhclient_invalid_classless_routes_2); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/dhcpcd-invalid-classless-routes-2", client); - g_test_add_data_func (path, client, test_dhcpcd_invalid_classless_routes_2); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/dhclient-invalid-classless-routes-3", client); - g_test_add_data_func (path, client, test_dhclient_invalid_classless_routes_3); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/dhcpcd-invalid-classless-routes-3", client); - g_test_add_data_func (path, client, test_dhcpcd_invalid_classless_routes_3); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/dhclient-gw-in-classless-routes", client); - g_test_add_data_func (path, client, test_dhclient_gw_in_classless_routes); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/dhcpcd-gw-in-classless-routes", client); - g_test_add_data_func (path, client, test_dhcpcd_gw_in_classless_routes); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/escaped-domain-searches", client); - g_test_add_data_func (path, client, test_escaped_domain_searches); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/invalid-escaped-domain-searches", client); - g_test_add_data_func (path, client, test_invalid_escaped_domain_searches); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/ip4-missing-prefix-24", client); - g_test_add_data_func (path, client, test_ip4_missing_prefix_24); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/ip4-missing-prefix-16", client); - g_test_add_data_func (path, client, test_ip4_missing_prefix_16); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/ip4-missing-prefix-8", client); - g_test_add_data_func (path, client, test_ip4_missing_prefix_8); - g_free (path); - - path = g_strdup_printf ("/dhcp/%s/ip4-prefix-classless", client); - g_test_add_data_func (path, client, test_ip4_prefix_classless); - g_free (path); - } + g_test_add_func ("/dhcp/generic-options", test_generic_options); + g_test_add_func ("/dhcp/wins-options", test_wins_options); + g_test_add_func ("/dhcp/classless-static-routes-1", test_classless_static_routes_1); + g_test_add_func ("/dhcp/classless-static-routes-2", test_classless_static_routes_2); + g_test_add_func ("/dhcp/fedora-dhclient-classless-static-routes", test_fedora_dhclient_classless_static_routes); + g_test_add_func ("/dhcp/dhclient-invalid-classless-routes-1", test_dhclient_invalid_classless_routes_1); + g_test_add_func ("/dhcp/dhcpcd-invalid-classless-routes-1", test_dhcpcd_invalid_classless_routes_1); + g_test_add_func ("/dhcp/dhclient-invalid-classless-routes-2", test_dhclient_invalid_classless_routes_2); + g_test_add_func ("/dhcp/dhcpcd-invalid-classless-routes-2", test_dhcpcd_invalid_classless_routes_2); + g_test_add_func ("/dhcp/dhclient-invalid-classless-routes-3", test_dhclient_invalid_classless_routes_3); + g_test_add_func ("/dhcp/dhcpcd-invalid-classless-routes-3", test_dhcpcd_invalid_classless_routes_3); + g_test_add_func ("/dhcp/dhclient-gw-in-classless-routes", test_dhclient_gw_in_classless_routes); + g_test_add_func ("/dhcp/dhcpcd-gw-in-classless-routes", test_dhcpcd_gw_in_classless_routes); + g_test_add_func ("/dhcp/escaped-domain-searches", test_escaped_domain_searches); + g_test_add_func ("/dhcp/invalid-escaped-domain-searches", test_invalid_escaped_domain_searches); + g_test_add_func ("/dhcp/ip4-missing-prefix-24", test_ip4_missing_prefix_24); + g_test_add_func ("/dhcp/ip4-missing-prefix-16", test_ip4_missing_prefix_16); + g_test_add_func ("/dhcp/ip4-missing-prefix-8", test_ip4_missing_prefix_8); + g_test_add_func ("/dhcp/ip4-prefix-classless", test_ip4_prefix_classless); return g_test_run (); } diff --git a/src/tests/Makefile.am b/src/tests/Makefile.am index 2370d57110..75e93f20ee 100644 --- a/src/tests/Makefile.am +++ b/src/tests/Makefile.am @@ -14,7 +14,6 @@ AM_CPPFLAGS = \ $(DBUS_CFLAGS) noinst_PROGRAMS = \ - test-dhcp-options \ test-general \ test-general-with-expect \ test-ip4-config \ @@ -22,19 +21,6 @@ noinst_PROGRAMS = \ test-dcb \ test-resolvconf-capture -####### DHCP options test ####### - -test_dhcp_options_SOURCES = \ - test-dhcp-options.c - -test_dhcp_options_CPPFLAGS = \ - $(AM_CPPFLAGS) \ - -DDHCLIENT_PATH=\"$(DHCLIENT_PATH)\" \ - -DDHCPCD_PATH=\"$(DHCPCD_PATH)\" - -test_dhcp_options_LDADD = \ - $(top_builddir)/src/libNetworkManager.la - ####### ip4 config test ####### test_ip4_config_SOURCES = \