[change] Handle autonaming of network devices #199

Maintain consistency with the naming used on OpenWISP.

Closes #199

openwisp-config/files/sbin/openwisp-uci-autoname.lua

@@ -63,6 +63,8 @@ for file in lfs.dir(standard_path) do
           new_name = 'defaults'
         elseif file == 'network' and section['.type'] == 'globals' then
           new_name = 'globals'
+        elseif file == 'network' and section['.type'] == 'device' then
+          new_name = 'device_' .. section['name']:gsub('br%-', '')
         elseif file == 'network' and
           (section['.type'] == 'route' or section['.type'] == 'route6') then
           new_name = nextAvailableName('route')

openwisp-config/tests/config/network

@@ -19,6 +19,15 @@ config interface 'wan'
 	option proto 'none'
 	option test_restore '1'
 
+config device
+	option name 'br-lan2'
+	list ports 'lan2'
+	option type 'bridge'
+
+config interface 'lan2'
+	option device 'br-lan2'
+	option proto 'dhcp'
+
 config interface 'wlan0'
 	option ifname 'wlan0'
 	list ipaddr '172.27.254.251/16'

openwisp-config/tests/test_uci_autoname.lua

@@ -35,6 +35,8 @@ function TestUciAutoname.test_default_behaviour()
   luaunit.assertNotNil(string.find(networkContents,
     "config switch_port 'switch0_port1"))
   luaunit.assertNotNil(string.find(networkContents, "config globals 'globals"))
+  luaunit.assertNotNil(string.find(networkContents,
+    "config device 'device_lan2"))
   luaunit.assertNotNil(string.find(networkContents, "config route 'route1"))
   luaunit.assertNotNil(string.find(networkContents, "config route 'route2"))
   -- ensure rest of config options are present