logger creation removed from __init__.py

handout/__init__.py

@@ -1,13 +1,5 @@
-import sys
-import logging
-
 from .handout import Handout
 
+from .logger import configure_logger
 
-# Set up logger.
-logger = logging.getLogger('handout')
-logger.setLevel(logging.INFO)
-logger.propagate = False  # Global logger should not print messages again.
-handler = logging.StreamHandler(sys.stdout)
-handler.setFormatter(logging.Formatter('%(message)s'))
-logger.addHandler(handler)
+configure_logger()

handout/handout.py

@@ -6,6 +6,7 @@
 import shutil
 
 from handout import blocks
+from handout import logger
 
 
 class Handout(object):
@@ -16,7 +17,7 @@ def __init__(self, directory, title='Handout'):
     self._title = title
     self._blocks = collections.defaultdict(list)
     self._pending = []
-    self._logger = logging.getLogger('handout')
+    self._logger = logger.get_logger()
     for info in inspect.stack():
       if info.filename == __file__:
         continue

handout/logger.py

@@ -0,0 +1,17 @@
+import sys
+import logging
+
+LOGGER_NAME = 'handout'
+
+def configure_logger():
+    logger = logging.getLogger(LOGGER_NAME)
+    logger.setLevel(logging.INFO) # A user can change this level later.
+    logger.propagate = False  # Global logger should not print messages again.
+    if not logger.handlers: # Prevents creation of multiple handlers.
+        handler = logging.StreamHandler(sys.stdout)
+        handler.setFormatter(logging.Formatter('%(message)s'))
+        logger.addHandler(handler)
+
+
+def get_logger(): 
+    return logging.getLogger(LOGGER_NAME)

handout/tests/test_logger.py

@@ -0,0 +1,7 @@
+import handout
+
+def test_get_logger_has_single_handler(): 
+    import handout # Simulate multiple imports of a package.
+    import handout
+    logger = handout.logger.get_logger()
+    assert len(logger.handlers) == 1