Display handlers in logger repr as list rather than dict

Author: Delgan

loguru/_handler.py

@@ -69,7 +69,7 @@ def __init__(
             self._thread.start()
 
     def __repr__(self):
-        return "(level=%d, writer=%s)" % (self._levelno, self._name)
+        return "(id=%d, level=%d, writer=%s)" % (self._id, self._levelno, self._name)
 
     def emit(self, record, level_id, is_ansi, is_raw):
         try:

loguru/_logger.py

@@ -854,7 +854,7 @@ def filter_func(record):
         return handler_id
 
     def __repr__(self):
-        return "<loguru.logger handlers=%r>" % Logger._handlers
+        return "<loguru.logger handlers=%r>" % list(Logger._handlers.values())
 
     def remove(self, handler_id=None):
         """Remove a previously added handler and stop sending logs to its sink.

tests/test_repr.py

@@ -5,36 +5,36 @@
 import re
 
 def test_no_handler():
-    assert repr(logger) == "<loguru.logger handlers={}>"
+    assert repr(logger) == "<loguru.logger handlers=[]>"
 
 
 def test_stderr():
     logger.add(sys.__stderr__)
-    assert repr(logger) == "<loguru.logger handlers={0: (level=10, writer=<stderr>)}>"
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=10, writer=<stderr>)]>"
 
 
 def test_stdout():
     logger.add(sys.__stdout__)
-    assert repr(logger) == "<loguru.logger handlers={0: (level=10, writer=<stdout>)}>"
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=10, writer=<stdout>)]>"
 
 
 def test_file_object(tmpdir):
     path = str(tmpdir.join("test.log"))
     file = open(path, "w")
     logger.add(file)
-    assert repr(logger) == "<loguru.logger handlers={0: (level=10, writer=%s)}>" % path
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=10, writer=%s)]>" % path
 
 
 def test_file_str(tmpdir):
     path = str(tmpdir.join("test.log"))
     logger.add(path)
-    assert repr(logger) == "<loguru.logger handlers={0: (level=10, writer=%s)}>" % path
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=10, writer=%s)]>" % path
 
 
 def test_file_pathlib(tmpdir):
     path = str(tmpdir.join("test.log"))
     logger.add(pathlib.Path(path))
-    assert repr(logger) == "<loguru.logger handlers={0: (level=10, writer=%s)}>" % path
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=10, writer=%s)]>" % path
 
 
 def test_stream_object():
@@ -49,7 +49,7 @@ def __repr__(self):
             return "MyStream()"
 
     logger.add(MyStream("<foobar>"))
-    assert repr(logger) == "<loguru.logger handlers={0: (level=10, writer=<foobar>)}>"
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=10, writer=<foobar>)]>"
 
 
 def test_stream_object_without_name_attr():
@@ -61,7 +61,7 @@ def __repr__(self):
             return "MyStream()"
 
     logger.add(MyStream())
-    assert repr(logger) == "<loguru.logger handlers={0: (level=10, writer=MyStream())}>"
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=10, writer=MyStream())]>"
 
 
 def test_class():
@@ -76,7 +76,7 @@ def __repr__(self):
             return "MyStream()"
 
     logger.add(MyStream)
-    assert repr(logger) == "<loguru.logger handlers={0: (level=10, writer=<foobar>)}>"
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=10, writer=<foobar>)]>"
 
 
 def test_class_without_name_attr():
@@ -88,15 +88,15 @@ def __repr__(self):
             return "MyStream()"
 
     logger.add(MyStream)
-    assert repr(logger) == "<loguru.logger handlers={0: (level=10, writer=MyStream())}>"
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=10, writer=MyStream())]>"
 
 
 def test_function():
     def my_function(message):
         pass
 
     logger.add(my_function)
-    assert repr(logger) == "<loguru.logger handlers={0: (level=10, writer=my_function)}>"
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=10, writer=my_function)]>"
 
 
 def test_function_without_name():
@@ -114,49 +114,49 @@ def __getattr__(self, name):
 
     function = Function()
     logger.add(function)
-    assert repr(logger) == "<loguru.logger handlers={0: (level=10, writer=<Function>)}>"
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=10, writer=<Function>)]>"
 
 
 def test_standard_handler():
     handler = logging.StreamHandler(sys.__stderr__)
     logger.add(handler)
     if sys.version_info >= (3, 6):
-        r = "<loguru.logger handlers={0: (level=10, writer=<StreamHandler <stderr> (NOTSET)>)}>"
+        r = "<loguru.logger handlers=[(id=0, level=10, writer=<StreamHandler <stderr> (NOTSET)>)]>"
         assert repr(logger) == r
     else:
-        r = r"<loguru\.logger handlers=\{0: \(level=10, writer=<logging\.StreamHandler .*>\)\}>"
+        r = r"<loguru\.logger handlers=\[\(id=0, level=10, writer=<logging\.StreamHandler .*>\)\]>"
         assert re.match(r, repr(logger))
 
 
 def test_standard_handler_class():
     handler = logging.Handler
     logger.add(handler)
     if sys.version_info >= (3, 6):
-        r = "<loguru.logger handlers={0: (level=10, writer=<Handler (NOTSET)>)}>"
+        r = "<loguru.logger handlers=[(id=0, level=10, writer=<Handler (NOTSET)>)]>"
         assert repr(logger) == r
     else:
-        r = r"<loguru\.logger handlers=\{0: \(level=10, writer=<logging\.Handler .*>\)\}>"
+        r = r"<loguru\.logger handlers=\[\(id=0, level=10, writer=<logging\.Handler .*>\)\]>"
         assert re.match(r, repr(logger))
 
 def test_multiple_handlers():
     logger.add(sys.__stdout__)
     logger.add(sys.__stderr__)
-    r = "<loguru.logger handlers={0: (level=10, writer=<stdout>), 1: (level=10, writer=<stderr>)}>"
+    r = "<loguru.logger handlers=[(id=0, level=10, writer=<stdout>), (id=1, level=10, writer=<stderr>)]>"
     assert repr(logger) == r
 
 
 def test_handler_removed():
     i = logger.add(sys.__stdout__)
     logger.add(sys.__stderr__)
     logger.remove(i)
-    assert repr(logger) == "<loguru.logger handlers={1: (level=10, writer=<stderr>)}>"
+    assert repr(logger) == "<loguru.logger handlers=[(id=1, level=10, writer=<stderr>)]>"
 
 
 def test_handler_level_name():
     logger.add(sys.__stderr__, level="TRACE")
-    assert repr(logger) == "<loguru.logger handlers={0: (level=5, writer=<stderr>)}>"
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=5, writer=<stderr>)]>"
 
 
 def test_handler_level_num():
     logger.add(sys.__stderr__, level=33)
-    assert repr(logger) == "<loguru.logger handlers={0: (level=33, writer=<stderr>)}>"
+    assert repr(logger) == "<loguru.logger handlers=[(id=0, level=33, writer=<stderr>)]>"