fix(datasets): default plotly.JSONDataset save encoding to UTF-8 (#1205)

* Fix:plotly.JSONDataset save encoding to UTF-8

* Fix test

Signed-off-by: Merel Theisen <[email protected]>

* Update release notes

Signed-off-by: Merel Theisen <[email protected]>

---------

Signed-off-by: Merel Theisen <[email protected]>
Co-authored-by: Sreekar Reddy <[email protected]>
Co-authored-by: Ankita Katiyar <[email protected]>
Co-authored-by: Merel Theisen <[email protected]>
Co-authored-by: Merel Theisen <[email protected]>

Author: 4 people

kedro-datasets/RELEASE.md

@@ -17,6 +17,7 @@
 
 ## Bug fixes and other changes
 - Updated `pandas.DeltaTableDataset` to be compatible with `deltalake` version 1.x.
+- Fix `plotly.JSONDataset` encoding errors by defaulting thesave encoding to UTF-8.
 
 ## Community contributions
 

kedro-datasets/kedro_datasets/plotly/json_dataset.py

@@ -52,7 +52,9 @@ class JSONDataset(AbstractVersionedDataset[go.Figure, go.Figure | go.FigureWidge
 
     DEFAULT_LOAD_ARGS: dict[str, Any] = {}
     DEFAULT_SAVE_ARGS: dict[str, Any] = {}
-    DEFAULT_FS_ARGS: dict[str, Any] = {"open_args_save": {"mode": "w"}}
+    DEFAULT_FS_ARGS: dict[str, Any] = {
+        "open_args_save": {"mode": "w", "encoding": "utf-8"}
+    }
 
     def __init__(  # noqa: PLR0913
         self,

kedro-datasets/tests/plotly/test_json_dataset.py

@@ -40,7 +40,7 @@ def test_save_and_load(self, json_dataset, dummy_plot):
         reloaded = json_dataset.load()
         assert dummy_plot == reloaded
         assert json_dataset._fs_open_args_load == {}
-        assert json_dataset._fs_open_args_save == {"mode": "w"}
+        assert json_dataset._fs_open_args_save == {"mode": "w", "encoding": "utf-8"}
 
     def test_exists(self, json_dataset, dummy_plot):
         """Test `exists` method invocation for both existing and