Improve handler cleanup by collecting handlers before removal

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

Author: Copilot

cli_test_helpers/decorators.py

@@ -35,11 +35,15 @@ def __exit__(self, exc_type, exc_val, exc_tb):
         """Restore logging handlers after exiting the context."""
         # Close and remove any handlers that were added during the context
         if hasattr(self, "_old_handlers"):
-            for handler in logging.root.handlers[:]:
-                # Close handlers that weren't in the original list
-                if handler not in self._old_handlers:
-                    handler.close()
-                    logging.root.handlers.remove(handler)
+            # Collect handlers to close (those not in the original list)
+            handlers_to_close = [
+                handler for handler in logging.root.handlers
+                if handler not in self._old_handlers
+            ]
+            # Close and remove them
+            for handler in handlers_to_close:
+                handler.close()
+                logging.root.handlers.remove(handler)
             # Restore the original handlers
             logging.root.handlers[:] = self._old_handlers
         # Call parent __exit__ if it exists (for cooperative inheritance)