Remove mutable kwargs

custom_components/nordpool/aio_price.py

@@ -171,7 +171,7 @@ async def _io(self, url, **kwargs):
 
         return await resp.json()
 
-    def _parse_json(self, data, areas=[]):
+    def _parse_json(self, data, areas=None):
         """
         Parse json response from fetcher.
         Returns dictionary with
@@ -184,7 +184,10 @@ def _parse_json(self, data, areas=[]):
                 - possible other values, such as min, max, average for hourly
         """
 
+        if areas is None:
+            areas = []
         # If areas isn't a list, make it one
+
         if not isinstance(areas, list):
             areas = list(areas)
 
@@ -244,8 +247,8 @@ def _parse_json(self, data, areas=[]):
     async def _fetch_json(self, data_type, end_date=None, areas=None):
         """Fetch JSON from API"""
         # If end_date isn't set, default to tomorrow
-        if areas is None:
-            areas = []
+        if areas is None or len(areas) == 0:
+            raise Exception("Cannot query with empty areas")
         if end_date is None:
             end_date = date.today() + timedelta(days=1)
         # If end_date isn't a date or datetime object, try to parse a string

custom_components/nordpool/test_parser.py

@@ -45,7 +45,7 @@ async def _io(self, url, **kwargs):
 
         return await resp.json()
 
-    def _parse_json(self, data, areas=[]):
+    def _parse_json(self, data, areas=None):
         """
         Parse json response from fetcher.
         Returns dictionary with
@@ -59,6 +59,8 @@ def _parse_json(self, data, areas=[]):
         """
 
         # If areas isn't a list, make it one
+        if areas is None:
+            areas = []
         if not isinstance(areas, list):
             areas = list(areas)