[fix] Update renderer to use regex for True/False coercion

Author: Viscous106

netjsonconfig/backends/openwrt/renderer.py

@@ -16,8 +16,10 @@ def cleanup(self, output):
             .replace("\noption", "\n\toption")
             .replace("\nlist", "\n\tlist")
         )
-        # convert True to 1 and False to 0
-        output = output.replace("True", "1").replace("False", "0")
+        import re
+        # convert 'True' to '1' and 'False' to '0', but only on whole words bounded by quotes e.g. 'True' -> '1'
+        output = re.sub(r"'True'", "'1'", output)
+        output = re.sub(r"'False'", "'0'", output)
         # max 2 consecutive \n delimiters
         output = output.replace("\n\n\n", "\n\n")
         # if output is present

tests/openwrt/test_wireless.py

@@ -63,6 +63,21 @@ def test_render_wifi_interface(self):
         expected = self._tabs(self._wifi_uci)
         self.assertEqual(o.render(), expected)
 
+    def test_render_wifi_issue_250_true_ssid(self):
+        netjson = deepcopy(self._wifi_netjson)
+        netjson["interfaces"][0]["wireless"]["ssid"] = "TrueGait Living Guest"
+        o = OpenWrt(netjson)
+        expected_uci = self._wifi_uci.replace("'MyWifiAP'", "'TrueGait Living Guest'")
+        self.assertEqual(o.render(), self._tabs(expected_uci))
+
+    def test_render_wifi_issue_250_false_ssid(self):
+        netjson = deepcopy(self._wifi_netjson)
+        netjson["interfaces"][0]["wireless"]["ssid"] = "FalseGait Living Guest"
+        o = OpenWrt(netjson)
+        expected_uci = self._wifi_uci.replace("'MyWifiAP'", "'FalseGait Living Guest'")
+        self.assertEqual(o.render(), self._tabs(expected_uci))
+
+
     def test_parse_wifi_interface(self):
         o = OpenWrt(native=self._wifi_uci)
         self.assertDictEqual(o.config, self._wifi_netjson)