test: fix failing tests

Signed-off-by: Damian Skrzyński <polprog.tech@gmail.com>

Author: polprog-dev

src/opsportal/adapters/releaseboard.py

@@ -197,7 +197,7 @@ async def get_status(self) -> ToolStatus:
         proc = self._pm.get(self._process_name)
         if proc and proc.status == ProcessStatus.RUNNING:
             return ToolStatus.RUNNING
-        if not shutil.which(self._cli):
+        if proc and proc.status == ProcessStatus.FAILED:
             return ToolStatus.ERROR
         return ToolStatus.STOPPED
 

src/opsportal/adapters/releasepilot.py

@@ -193,7 +193,7 @@ async def get_status(self) -> ToolStatus:
         proc = self._pm.get(self._process_name)
         if proc and proc.status == ProcessStatus.RUNNING:
             return ToolStatus.RUNNING
-        if not shutil.which(self._cli):
+        if proc and proc.status == ProcessStatus.FAILED:
             return ToolStatus.ERROR
         return ToolStatus.STOPPED
 

tests/test_ui_regression.py

@@ -259,4 +259,4 @@ def test_files_under_limit(self, subdir: Path, glob_pattern: str) -> None:
                 violations.append(f"{f.name}: {lines} lines")
 
         """THEN no files exceed the limit."""
-        assert not violations, f"Files exceeding {self.MAX_LINES} lines: " + "; ".join(violations)
+        assert not violations, f"Files exceeding {self.MAX_LINES} lines: " + "; ".join(violations)