add a warning: setup_logging() should only be called once.

Author: Dragon2fly

README.md

@@ -656,6 +656,9 @@ there are loggers in submodules, these loggers do not inspect exception when you
 That is because there class was different than loggers created after importing `logger_tt`.
 Now all loggers have the same new class regardless the point of importing `logger_tt`.
 
+**setup_logging()**: This function should only be called once. 
+Add a warning if it is called more than one time.
+
 ## 1.5.1
 * Use `socketHandler` as default for multiprocessing.
  Under linux, to use `queueHandler`, user must pass `use_multiprocessing="fork"` to `setup_logging` 

logger_tt/__init__.py

@@ -111,6 +111,9 @@ def setup_logging(config_path="", log_path="", **logger_tt_config) -> LogConfig:
                                     This flag switches the queue used for logging from
                                     queue.Queue to multiprocessing.Queue . This option can only be used here.
     """
+    if internal_config.initialized:
+        logger.warning('Re-initializing logger_tt. "setup_logging()" should only be called one.')
+
     if config_path:
         cfgpath = Path(config_path)
         assert cfgpath.is_file(), 'Input config path is not a file!'

logger_tt/core.py

@@ -43,6 +43,13 @@ def __init__(self):
         # suppress logger list for usage in filter
         self.suppress_list = set()
 
+        # initialized counter
+        self.__initialized = 0
+
+    @property
+    def initialized(self):
+        return self.__initialized
+
     def from_dict(self, odict: dict):
         # store basic settings
         for key in ['full_context', 'strict', 'guess_level']:
@@ -63,6 +70,8 @@ def from_dict(self, odict: dict):
         # set logging mode accordingly
         self.set_mode(odict['use_multiprocessing'])
 
+        self.__initialized += 1
+
     def set_mode(self, use_multiprocessing):
         """Select logging method according to platform and multiprocessing"""
         os_name = platform.system()

tests/test_simple.py

@@ -109,17 +109,12 @@ def test_suppress_logger(capsys, level):
 
 def test_suppress_logger2(capsys):
     # suppress by the code
-    with setup_logging(suppress=['urllib3']):
-        from tests.exchangelib_logger import fox_run
-
-        fox_run()
+    with setup_logging(suppress=['suppressed_logger']):
+        logger = getLogger('suppressed_logger')
+        logger.info('Ha ha, this should not be logged')
 
     stdout_data = capsys.readouterr().out
-    assert 'DEBUG' not in stdout_data
-    assert 'INFO' in stdout_data
-    assert 'WARNING' in stdout_data
-    assert 'ERROR' in stdout_data
-    assert 'CRITICAL' in stdout_data
+    assert 'INFO' not in stdout_data
 
 
 def test_logging_disabled(capsys):
@@ -190,3 +185,13 @@ def test_default_logger_suppress():
 
     log_data = log.read_text()
     assert len(re.findall(r"tests.sub_module:\d+ INFO", log_data)) == 0
+
+
+def test_double_setup_logging():
+    # there should be a warning if setup_logging() is called multiple times
+    with setup_logging() as log_config:
+        with setup_logging() as log_config:
+            pass
+
+    log_data = log.read_text()
+    assert "WARNING" in log_data