test: verify RoutesToDNS= is independent of UseGateway=

diff --git a/test/test-network/conf/dhcp-client-ipv4-use-routes-use-gateway.network b/test/test-network/conf/dhcp-client-ipv4-use-routes-use-gateway.network
index b0d0d8605e3bb549c1e30d814f381786911202a7..c980bf9fca5ccce1c4897edb1af8dfbbb47bee16 100644 (file)
--- a/test/test-network/conf/dhcp-client-ipv4-use-routes-use-gateway.network
+++ b/test/test-network/conf/dhcp-client-ipv4-use-routes-use-gateway.network
@@ -4,6 +4,3 @@ Name=veth99
 [Network]
 DHCP=ipv4
 IPv6AcceptRA=false
-
-[DHCPv4]
-RoutesToDNS=yes

diff --git a/test/test-network/conf/dhcp-client-ipv4-use-routes-use-gateway.network.d/use-dns-routes-False.conf b/test/test-network/conf/dhcp-client-ipv4-use-routes-use-gateway.network.d/use-dns-routes-False.conf
new file mode 100644 (file)
index 0000000..9c561fb
--- /dev/null
+++ b/test/test-network/conf/dhcp-client-ipv4-use-routes-use-gateway.network.d/use-dns-routes-False.conf
@@ -0,0 +1,2 @@
+[DHCPv4]
+RoutesToDNS=no

diff --git a/test/test-network/conf/dhcp-client-ipv4-use-routes-use-gateway.network.d/use-dns-routes-True.conf b/test/test-network/conf/dhcp-client-ipv4-use-routes-use-gateway.network.d/use-dns-routes-True.conf
new file mode 100644 (file)
index 0000000..2504c20
--- /dev/null
+++ b/test/test-network/conf/dhcp-client-ipv4-use-routes-use-gateway.network.d/use-dns-routes-True.conf
@@ -0,0 +1,2 @@
+[DHCPv4]
+RoutesToDNS=yes

diff --git a/test/test-network/systemd-networkd-tests.py b/test/test-network/systemd-networkd-tests.py
index efc3b2ad09f0e205e2721874ac6000b0f7886407..3fc93a725f7acf8c464fa7f1feec6ccd05eb76e0 100755 (executable)
--- a/test/test-network/systemd-networkd-tests.py
+++ b/test/test-network/systemd-networkd-tests.py
@@ -3026,19 +3026,21 @@ class NetworkdDHCPClientTests(unittest.TestCase, Utilities):
         self.assertRegex(output, r'192.168.5.8 proto dhcp scope link src 192.168.5.181 metric 1024')
 
     def test_dhcp_client_ipv4_use_routes_gateway(self):
-        for (routes, gateway) in itertools.product([True, False, None], repeat=2):
+        for (routes, gateway, dnsroutes) in itertools.product([True, False, None], repeat=3):
             self.setUp()
-            with self.subTest(routes=routes, gateway=gateway):
-                self._test_dhcp_client_ipv4_use_routes_gateway(routes, gateway)
+            with self.subTest(routes=routes, gateway=gateway, dnsroutes=dnsroutes):
+                self._test_dhcp_client_ipv4_use_routes_gateway(routes, gateway, dnsroutes)
             self.tearDown()
 
-    def _test_dhcp_client_ipv4_use_routes_gateway(self, routes, gateway):
+    def _test_dhcp_client_ipv4_use_routes_gateway(self, routes, gateway, dnsroutes):
         testunit = 'dhcp-client-ipv4-use-routes-use-gateway.network'
         testunits = ['25-veth.netdev', 'dhcp-server-veth-peer.network', testunit]
         if routes != None:
             testunits.append(f'{testunit}.d/use-routes-{routes}.conf');
         if gateway != None:
             testunits.append(f'{testunit}.d/use-gateway-{gateway}.conf');
+        if dnsroutes != None:
+            testunits.append(f'{testunit}.d/use-dns-routes-{dnsroutes}.conf');
         copy_unit_to_networkd_unit_path(*testunits, dropins=False)
 
         start_networkd()
@@ -3066,8 +3068,8 @@ class NetworkdDHCPClientTests(unittest.TestCase, Utilities):
         else:
             self.assertNotRegex(output, r'default via 192.168.5.1')
 
-        # check for routes to DNS server, only if using gateway
-        if usegateway:
+        # Check RoutesToDNS=, which defaults to false
+        if dnsroutes:
             self.assertRegex(output, r'192.168.5.6 proto dhcp scope link src 192.168.5.181 metric 1024')
             self.assertRegex(output, r'192.168.5.7 proto dhcp scope link src 192.168.5.181 metric 1024')
         else: