From f384656d87e1a1693c376d7509b78723036cd6e0 Mon Sep 17 00:00:00 2001 From: Samuel Abels Date: Mon, 19 May 2025 21:02:41 +0200 Subject: [PATCH] CAM: Fix Gui test for ToolBitBrowserWidget --- src/Mod/CAM/CAMTests/TestPathToolBitBrowserWidget.py | 6 ++++++ src/Mod/CAM/Path/Tool/toolbit/ui/browser.py | 2 +- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/src/Mod/CAM/CAMTests/TestPathToolBitBrowserWidget.py b/src/Mod/CAM/CAMTests/TestPathToolBitBrowserWidget.py index a0ee20711b..df4dc9ae23 100644 --- a/src/Mod/CAM/CAMTests/TestPathToolBitBrowserWidget.py +++ b/src/Mod/CAM/CAMTests/TestPathToolBitBrowserWidget.py @@ -41,6 +41,9 @@ class TestToolBitBrowserWidget(PathTestWithAssets): self.widget = ToolBitBrowserWidget(self.assets) def test_initial_fetch(self): + # Simulate expose to trigger initial fetch + self.widget.showEvent(None) + # Verify that the list widget is populated after initialization # The default test assets include some toolbits. self.assertGreater(self.widget._tool_list_widget.count(), 0) @@ -79,6 +82,9 @@ class TestToolBitBrowserWidget(PathTestWithAssets): if is_expected: expected_visible_uris.add(str(tb.get_uri())) + # Simulate expose to trigger initial fetch + self.widget.showEvent(None) + actual_visible_uris = set() for i in range(self.widget._tool_list_widget.count()): item = self.widget._tool_list_widget.item(i) diff --git a/src/Mod/CAM/Path/Tool/toolbit/ui/browser.py b/src/Mod/CAM/Path/Tool/toolbit/ui/browser.py index 55b54e485e..0828143414 100644 --- a/src/Mod/CAM/Path/Tool/toolbit/ui/browser.py +++ b/src/Mod/CAM/Path/Tool/toolbit/ui/browser.py @@ -133,9 +133,9 @@ class ToolBitBrowserWidget(QtGui.QWidget): ), ) self._sort_assets() - self._trigger_fetch() finally: self._is_fetching = False + self._trigger_fetch() def _sort_assets(self): """Sorts the in-memory assets based on the current sort key."""