Fixed bug in primitive array serialization.

Author: hcpchris

highcharts_core/js_literal_functions.py

@@ -25,7 +25,7 @@ def serialize_to_js_literal(item, encoding = 'utf-8') -> Optional[str]:
     :rtype: :class:`str <python:str>` or :obj:`None <python:None>`
     """
     if checkers.is_iterable(item, forbid_literals = (str, bytes, dict, UserDict)):
-        requires_js_objects = all([getattr(x, 'requires_js_object', True) 
+        requires_js_objects = any([getattr(x, 'requires_js_object', True) 
                                    for x in item])
         if requires_js_objects:
             return [serialize_to_js_literal(x, encoding = encoding) 

tests/options/series/data/test_base.py

@@ -1,6 +1,7 @@
 """Tests for ``highcharts.no_data``."""
 
 import pytest
+from typing import List
 
 from json.decoder import JSONDecodeError
 
@@ -17,13 +18,38 @@ class NonAbstractDataBase(DataBase):
     def from_array(cls, value):
         pass
 
+    def _get_props_from_array(self) -> List[str]:
+        """Returns a list of the property names that can be set using the
+        :meth:`.from_array() <highcharts_core.options.series.data.base.DataBase.from_array>`
+        method.
+        
+        :rtype: :class:`list <python:list>` of :class:`str <python:str>`
+        """
+        return ['fromArrayProp1', 'fromArrayProp2']
+
+
+
 cls = NonAbstractDataBase
 
 
 class RequiringJSObject(NonAbstractDataBase):
     def _to_untrimmed_dict(self):
         return {'someKey': 123}
 
+class NotRequiringJSObject(NonAbstractDataBase):
+    def _to_untrimmed_dict(self):
+        return {
+            'fromArrayProp1': 456,
+            'fromArrayProp2': 789
+        }
+
+
+class RequiringJSObject2(NonAbstractDataBase):
+    def _to_untrimmed_dict(self):
+        return {'someKey': 123,
+                'fromArrayProp1': 456,
+                'fromArrayProp2': 789}
+
 
 STANDARD_PARAMS = [
     ({}, None),
@@ -102,8 +128,10 @@ def test_from_js_literal(input_files, filename, as_file, error):
 
 @pytest.mark.parametrize('cls, expected', [
     (NonAbstractDataBase, False),
+    (NotRequiringJSObject, False),
     (RequiringJSObject, True),
-    
+    (RequiringJSObject2, True)
+
 ])
 def test_requires_js_object(cls, expected):
     obj = cls()