Initial analysis of logging issue with ArgvContext

Co-authored-by: bittner <[email protected]>

Author: Copilot

cli_test_logging.py

@@ -0,0 +1,14 @@
+import sys
+import logging
+
+def main():
+    logfile = sys.argv[1]
+    print("main")
+    print(f"Handlers before basicConfig: {logging.root.handlers}")
+    logger = logging.getLogger(__name__)
+    logging.basicConfig(filename=logfile, encoding='utf-8', level=logging.INFO)
+    print(f"Handlers after basicConfig: {logging.root.handlers}")
+    logger.info("This is an info message")
+
+if __name__ == "__main__":
+    main()

test_cli.py

@@ -0,0 +1,13 @@
+from pathlib import Path
+from cli_test_helpers import ArgvContext
+import logging
+
+import cli_test_logging
+
+def test_cli():
+    print(f"Handlers at test start: {logging.root.handlers}")
+    logfile = "test.log"
+    with ArgvContext('cli_test_logging.py', logfile):
+        cli_test_logging.main()
+    print(f"Handlers after main: {logging.root.handlers}")
+    assert Path(logfile).exists() # fails

test_cli_workaround.py

@@ -0,0 +1,23 @@
+from pathlib import Path
+from cli_test_helpers import ArgvContext
+import logging
+
+import cli_test_logging
+
+def test_cli_with_workaround():
+    """Test with logging handlers reset"""
+    # Save and clear existing handlers
+    original_handlers = logging.root.handlers[:]
+    logging.root.handlers.clear()
+    
+    logfile = "test.log"
+    try:
+        with ArgvContext('cli_test_logging.py', logfile):
+            cli_test_logging.main()
+        assert Path(logfile).exists()
+    finally:
+        # Restore original handlers
+        logging.root.handlers = original_handlers
+        # Clean up test file
+        if Path(logfile).exists():
+            Path(logfile).unlink()