fix: move disk space check parsing inside try-except block

The parsing code for df output was outside the try-except block, causing
uncaught exceptions to bubble up and return UNKNOWN status instead of
HEALTHY or CRITICAL. Moving the parsing logic inside the try-except block
ensures proper error handling and correct status reporting.

Fixes test failures:
- TestDiskSpaceCheck.test_disk_space_check_run
- TestDiskSpaceCheck.test_disk_space_high_requirement

Author: vietcgi

cli/health_check.py

@@ -161,21 +161,20 @@ def run(self) -> tuple[str, str, dict[str, Any]]:
         """Check available disk space."""
         try:
             result = run_command(["df", "-B1G", "/"], timeout=2, check=True)
+            lines = result.stdout.strip().split("\n")
+            if len(lines) < 2:
+                return (HealthStatus.UNKNOWN, "Could not parse disk space", {})
+
+            # Parse df output
+            parts = lines[1].split()
+            available_gb = int(parts[3].rstrip("G"))
         except (subprocess.SubprocessError, ValueError, IndexError, OSError) as e:
             return (
                 HealthStatus.UNKNOWN,
                 f"Failed to check disk space: {e}",
                 {"error": str(e)},
             )
 
-        lines = result.stdout.strip().split("\n")
-        if len(lines) < 2:
-            return (HealthStatus.UNKNOWN, "Could not parse disk space", {})
-
-        # Parse df output
-        parts = lines[1].split()
-        available_gb = int(parts[3].rstrip("G"))
-
         if available_gb >= self.min_gb:
             return (
                 HealthStatus.HEALTHY,