synodic-client 0.0.1.dev69__tar.gz → 0.0.1.dev71__tar.gz

{synodic_client-0.0.1.dev69 → synodic_client-0.0.1.dev71}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: synodic_client
-Version: 0.0.1.dev69
+Version: 0.0.1.dev71
 Author-Email: Synodic Software <contact@synodic.software>
 License: LGPL-3.0-or-later
 Project-URL: homepage, https://github.com/synodic/synodic-client
@@ -8,7 +8,7 @@ Project-URL: repository, https://github.com/synodic/synodic-client
 Requires-Python: <3.15,>=3.14
 Requires-Dist: pyside6>=6.10.2
 Requires-Dist: packaging>=26.0
-Requires-Dist: porringer>=0.2.1.dev80
+Requires-Dist: porringer>=0.2.1.dev81
 Requires-Dist: qasync>=0.28.0
 Requires-Dist: velopack>=0.0.1444.dev49733
 Requires-Dist: typer>=0.24.1

{synodic_client-0.0.1.dev69 → synodic_client-0.0.1.dev71}/pyproject.toml

@@ -10,12 +10,12 @@ requires-python = ">=3.14, <3.15"
 dependencies = [
     "pyside6>=6.10.2",
     "packaging>=26.0",
-    "porringer>=0.2.1.dev80",
+    "porringer>=0.2.1.dev81",
     "qasync>=0.28.0",
     "velopack>=0.0.1444.dev49733",
     "typer>=0.24.1",
 ]
-version = "0.0.1.dev69"
+version = "0.0.1.dev71"
 
 [project.license]
 text = "LGPL-3.0-or-later"
@@ -36,7 +36,7 @@ build = [
 ]
 lint = [
     "ruff>=0.15.5",
-    "pyrefly>=0.55.0",
+    "pyrefly>=0.56.0",
 ]
 test = [
     "pytest>=9.0.2",

{synodic_client-0.0.1.dev69 → synodic_client-0.0.1.dev71}/synodic_client/application/screen/settings.py

@@ -8,7 +8,7 @@ Updates* button with inline status feedback.
 import logging
 import sys
 import traceback
-from collections.abc import Iterator
+from collections.abc import Generator
 from contextlib import contextmanager
 
 from PySide6.QtCore import Qt, QUrl, Signal
@@ -334,7 +334,7 @@ class SettingsWindow(QMainWindow):
         self._store.update(**changes)
 
     @contextmanager
-    def _block_signals(self) -> Iterator[None]:
+    def _block_signals(self) -> Generator[None]:
         """Temporarily block signals on all settings controls."""
         widgets = (
             self._channel_combo,

{synodic_client-0.0.1.dev69 → synodic_client-0.0.1.dev71}/synodic_client/application/update_controller.py

@@ -236,7 +236,17 @@ class UpdateController:
         )
 
     def _reinitialize_updater(self, config: ResolvedConfig) -> None:
-        """Re-derive update settings and restart the updater and timer."""
+        """Re-derive update settings and restart the updater and timer.
+
+        Cancels any in-flight check/download task and clears cached
+        state so the new updater starts with a clean slate.
+        """
+        if self._update_task is not None and not self._update_task.done():
+            self._update_task.cancel()
+            self._update_task = None
+        self._pending_version = None
+        self._failed_version = None
+
         update_cfg = resolve_update_config(config)
         self._client.initialize_updater(update_cfg)
         self._restart_auto_update_timer()
@@ -414,6 +424,10 @@ class UpdateController:
         if self._client.updater is None:
             return
 
+        if self._pending_version is None:
+            self._report_error('No downloaded update to apply — please check for updates again.', silent=silent)
+            return
+
         try:
             # Re-register the startup entry with the current exe path so
             # the registry value stays valid even if Velopack relocates

{synodic_client-0.0.1.dev69 → synodic_client-0.0.1.dev71}/synodic_client/subprocess_patch.py

@@ -75,8 +75,8 @@ def _inject_hidden_flags(kwargs: dict[str, Any]) -> None:
 def _patch_popen() -> None:
     _original_init = subprocess.Popen.__init__
 
-    def _patched_init(self: subprocess.Popen, *args: Any, **kwargs: Any) -> None:  # type: ignore[type-arg]
+    def _patched_init(self: subprocess.Popen, *args: Any, **kwargs: Any) -> None:
         _inject_hidden_flags(kwargs)
         _original_init(self, *args, **kwargs)
 
-    subprocess.Popen.__init__ = _patched_init  # type: ignore[method-assign]
+    subprocess.Popen.__init__ = _patched_init

{synodic_client-0.0.1.dev69 → synodic_client-0.0.1.dev71}/tests/unit/qt/test_update_controller.py

@@ -347,6 +347,7 @@ class TestApplyUpdate:
     def test_apply_update_calls_client_and_quits() -> None:
         """_apply_update should call client.apply_update_on_exit and app.quit."""
         ctrl, app, client, banner, model = _make_controller()
+        ctrl._pending_version = '2.0.0'
         ctrl._apply_update()
 
         client.apply_update_on_exit.assert_called_once_with(restart=True, silent=False)
@@ -366,6 +367,7 @@ class TestApplyUpdate:
     def test_apply_update_refreshes_startup_registry_when_frozen() -> None:
         """_apply_update should call sync_startup before quitting."""
         ctrl, app, client, banner, model = _make_controller()
+        ctrl._pending_version = '2.0.0'
 
         with (
             patch('synodic_client.application.update_controller.sync_startup') as mock_sync,
@@ -497,3 +499,89 @@ class TestPersistCheckTimestamp:
             mock_update.assert_called_once()
 
         assert len(spy.last_checked) == 1
+
+
+# ---------------------------------------------------------------------------
+# Reinitialise updater — stale state cleared
+# ---------------------------------------------------------------------------
+
+
+class TestReinitializeUpdater:
+    """Verify _reinitialize_updater clears stale state."""
+
+    @staticmethod
+    def test_reinit_resets_state_and_cancels_task() -> None:
+        """Reinitialising should clear pending/failed versions and cancel in-flight tasks."""
+        ctrl, _app, _client, _banner, _model = _make_controller()
+        ctrl._pending_version = '1.0.0'
+        ctrl._failed_version = '1.0.0'
+        fake_task = MagicMock()
+        fake_task.done.return_value = False
+        ctrl._update_task = fake_task
+
+        new_config = _make_config(update_channel='dev')
+        with patch('synodic_client.application.update_controller.resolve_update_config') as mock_ucfg:
+            mock_ucfg.return_value = MagicMock(
+                auto_update_interval_minutes=0,
+                channel=MagicMock(name='DEVELOPMENT'),
+                repo_url='https://example.com',
+            )
+            ctrl._reinitialize_updater(new_config)
+
+        assert ctrl._pending_version is None  # pyrefly: ignore
+        assert ctrl._failed_version is None  # pyrefly: ignore
+        fake_task.cancel.assert_called_once()
+
+    @staticmethod
+    def test_reinit_then_check_redownloads_same_version() -> None:
+        """After reinit, a check for the same version should download instead of showing ready."""
+        ctrl, _app, _client, _banner, _model = _make_controller(auto_apply=False)
+        ctrl._pending_version = '2.0.0'
+
+        new_config = _make_config(update_channel='dev')
+        with patch('synodic_client.application.update_controller.resolve_update_config') as mock_ucfg:
+            mock_ucfg.return_value = MagicMock(
+                auto_update_interval_minutes=0,
+                channel=MagicMock(name='DEVELOPMENT'),
+                repo_url='https://example.com',
+            )
+            ctrl._reinitialize_updater(new_config)
+
+        result = UpdateInfo(available=True, current_version=Version('1.0.0'), latest_version=Version('2.0.0'))
+        with patch.object(ctrl, '_start_download') as mock_dl:
+            ctrl._on_check_finished(result, silent=True)
+
+        mock_dl.assert_called_once_with('2.0.0', silent=True)
+
+
+# ---------------------------------------------------------------------------
+# Apply without pending download — guard
+# ---------------------------------------------------------------------------
+
+
+class TestApplyGuard:
+    """Verify _apply_update rejects apply when no download is pending."""
+
+    @staticmethod
+    def test_apply_without_pending_shows_error() -> None:
+        """Applying with no pending version should show an error, not crash."""
+        ctrl, app, client, banner, model = _make_controller()
+        ctrl._pending_version = None
+
+        ctrl._apply_update(silent=False)
+
+        assert banner.state.name == 'ERROR'
+        client.apply_update_on_exit.assert_not_called()
+        app.quit.assert_not_called()
+
+    @staticmethod
+    def test_apply_without_pending_silent_logs_only() -> None:
+        """Applying silently with no pending version should not show the banner."""
+        ctrl, app, client, banner, model = _make_controller()
+        ctrl._pending_version = None
+
+        ctrl._apply_update(silent=True)
+
+        assert banner.state.name == 'HIDDEN'
+        client.apply_update_on_exit.assert_not_called()
+        app.quit.assert_not_called()