refactor: address review nits in health_check_timeout PR

- Remove unused mock_docker_workspace fixture parameter from test
- Fix execute_command mock scope to be active during resume()
- Shorten stale comment in resume() to match _start_container

Co-authored-by: openhands <openhands@all-hands.dev>

Author: openhands-agent

openhands-workspace/openhands/workspace/docker/workspace.py

@@ -422,6 +422,6 @@ def resume(self) -> None:
         if result.returncode != 0:
             raise RuntimeError(f"Failed to resume container: {result.stderr}")
 
-        # Wait for health after resuming (use same timeout as initial startup)
+        # Wait for container to be healthy
         self._wait_for_health(timeout=self.health_check_timeout)
         logger.info(f"Container resumed: {self._container_id}")

tests/workspace/test_docker_workspace.py

@@ -226,7 +226,7 @@ def test_docker_workspace_startup_uses_health_check_timeout():
         mock_wait.assert_called_once_with(timeout=60.0)
 
 
-def test_docker_workspace_resume_uses_health_check_timeout(mock_docker_workspace):
+def test_docker_workspace_resume_uses_health_check_timeout():
     """Test that resume() passes health_check_timeout to _wait_for_health."""
     with patch.object(DockerWorkspace, "_start_container"):
         with patch("openhands.workspace.docker.workspace.execute_command") as mock_exec:
@@ -237,7 +237,10 @@ def test_docker_workspace_resume_uses_health_check_timeout(mock_docker_workspace
 
     workspace._container_id = "container_id_123"
 
-    with patch.object(workspace, "_wait_for_health") as mock_wait:
+    with (
+        patch("openhands.workspace.docker.workspace.execute_command") as mock_exec,
+        patch.object(workspace, "_wait_for_health") as mock_wait,
+    ):
         mock_exec.return_value = Mock(returncode=0, stdout="", stderr="")
         workspace.resume()
         mock_wait.assert_called_once_with(timeout=30.0)