Updated main window and tests

Author: chinefed

dna2graph/gui/main_window.py

@@ -46,6 +46,10 @@ def __init__(self):
         self.stop_event = mp.Event()
 
         self._build_ui()
+        self.update_idletasks()
+        required_width = max(720, self.winfo_reqwidth())
+        required_height = max(570, self.winfo_reqheight() + 10)
+        self.geometry(f'{required_width}x{required_height}')
 
     def _build_ui(self):
         # === Path Selectors ====
@@ -324,4 +328,4 @@ def _start_analysis(self):
     def open_feedback_email(self, event=None):
         webbrowser.open(
             f'mailto:{DEVELOPER_EMAIL}?subject=Feedback {APP_NAME}'
-        )
+        )

tests/test_gui.py

@@ -21,6 +21,41 @@ def _find_button(parent, text):
     return None
 
 
+def _capture_window(window, output_path):
+    try:
+        window.deiconify()
+        window.lift()
+        window.attributes("-topmost", True)
+        window.focus_force()
+        window.update_idletasks()
+        window.update()
+        window.after(250)
+        window.update()
+
+        left = window.winfo_rootx()
+        top = window.winfo_rooty()
+        right = left + window.winfo_width()
+        bottom = top + window.winfo_height()
+
+        grab_options = {}
+        if "DISPLAY" in os.environ:
+            grab_options["xdisplay"] = os.environ["DISPLAY"]
+
+        screenshot = ImageGrab.grab(
+            bbox=(left, top, right, bottom),
+            **grab_options,
+        )
+        screenshot.save(output_path)
+        window.attributes("-topmost", False)
+    except Exception as error:
+        warnings.warn(
+            f"Unable to capture {output_path.name} on {sys.platform}: "
+            f"{error}",
+            RuntimeWarning,
+            stacklevel=1,
+        )
+
+
 def test_gui_opens(tmp_path):
     window = MainWindow()
 
@@ -43,6 +78,11 @@ def test_gui_opens(tmp_path):
         assert advanced_window.winfo_exists()
         assert advanced_window.title() == "Advanced Preferences"
 
+        _capture_window(
+            advanced_window,
+            tmp_path / "dna2graph_advanced_preferences.png",
+        )
+
         cancel_button = _find_button(advanced_window, "Cancel")
         assert cancel_button is not None
 
@@ -51,38 +91,6 @@ def test_gui_opens(tmp_path):
         window.update()
 
         assert not advanced_window.winfo_exists()
-
-        try:
-            window.deiconify()
-            window.lift()
-            window.attributes("-topmost", True)
-            window.focus_force()
-            window.update_idletasks()
-            window.update()
-            window.after(250)
-            window.update()
-
-            left = window.winfo_rootx()
-            top = window.winfo_rooty()
-            right = left + window.winfo_width()
-            bottom = top + window.winfo_height()
-
-            grab_options = {}
-            if "DISPLAY" in os.environ:
-                grab_options["xdisplay"] = os.environ["DISPLAY"]
-
-            screenshot = ImageGrab.grab(
-                bbox=(left, top, right, bottom),
-                **grab_options,
-            )
-            screenshot.save(tmp_path / "dna2graph_gui.png")
-            window.attributes("-topmost", False)
-        except Exception as error:
-            warnings.warn(
-                f"Unable to capture the GUI screenshot on {sys.platform}: "
-                f"{error}",
-                RuntimeWarning,
-                stacklevel=1,
-            )
+        _capture_window(window, tmp_path / "dna2graph_gui.png")
     finally:
         window.destroy()