Merge pull request #5 from highcharts-for-python/enhancement/add-unit-tests

Added unit tests with 73% coverage.

Author: hcpchris

.gitignore

@@ -131,4 +131,4 @@ dmypy.json
 
 # temp files
 ~$*.*
-tests/input_files/headless_export/output/
+#tests/input_files/headless_export/output/

highcharts_gantt/chart.py

@@ -15,3 +15,16 @@
 ]
 
 GANTT_INCLUDE_STR = """<script src="https://code.highcharts.com/gantt/modules/gantt.js"></script>"""
+
+
+MONDAY_TEMPLATES = {
+    'task-management': {
+        'id': 'id',
+        'name': 'name',
+        'start': 'start',
+        'end': 'end',
+        'parent': 'parent_id',
+        'milestone': 'is_milestone',
+        'completed': 'completed'
+    },
+}

highcharts_gantt/constants.py

@@ -15,10 +15,13 @@ class MondayAuthenticationError(HighchartsValueError):
     pass
 
 
-class JIRAAuthenticationError(HighchartsValueError):
-    """:exc:`ValueError <python:ValueError>` encountered when trying to call 
-    :meth:`GanttSeries.from_jira() <highcharts_gantt.options.series.gantt.GanttSeries.from_jira>` 
-    with improperly configured authentication."""
+class MondayBoardNotFoundError(HighchartsValueError):
+    """:exc:`ValueError <python:ValueError>` encountered when an indicated Monday.com board was not found."""
+    pass
+
+
+class MondayItemNotFoundError(HighchartsValueError):
+    """:exc:`ValueError <python:ValueError>` encountered when an indicated Monday.com item (task) was not found."""
     pass
 
 
@@ -29,7 +32,14 @@ class MondayTemplateError(HighchartsValueError):
     pass
 
 
-class DuplicateJIRAIssueError(HighchartsValueError):
+class JIRAAuthenticationError(HighchartsValueError):
+    """:exc:`ValueError <python:ValueError>` encountered when trying to call 
+    :meth:`GanttSeries.from_jira() <highcharts_gantt.options.series.gantt.GanttSeries.from_jira>` 
+    with improperly configured authentication."""
+    pass
+
+
+class JIRADuplicateIssueError(HighchartsValueError):
     """:exc:`ValueError <python:ValueError>` encountered when encountering a JIRA issue 
     that is a duplicate of another issue."""
     pass

highcharts_gantt/errors.py

@@ -0,0 +1 @@
+from highcharts_stock.global_options.language.stock_tools import *

highcharts_gantt/global_options/language/stock_tools.py

@@ -7,7 +7,7 @@
 
 from highcharts_gantt import errors
 from highcharts_gantt.decorators import validate_types
-from highcharts_core.global_options.shared_options import SharedOptions
+from highcharts_stock.global_options.shared_options import SharedOptions, SharedStockOptions
 
 
 class SharedGanttOptions(HighchartsGanttOptions):

highcharts_gantt/global_options/shared_options.py

@@ -6,8 +6,9 @@
 from highcharts_core.headless_export import ExportServer as ExportServerBase
 
 from highcharts_gantt.decorators import validate_types
-from highcharts_gantt.options import HighchartsOptions, HighchartsGanttOptions
+from highcharts_gantt.options import HighchartsOptions, HighchartsGanttOptions, HighchartsStockOptions
 from highcharts_gantt.global_options.shared_options import (SharedOptions,
+                                                            SharedStockOptions,
                                                             SharedGanttOptions)
 
 load_dotenv()
@@ -33,7 +34,7 @@ def __init__(self, **kwargs):
         super().__init__(**kwargs)
 
     @property
-    def options(self) -> Optional[HighchartsOptions | HighchartsGanttOptions]:
+    def options(self) -> Optional[HighchartsOptions | HighchartsGanttOptions | HighchartsStockOptions]:
         """The :class:`HighchartsOptions` which should be applied to render the exported
         chart. Defaults to :obj:`None <python:None>`.
 
@@ -45,7 +46,7 @@ def options(self) -> Optional[HighchartsOptions | HighchartsGanttOptions]:
     def options(self, value):
         if not value:
             self._options = None
-        elif checkers.is_type(value, ('HighchartsGanttOptions', 'HighchartsOptions')):
+        elif checkers.is_type(value, ('HighchartsGanttOptions', 'HighchartsOptions', 'HighchartsStockOptions')):
             self._options = value
         elif ('navigator' in value
               or 'range_selector' in value
@@ -56,7 +57,7 @@ def options(self, value):
             self._options = validate_types(value, HighchartsOptions)
 
     @property
-    def global_options(self) -> Optional[SharedOptions | SharedGanttOptions]:
+    def global_options(self) -> Optional[SharedOptions | SharedGanttOptions | SharedStockOptions]:
         """The global options which will be passed to the (JavaScript)
         ``Highcharts.setOptions()`` method, and which will be applied to the exported
         chart. Defaults to :obj:`None <python:None>`.
@@ -73,6 +74,8 @@ def global_options(self, value):
         else:
             if checkers.is_type(value, 'SharedGanttOptions'):
                 self._global_options = value
+            elif checkers.is_type(value, 'SharedStockOptions'):
+                self._global_options = value
             elif checkers.is_type(value, 'SharedOptions'):
                 self._global_options = value
             elif ('navigator' in value