Debugged DateTimeLabelUnitOptions and implemented unit tests.

Author: hcpchris

highcharts_core/utility_classes/date_time_label_formats.py

@@ -113,7 +113,7 @@ def _to_untrimmed_dict(self, in_cls=None) -> dict:
         untrimmed = {
             "list": self.list,
             "main": self.main,
-            'range': slef.range,
+            'range': self.range,
         }
 
         return untrimmed
@@ -182,7 +182,7 @@ def hour(self, value):
         elif isinstance(value, str):
             self._hour = value
         else:
-            self._hour = validate_types(value, DateTimeLabelUnitOptions)
+            self._hour = validate_types(value, DateTimeLabelRangedUnitOptions)
 
     @property
     def millisecond(self) -> Optional[str | DateTimeLabelRangedUnitOptions]:
@@ -202,7 +202,7 @@ def millisecond(self, value):
         elif isinstance(value, str):
             self._millisecond = value
         else:
-            self._millisecond = validate_types(value, DateTimeLabelUnitOptions)
+            self._millisecond = validate_types(value, DateTimeLabelRangedUnitOptions)
 
     @property
     def minute(self) -> Optional[str | DateTimeLabelRangedUnitOptions]:
@@ -222,7 +222,7 @@ def minute(self, value):
         elif isinstance(value, str):
             self._minute = value
         else:
-            self._minute = validate_types(value, DateTimeLabelUnitOptions)
+            self._minute = validate_types(value, DateTimeLabelRangedUnitOptions)
 
     @property
     def month(self) -> Optional[str | DateTimeLabelUnitOptions]:
@@ -262,7 +262,7 @@ def second(self, value):
         elif isinstance(value, str):
             self._second = value
         else:
-            self._second = validate_types(value, DateTimeLabelUnitOptions)
+            self._second = validate_types(value, DateTimeLabelRangedUnitOptions)
 
     @property
     def week(self) -> Optional[str | DateTimeLabelUnitOptions]:

tests/utility_classes/test_date_time_label_formats.py

@@ -12,16 +12,39 @@
 
 STANDARD_PARAMS = [
     ({}, None),
-    ({
-      'day': 'test',
-      'hour': 'test',
-      'millisecond': 'test',
-      'minute': 'test',
-      'month': 'test',
-      'second': 'test',
-      'week': 'test',
-      'year': 'test'
-    }, None),
+    (
+        {
+            "day": "test",
+            "hour": "test",
+            "millisecond": "test",
+            "minute": "test",
+            "month": "test",
+            "second": "test",
+            "week": "test",
+            "year": "test",
+        },
+        None,
+    ),
+    (
+        {
+            "day": {
+                'main': 'test',
+                'list': ['Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday', 'Saturday', 'Sunday'],
+            },
+            "hour": {
+                'main': 'test',
+                'list': ['Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday', 'Saturday', 'Sunday'],
+                'range': True,
+            },
+            "millisecond": "test",
+            "minute": "test",
+            "month": "test",
+            "second": "test",
+            "week": "test",
+            "year": "test",
+        },
+        None,
+    ),
 ]