[airos] renamed dhcp converter to match section name

Author: edoput

netjsonconfig/backends/airos/airos.py

@@ -1,11 +1,11 @@
 from collections import OrderedDict
 
 from ..base.backend import BaseBackend
-from .converters import (Aaa, Bridge, Discovery, Dyndns, Ebtables, Gui, Httpd,
-                         Igmpproxy, Iptables, Netconf, Netmode, Ntpclient,
-                         Pwdog, Radio, Resolv, Route, Snmp, Sshd, Syslog,
-                         System, Telnetd, Tshaper, Unms, Update, Users, Vlan,
-                         Wireless, Wpasupplicant)
+from .converters import (Aaa, Bridge, Dhcpc, Discovery, Dyndns, Ebtables, Gui,
+                         Httpd, Igmpproxy, Iptables, Netconf, Netmode,
+                         Ntpclient, Pwdog, Radio, Resolv, Route, Snmp, Sshd,
+                         Syslog, System, Telnetd, Tshaper, Unms, Update, Users,
+                         Vlan, Wireless, Wpasupplicant)
 from .intermediate import flatten, intermediate_to_list
 from .renderers import AirOsRenderer
 from .schema import schema

netjsonconfig/backends/airos/converters.py

@@ -122,7 +122,7 @@ def to_intermediate(self):
         return (('discovery', result),)
 
 
-class Dhcp(AirOsConverter):
+class Dhcpc(AirOsConverter):
 
     @classmethod
     def should_run_forward(cls, config):

tests/airos/mock.py

@@ -3,6 +3,7 @@
 from netjsonconfig import AirOs
 from netjsonconfig.backends.airos.airos import to_ordered_list
 from netjsonconfig.backends.airos.converters import (Aaa, Bridge, Discovery,
+                                                     Dhcpc,
                                                      Dyndns, Ebtables, Gui,
                                                      Httpd, Igmpproxy,
                                                      Iptables, Netconf,
@@ -57,6 +58,15 @@ class BridgeAirOs(AirOs):
     ]
 
 
+class DhcpcAirOs(AirOs):
+    """
+    Mock backend with converter for network hardware discovery
+    """
+    converters = [
+        Dhcpc,
+    ]
+
+
 class DiscoveryAirOs(AirOs):
     """
     Mock backend with converter for network hardware discovery

tests/airos/test_dhcpc.py

@@ -0,0 +1,36 @@
+from .mock import ConverterTest, DhcpcAirOs
+
+
+class TestNetmodeConverter(ConverterTest):
+
+    backend = DhcpcAirOs
+
+    def test_bridge(self):
+        o = self.backend({
+            'netmode': 'bridge',
+        })
+        o.to_intermediate()
+
+        with self.assertRaises(KeyError):
+            o.intermediate_data['dhcpc']
+
+    def test_router(self):
+        o = self.backend({
+            'netmode': 'router',
+        })
+        o.to_intermediate()
+        expected = [
+            {
+                'status': 'enabled',
+            },
+            [
+                {
+                    'devname': 'br0',
+                    'fallback': '192.168.10.1',
+                    'fallback_netmask': '255.255.255.0',
+                    'status': 'enabled',
+                },
+            ]
+        ]
+
+        self.assertEqualConfig(o.intermediate_data['dhcpc'], expected)