From 3313576a6304ea925ed69fcd5cba2e17bd2e3c94 Mon Sep 17 00:00:00 2001 From: Phil Ruff Date: Wed, 19 Aug 2026 17:49:00 +0100 Subject: [PATCH] fix(tests): update container startup tests for preiweb service rename MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit test_container_startup.py still looked up docker-compose.yml's web service by its old key ("web"), which became "preiweb" in 7253ef4 (rename compose service web to preiweb for observability clarity). The compose file's actual config was never wrong — the test's dict lookups just returned {} for everything, failing 3 assertions on every PR's Integration Tests job regardless of what the PR touched. --- tests/test_container_startup.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/test_container_startup.py b/tests/test_container_startup.py index 36d98b6a..a77b67bd 100644 --- a/tests/test_container_startup.py +++ b/tests/test_container_startup.py @@ -91,7 +91,7 @@ def test_render_blueprint_excludes_celery_redis_config() -> None: def test_compose_healthcheck_allows_longer_prestart_migrations() -> None: """Verify Compose grants extra startup time before healthcheck failures.""" compose = _load_yaml("docker-compose.yml") - web = compose.get("services", {}).get("web", {}) + web = compose.get("services", {}).get("preiweb", {}) hc = web.get("healthcheck", {}) assert hc.get("start_period") == "90s", ( @@ -105,7 +105,7 @@ def test_compose_healthcheck_allows_longer_prestart_migrations() -> None: def test_compose_web_service_has_required_config() -> None: """Verify essential web-service configuration keys exist.""" compose = _load_yaml("docker-compose.yml") - web = compose.get("services", {}).get("web", {}) + web = compose.get("services", {}).get("preiweb", {}) assert web.get("image") == "ghcr.io/paruff/prei:latest" assert web.get("env_file") == ".env" @@ -120,7 +120,7 @@ def test_compose_web_service_has_required_config() -> None: def test_compose_runtime_env_vars_set() -> None: """Verify runtime environment overrides exist in the web service.""" compose = _load_yaml("docker-compose.yml") - env = compose.get("services", {}).get("web", {}).get("environment", {}) + env = compose.get("services", {}).get("preiweb", {}).get("environment", {}) # May be a dict or list of KEY=value strings env_str = str(env)