smanager 0.2.1__tar.gz → 0.2.2__tar.gz

{smanager-0.2.1 → smanager-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: smanager
-Version: 0.2.1
+Version: 0.2.2
 Requires-Python: >=3.9
 Requires-Dist: PyYAML>=6.0
 Requires-Dist: click>=8.0

{smanager-0.2.1 → smanager-0.2.2}/smanager/__init__.py

@@ -1,4 +1,4 @@
-__version__ = "0.2.1"
+__version__ = "0.2.2"
 
 from .global_manager import GlobalManager
 from .datatypes import GlobalConfig, Service, ServiceStatus, MultiStatus

{smanager-0.2.1 → smanager-0.2.2}/smanager/service_managers/local_manager.py

@@ -2,6 +2,7 @@ from . import BaseSDV2Manager
 from ..datatypes import Service
 from ..vpn import VPNService
 import time
+import subprocess
 from ..datatypes import ServiceStatus, SingleStatus
 
 class LocalManager(BaseSDV2Manager):
@@ -105,10 +106,18 @@ class LocalManager(BaseSDV2Manager):
         """Check the status of the local systemd service and return a ServiceStatus object."""
         service_statuses = []
         for service_name in self._get_systemd_services():
-            result = self.subprocess_run(["systemctl", "is-active", service_name])
+            result = subprocess.run(
+                ["systemctl", "is-active", service_name],
+                capture_output=True,
+                text=True,
+                check=False,
+            )
+            status_text = result.stdout.strip() or result.stderr.strip()
+            if not status_text:
+                status_text = "active" if result.returncode == 0 else "inactive"
             status = SingleStatus(
                 name=service_name,
-                status=result.stdout.strip()
+                status=status_text
             )
             service_statuses.append(status)
         return ServiceStatus(

{smanager-0.2.1 → smanager-0.2.2}/smanager.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: smanager
-Version: 0.2.1
+Version: 0.2.2
 Requires-Python: >=3.9
 Requires-Dist: PyYAML>=6.0
 Requires-Dist: click>=8.0

{smanager-0.2.1 → smanager-0.2.2}/tests/test_service_manager_events.py

@@ -4,7 +4,7 @@ from pathlib import Path
 from tempfile import TemporaryDirectory
 from types import SimpleNamespace
 import unittest
-from unittest.mock import Mock
+from unittest.mock import Mock, patch
 
 from smanager.datatypes import (
     DockerSection,
@@ -161,3 +161,18 @@ class LocalManagerEventTests(unittest.TestCase):
             manager.stop()
 
             self.assertEqual(events, ["service.stop.before", "service.stop.after"])
+
+    def test_status_returns_inactive_when_systemctl_reports_inactive(self):
+        with TemporaryDirectory() as tmpdir:
+            service_dir = Path(tmpdir) / "local"
+            service_dir.mkdir()
+            service = make_local_service(service_dir, active=True)
+
+            manager = LocalManager(service, DummyVPN())
+
+            fake_result = SimpleNamespace(stdout="inactive\n", stderr="", returncode=4)
+            with patch("smanager.service_managers.local_manager.subprocess.run", return_value=fake_result) as run_mock:
+                result = manager.status()
+
+            self.assertEqual(result.statuses[0].status, "inactive")
+            run_mock.assert_called_once()