[openwrt] Fixed empty name bug introduced in 2a6cfb2

Fixed a bug that was introduced in 0.6.0 alpha (2a6cfb2)

Author: nemesifier

netjsonconfig/backends/openwrt/converters.py

@@ -78,8 +78,7 @@ def to_intermediate(self):
         result = []
         for interface in get_copy(self.netjson, 'interfaces'):
             i = 1
-            # determine uci logical interface name
-            uci_name = logical_name(interface.get('network', interface['name']))
+            uci_name = self.__get_uci_name(interface)
             address_list = self.__get_addresses(interface)
             interface = self.__get_interface(interface, uci_name)
             # create one or more "config interface" UCI blocks
@@ -101,6 +100,13 @@ def to_intermediate(self):
                 i += 1
         return (('network', result),)
 
+    def __get_uci_name(self, interface):
+        """
+        determines uci logical interface name
+        """
+        name = interface.get('network') or interface['name']
+        return logical_name(name)
+
     def __get_addresses(self, interface):
         """
         converts NetJSON address to

tests/openwrt/test_network.py

@@ -1225,3 +1225,21 @@ def test_interface_disabled_bug(self):
         })
         self.assertNotIn("disabled '0'", o.render())
         self.assertIn("enabled '1'", o.render())
+
+    def test_empty_network(self):
+        o = OpenWrt({
+            "interfaces": [
+                {
+                    "name": "eth0",
+                    "type": "ethernet",
+                    "network": ""
+                }
+            ]
+        })
+        expected = self._tabs("""package network
+
+config interface 'eth0'
+    option ifname 'eth0'
+    option proto 'none'
+""")
+        self.assertEqual(o.render(), expected)