Only propagate logger in pytest

Pytest hooks into the root logger, which breaks the use case here where
we only want a single handler to emit a log record. Create a special
carve out for pytest and put it in `get_script_logger` to make testing
logging results of any function easy.

flepimop/gempyor_pkg/src/gempyor/logging.py

@@ -13,6 +13,8 @@
 
 
 import logging
+import os
+import sys
 from typing import Any, IO
 
 import click
@@ -119,7 +121,8 @@ def get_script_logger(
         logger.removeHandler(old_handler)
     handler.setFormatter(log_formatter)
     logger.addHandler(handler)
-    logger.propagate = False
+    # pytest-dev/pytest#3697
+    logger.propagate = os.path.basename(sys.argv[0]) == "pytest" if sys.argv else False
     return logger
 
 

flepimop/gempyor_pkg/tests/logging/test_get_script_logger.py

@@ -41,9 +41,6 @@ def test_get_script_logger(
             name, verbosity, handler=handler, log_format=log_format
         )
 
-    # pytest-dev/pytest#3697
-    logger.propagate = True
-
     assert isinstance(logger, logging.Logger)
     assert logger.name == name
     assert logger.level == _get_logging_level(verbosity)