Add unit test

Author: pmcollins

opentelemetry-sdk/src/opentelemetry/sdk/_configuration/__init__.py

@@ -264,29 +264,18 @@ def _patch_basic_config():
     original_basic_config = logging.basicConfig
 
     def patched_basic_config(*args, **kwargs):
-        # Get the root logger
         root = logging.getLogger()
-
-        # Check if the only handler is our OTel handler
         has_only_otel = len(root.handlers) == 1 and isinstance(
             root.handlers[0], LoggingHandler
         )
-
         if has_only_otel:
-            # Temporarily remove OTel handler
             otel_handler = root.handlers[0]
             root.handlers = []
-
-            # Call original basicConfig
             original_basic_config(*args, **kwargs)
-
-            # Add OTel handler back
             root.addHandler(otel_handler)
         else:
-            # Normal behavior
             original_basic_config(*args, **kwargs)
 
-    # Apply the monkey patch
     logging.basicConfig = patched_basic_config
 
 

opentelemetry-sdk/tests/test_configurator.py

@@ -842,30 +842,58 @@ def test_initialize_components_kwargs(
 
     def test_basicConfig_works_with_otel_handler(self):
         with ClearLoggingHandlers():
-            # Initialize auto-instrumentation with logging enabled
             _init_logging(
                 {"otlp": DummyOTLPLogExporter},
                 Resource.create({}),
                 setup_logging_handler=True,
             )
 
-            # Call basicConfig - this should work despite OTel handler being present
             logging.basicConfig(level=logging.INFO)
 
-            # Verify a StreamHandler was added
             root_logger = logging.getLogger()
             stream_handlers = [
                 h
                 for h in root_logger.handlers
                 if isinstance(h, logging.StreamHandler)
-                and not isinstance(h, LoggingHandler)
             ]
             self.assertEqual(
                 len(stream_handlers),
                 1,
                 "basicConfig should add a StreamHandler even when OTel handler exists",
             )
 
+    def test_basicConfig_preserves_otel_handler(self):
+        with ClearLoggingHandlers():
+            _init_logging(
+                {"otlp": DummyOTLPLogExporter},
+                Resource.create({}),
+                setup_logging_handler=True,
+            )
+
+            root_logger = logging.getLogger()
+            self.assertEqual(
+                len(root_logger.handlers),
+                1,
+                "Should be exactly one OpenTelemetry LoggingHandler",
+            )
+            handler = root_logger.handlers[0]
+            self.assertIsInstance(handler, LoggingHandler)
+
+            logging.basicConfig()
+
+            self.assertGreater(len(root_logger.handlers), 1)
+
+            logging_handlers = [
+                h
+                for h in root_logger.handlers
+                if isinstance(h, LoggingHandler)
+            ]
+            self.assertEqual(
+                len(logging_handlers),
+                1,
+                "Should still have exactly one OpenTelemetry LoggingHandler",
+            )
+
 
 class TestMetricsInit(TestCase):
     def setUp(self):