update comments

Author: keivenchang

tests/conftest.py

@@ -646,7 +646,7 @@ def num_system_ports(request) -> int:
     Default: 1 port.
 
     Tests that need multiple system ports (e.g. SYSTEM_PORT1 + SYSTEM_PORT2) must
-    explicitly request them via indirect parametrization, e.g.:
+    explicitly request them via indirect parametrization:
       @pytest.mark.parametrize("num_system_ports", [2], indirect=True)
     """
     return getattr(request, "param", 1)

tests/utils/managed_process.py

@@ -600,15 +600,9 @@ def __init__(
     ):
         # TODO: Refactor remaining duplicate "DynamoFrontendProcess" helpers in tests to
         # use this shared implementation (and delete the copies):
-        # - tests/frontend/test_vllm.py
-        # - tests/router/common.py
-        # - tests/router/test_router_e2e_with_vllm.py
-        # - tests/router/test_router_e2e_with_sglang.py
-        # - tests/router/test_router_e2e_with_trtllm.py
         # - tests/fault_tolerance/cancellation/utils.py
         # - tests/fault_tolerance/migration/utils.py
         # - tests/fault_tolerance/etcd_ha/utils.py
-        # - tests/fault_tolerance/test_vllm_health_check.py
         self._allocated_http_port: Optional[int] = None
         if frontend_port == 0:
             # Treat `0` as "allocate a random free port" for xdist-safe tests.