fixed stringify, v2.8.4

satella/__init__.py

-__version__ = '2.8.4_a3'
+__version__ = '2.8.4'

satella/coding/transforms/__init__.py

@@ -11,7 +11,10 @@ def stringify(obj: tp.Union[tp.Any], stringifier: tp.Callable[[tp.Any], str] = s
 
     ie. if a dict, put every item and key (if a dict is given) through stringify.
         if a list, put every item through stringify
-        else just call stringify on it
+        else just call stringify on it.
+
+        Note that if you use recursively, then dicts and lists are allowed to be valid elements of the returned
+        representation!
 
     :param obj: a list or a dict
     :param stringifier: function that accepts any arguments and returns a string representation
@@ -19,13 +22,10 @@ def stringify(obj: tp.Union[tp.Any], stringifier: tp.Callable[[tp.Any], str] = s
     :return: stringified object
     """
     if isinstance(obj, collections.abc.Mapping):
-        if recursively:
-            stringifier = lambda obj2: stringify(obj2, stringifier=stringifier, recursively=True)
-        return {stringifier(k): stringifier(v) for k, v in obj.items()}
+        make_str = (lambda obj2: stringify(obj2, stringifier, True)) if recursively else stringifier
+        return {make_str(k): make_str(v) for k, v in obj.items()}
     elif isinstance(obj, collections.abc.Sequence):
-        if recursively:
-            stringifier = lambda obj2: stringify(obj2, stringifier=stringifier, recursively=True)
-        return [stringifier(v) for v in obj]
+        make_str = (lambda obj2: stringify(obj2, stringifier, True)) if recursively else stringifier
+        return [make_str(v) for v in obj]
     else:
         return stringifier(obj)
-

tests/test_coding/test_transforms.py

@@ -13,3 +13,6 @@ class MyTestCase(unittest.TestCase):
         lst2 = [[1, 2, 3], 3, 4, 5]
         self.assertEqual(stringify(lst2), ['[1, 2, 3]', '3', '4', '5'])
         self.assertEqual(stringify(lst2, recursively=True), [['1', '2', '3'], '3', '4', '5'])
+
+        dct2 = {1: [1, 2, 3]}
+        self.assertEqual(stringify(dct2, recursively=True), {'1': ['1', '2', '3']})