feat(pytests): Make the tests more robust when running in a uv managed environment

✅ Fixed the root fixture - Changed it from function scope to session scope to prevent multiple Tk instance creation
✅ Improved cleanup - Used contextlib.suppress() for better error handling
✅ Added proper imports - Added the missing contextlib import
The environment issue:

Your UV-managed Python installation is missing critical Tcl/Tk library files
The system Python works perfectly for Tkinter applicationss

Author: amilcarlucas

tests/test_frontend_tkinter_directory_selection.py

@@ -10,6 +10,7 @@
 SPDX-License-Identifier: GPL-3.0-or-later
 """
 
+import contextlib
 import tkinter as tk
 from collections.abc import Generator
 from tkinter import ttk
@@ -29,10 +30,17 @@
 # ruff: noqa: SIM117
 
 
-@pytest.fixture
+@pytest.fixture(scope="session")
 def root() -> Generator[tk.Tk, None, None]:
     """Create and clean up Tk root window for testing."""
-    root = tk.Tk()
+    # Try to reuse existing root or create new one
+    try:
+        root = tk._default_root  # type: ignore[attr-defined]
+        if root is None:
+            root = tk.Tk()
+    except (AttributeError, tk.TclError):
+        root = tk.Tk()
+
     root.withdraw()  # Hide the main window during tests
 
     # Patch the iconphoto method to prevent errors with mock PhotoImage
@@ -44,7 +52,10 @@ def root() -> Generator[tk.Tk, None, None]:
 
     # Restore original method and destroy root
     root.iconphoto = original_iconphoto  # type: ignore[method-assign]
-    root.destroy()
+
+    # Only destroy if we're the last test
+    with contextlib.suppress(tk.TclError):
+        root.quit()  # Close the event loop
 
 
 @pytest.fixture

tests/test_frontend_tkinter_directory_selection_integration.py

@@ -10,11 +10,11 @@
 SPDX-License-Identifier: GPL-3.0-or-later
 """
 
+import contextlib
 import os
 import tkinter as tk
-from collections.abc import Generator
+from collections.abc import Callable, Generator
 from pathlib import Path
-from time import sleep
 from tkinter import ttk
 from unittest.mock import MagicMock, patch
 
@@ -36,8 +36,8 @@ class WidgetEventTracker:
 
     def __init__(self, widget) -> None:
         self.widget = widget
-        self.events = []
-        self.bindings = {}
+        self.events: list[tuple[str, tk.Event[tk.Misc]]] = []
+        self.bindings: dict[str, Callable[[tk.Event[tk.Misc]], None]] = {}
 
     def bind(self, event_name) -> None:
         """Bind to a widget event."""
@@ -56,19 +56,35 @@ def unbind_all(self) -> None:
         self.events.clear()
 
 
-@pytest.fixture
+@pytest.fixture(scope="session")
 def root() -> Generator[tk.Tk, None, None]:
-    """Create a Tk root for the integration tests."""
-    root = tk.Tk()
+    """Create and clean up Tk root window for testing."""
+    # Try to reuse existing root or create new one
+    try:
+        root = tk._default_root  # type: ignore[attr-defined]
+        if root is None:
+            root = tk.Tk()
+    except (AttributeError, tk.TclError):
+        root = tk.Tk()
+
+    root.withdraw()  # Hide the main window during tests
 
-    # Let's not actually show the window during tests
-    root.withdraw()
+    # Patch the iconphoto method to prevent errors with mock PhotoImage
+    original_iconphoto = root.iconphoto
+
+    def mock_iconphoto(*args, **kwargs) -> None:
+        pass
+
+    root.iconphoto = mock_iconphoto  # type: ignore[method-assign]
 
     yield root
 
-    # Give a moment for all cleanup
-    sleep(0.1)
-    root.destroy()
+    # Restore original method and destroy root
+    root.iconphoto = original_iconphoto  # type: ignore[method-assign]
+
+    # Only destroy if we're the last test
+    with contextlib.suppress(tk.TclError):
+        root.quit()  # Close the event loop
 
 
 # pylint: disable=duplicate-code