set_str & remove useless defaults

Signed-off-by: Alexey Stepanov <penguinolog@gmail.com>

Author: penguinolog

README.rst

@@ -143,8 +143,8 @@ Object signature:
 
     def __init__(
         self,
-        simple_formatters=None,  # Will be used to repr not complex. Keys is data types and 'default'.
-        complex_formatters=None,  # Currently only legacy pretty_repr formatters is supported, will be extended in the future
+        simple_formatters,  # Will be used to repr not complex. Keys is data types and 'default'.
+        complex_formatters,  # Currently only legacy pretty_repr formatters is supported, will be extended in the future
         keyword='repr',  # Currently 'repr' is supported, will be extended in the future
         max_indent=20,  # maximum allowed indent level
         indent_step=4,  # step between indents

logwrap/_formatters.py

@@ -101,14 +101,14 @@ def _set_str(indent, val):
     return "{spc:<{indent}}{val}".format(
         spc='',
         indent=indent,
-        val='{' + "{}".format(
+        val="set({})".format(
             ' ,'.join(
                 map(
-                    '{!s}'.format,  # unicode -> !repr
+                    '{!s}'.format,
                     val
                 )
             )
-        ) + '}'
+        )
     )
 
 

logwrap/_repr_utils.py

@@ -104,8 +104,8 @@ class PrettyFormat(object):
 
     def __init__(
         self,
-        simple_formatters=None,
-        complex_formatters=None,
+        simple_formatters,
+        complex_formatters,
         keyword='repr',
         max_indent=20,
         indent_step=4,
@@ -125,11 +125,6 @@ def __init__(
         :param py2_str: use Python 2.x compatible strings instead of unicode
         :type py2_str: bool
         """
-        if simple_formatters is None:
-            simple_formatters = s_repr_formatters
-        if complex_formatters is None:
-            complex_formatters = c_repr_formatters
-
         self.__s_formatters = simple_formatters
         self.__c_formatters = complex_formatters
         self.__keyword = keyword

test/test_pretty_str.py

@@ -56,6 +56,12 @@ def test_iterable(self):
             res.startswith('\n{') and res.endswith('\n}')
         )
 
+    def test_simple_set(self):
+        self.assertEqual(
+            logwrap.pretty_str(set()),
+            'set()'
+        )
+
     def test_dict(self):
         self.assertEqual(
             logwrap.pretty_str({1: 1, 2: 2, 33: 33}),