gui PySide6 port: Fix ThreadWorker tests on Windows

Author: lschr

.github/workflows/testing.yml

@@ -13,7 +13,8 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu, macOS, windows]
+        # os: [ubuntu, macOS, windows]
+        os: [windows]
 
     steps:
     - uses: actions/checkout@v4
@@ -34,10 +35,10 @@ jobs:
       shell: pwsh
       run: uv sync --extra gui
 
-    - name: Test with pytest, no GUI tests
-      shell: pwsh
-      run: |
-        uv run pytest -v -rs --ignore=tests/test_gui tests
+    # - name: Test with pytest, no GUI tests
+    #   shell: pwsh
+    #   run: |
+    #     uv run pytest -v -rs --ignore=tests/test_gui tests
 
     - name: Test with pytest, GUI tests only
       shell: pwsh

sdt/gui/thread_worker.py

@@ -112,7 +112,7 @@ def busy(self) -> bool:
         """True if a function call is currently executed."""
         return self._busy
 
-    enabledChanged = QtCore.Signal(bool)
+    enabledChanged = QtCore.Signal()
     """Enabled status changed"""
 
     @QtCore.Property(bool, notify=enabledChanged)
@@ -143,7 +143,7 @@ def enabled(self, e: bool):
                 # May be necessary to wake up
                 self._callCondition.notify()
             self._workerThread = None
-        self.enabledChanged.emit(e)
+        self.enabledChanged.emit()
 
     def _workerFunc(self):
         """Actual function to be executed in worker process
@@ -207,6 +207,7 @@ def _workerFunc(self):
                 # No exception, also no `return` due to `_stopRequested`
                 self.finished.emit(result)
 
+    @QtCore.Slot()
     def _disable(self):
         """Set ``enabled = False``
 

tests/test_gui/test_thread_worker.py

@@ -70,16 +70,26 @@ def checkBlockingResult(res):
         w3.finished.connect(checkBlockingResult)
         w3.error.connect(assertionOnError)
 
-        with qtbot.assertNotEmitted(w3.finished), \
-                qtbot.assertNotEmitted(w3.error), \
-                qtbot.waitSignal(w3.busyChanged):
+        with (
+            qtbot.assertNotEmitted(w3.finished),
+            qtbot.assertNotEmitted(w3.error),
+            qtbot.waitSignal(w3.busyChanged),
+        ):
             w3(0, True)
             assert w3.busy is True
         barr.wait()
         w3.abort()
         threadBlock.set()
-        with qtbot.waitSignals([w3.finished, w3.busyChanged]):
+        ws = qtbot.waitSignals([(w3.finished, "finished"), (w3.busyChanged, "busyChanged")])
+        print("###")
+        print(ws._signals_map)
+        print("###")
+        with ws:
             w3(1, False)
+        print("***")
+        print(ws._signals_map)
+        print(w3.metaObject())
+        print("***")
 
         qtbot.waitUntil(lambda: w3.busy is False)
     finally: