fix healthcheck bug (#247)

Author: hubertdeng123

devservices/commands/up.py

@@ -225,7 +225,7 @@ def _up(
     for cmd in up_commands:
         try:
             container_names = get_container_names_for_project(
-                cmd.project_name, cmd.config_path
+                cmd.project_name, cmd.config_path, cmd.services
             )
             containers_to_check.extend(container_names)
         except DockerComposeError as dce:

devservices/utils/docker_compose.py

@@ -95,7 +95,7 @@ def install_docker_compose() -> None:
 
 
 def get_container_names_for_project(
-    project_name: str, config_path: str
+    project_name: str, config_path: str, services: list[str]
 ) -> list[ContainerNames]:
     try:
         output = subprocess.check_output(
@@ -107,6 +107,9 @@ def get_container_names_for_project(
                 "-f",
                 config_path,
                 "ps",
+            ]
+            + sorted(services)
+            + [
                 "--format",
                 '{"name":"{{.Names}}", "short_name":"{{.Service}}"}',
             ],

tests/utils/test_docker_compose.py

@@ -775,7 +775,9 @@ def test_get_all_commands_to_run_complex_shared_dependency(
 @mock.patch("devservices.utils.docker_compose.subprocess.check_output")
 def test_get_container_names_for_project_success(_mock_check_output: mock.Mock) -> None:
     _mock_check_output.return_value = '{"name": "devservices-container1", "short_name": "container1"}\n{"name": "devservices-container2", "short_name": "container2"}'
-    assert get_container_names_for_project("project", "config_path") == [
+    assert get_container_names_for_project(
+        "project", "config_path", ["container1", "container2"]
+    ) == [
         ContainerNames(name="devservices-container1", short_name="container1"),
         ContainerNames(name="devservices-container2", short_name="container2"),
     ]
@@ -787,5 +789,7 @@ def test_get_container_names_for_project_error(_mock_check_output: mock.Mock) ->
         returncode=1, cmd="docker compose ps --format", stderr="command failed"
     )
     with pytest.raises(DockerComposeError) as e:
-        get_container_names_for_project("project", "config_path")
+        get_container_names_for_project(
+            "project", "config_path", ["container1", "container2"]
+        )
     assert e.value.stderr == "command failed"