make settable keys immutable

mietzen · Aug 1, 2023 · 6b3cacb · 6b3cacb
1 parent 2b8cac2
commit 6b3cacb
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/fritz_advanced_thermostat/fritz_advanced_thermostat.py b/fritz_advanced_thermostat/fritz_advanced_thermostat.py
@@ -67,28 +67,29 @@ def __init__(self,
         self._thermostat_data = {}
         self._valid_device_types = ['Heizkörperregler']
         self._settable_keys = {
-            "common": [
+            "common": (
                 "Offset",
                 "WindowOpenTimer",
                 "WindowOpenTrigger",
                 "locklocal",
                 "lockuiapp",
-            ],
-            "ungrouped": [
+            ),
+            "ungrouped": (
                 "Absenktemp", "Heiztemp", "Holiday1Enabled", "Holiday1EndDay", "Holiday1EndHour", "Holiday1EndMonth",
                 "Holiday1StartDay", "Holiday1StartHour", "Holiday1StartMonth", "Holiday2Enabled", "Holiday2EndDay",
                 "Holiday2EndHour", "Holiday2EndMonth", "Holiday2StartDay", "Holiday2StartHour", "Holiday2StartMonth",
                 "Holiday3Enabled", "Holiday3EndDay", "Holiday3EndHour", "Holiday3EndMonth", "Holiday3StartDay",
                 "Holiday3StartHour", "Holiday3StartMonth", "Holiday4Enabled", "Holiday4EndDay", "Holiday4EndHour",
                 "Holiday4EndMonth", "Holiday4StartDay", "Holiday4StartHour", "Holiday4StartMonth", "Holidaytemp",
                 "SummerEnabled", "SummerEndDay", "SummerEndMonth", "SummerStartDay", "SummerStartMonth"
-            ]
+            )
         }
 
         self._supported_thermostats = ['FRITZ!DECT 301']
         self._thermostats = []
         # Setup selenium options
         self._selenium_options = Options()
+        # TODO Remove
         self._selenium_options.binary_location = "/Applications/Google Chrome.app/Contents/MacOS/Google Chrome"
 
         self._selenium_options.add_argument('--headless')
@@ -183,9 +184,9 @@ def _scrape_thermostat_data(self, device_name):
 
     def _set_thermostat_values(self, device_name, **kwargs):
         self._load_raw_thermostat_data(device_name)
-        settable_keys = self._settable_keys["common"]
+        settable_keys = list(self._settable_keys["common"])
         if not self._thermostat_data[device_name]['Grouped']:
-            settable_keys += self._settable_keys["ungrouped"]
+            settable_keys += list(self._settable_keys["ungrouped"])
         for key, value in kwargs.items():
             if key in settable_keys:
                 if key in self._thermostat_data[device_name].keys():