logger crash should crash, don't want crash? don't init logger and crash callback

Author: azkrishpy

test/test_io.py

@@ -326,37 +326,6 @@ def test_log_subject_hierarchy(self):
         finally:
             logger.removeHandler(handler)
 
-    def test_callback_exception_does_not_crash(self):
-        """If a logging handler raises, CRT should not crash and subsequent logging should still work."""
-
-        class BrokenHandler(logging.Handler):
-            def emit(self, record):
-                raise RuntimeError("handler exploded")
-
-        logger = logging.getLogger('awscrt')
-        logger.setLevel(logging.DEBUG)
-        broken = BrokenHandler()
-        logger.addHandler(broken)
-
-        try:
-            # This triggers the C callback -> Python callback -> handler.emit -> raise
-            # Should not crash or leave the process in a bad state
-            log(logging.INFO, LogSubject.COMMON_GENERAL, "this should not crash")
-        finally:
-            logger.removeHandler(broken)
-
-        # Verify logging still works after the error
-        good = logging.Handler()
-        good.records = []
-        good.emit = lambda record: good.records.append(record)
-        logger.addHandler(good)
-        try:
-            log(logging.INFO, LogSubject.COMMON_GENERAL, "after error")
-            self.assertEqual(len(good.records), 1)
-            self.assertIn("after error", good.records[0].getMessage())
-        finally:
-            logger.removeHandler(good)
-
     def _emit_all_subjects_and_levels(self):
         """Emit logs across all subjects at INFO and DEBUG levels."""
         for subject in (LogSubject.IO_EVENT_LOOP, LogSubject.HTTP_GENERAL,