From 0e3c655d27f93530aa4cffdb21d7dea9d988fd86 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alejandro=20Gonz=C3=A1lez?= <7822554+AlexTMjugador@users.noreply.github.com> Date: Sat, 15 Nov 2025 14:47:03 +0100 Subject: [PATCH] fix(tests): ensure Nginx is fully spun up before starting MC container (#3763) --- .../generic-packs/docker-compose.yml | 8 +++++++- .../icon-gif-multiframe/docker-compose.yml | 9 ++++++++- .../icon-png-atscale/docker-compose.yml | 14 +++++++++++++- .../icon-png-scale/docker-compose.yml | 14 +++++++++++++- tests/setuponlytests/packwiz/docker-compose.yml | 8 +++++++- 5 files changed, 48 insertions(+), 5 deletions(-) diff --git a/tests/setuponlytests/generic-packs/docker-compose.yml b/tests/setuponlytests/generic-packs/docker-compose.yml index 52a3c9cf..2e6bd70a 100644 --- a/tests/setuponlytests/generic-packs/docker-compose.yml +++ b/tests/setuponlytests/generic-packs/docker-compose.yml @@ -3,9 +3,15 @@ services: image: nginx volumes: - ./web:/usr/share/nginx/html + healthcheck: + test: ["CMD", "curl", "--fail", "http://localhost/configs.zip"] + interval: 3s + timeout: 5s + retries: 3 mc: depends_on: - - web + web: + condition: service_healthy image: ${IMAGE_TO_TEST:-itzg/minecraft-server} environment: EULA: "true" diff --git a/tests/setuponlytests/icon-gif-multiframe/docker-compose.yml b/tests/setuponlytests/icon-gif-multiframe/docker-compose.yml index be0d5758..590b18f1 100644 --- a/tests/setuponlytests/icon-gif-multiframe/docker-compose.yml +++ b/tests/setuponlytests/icon-gif-multiframe/docker-compose.yml @@ -3,9 +3,16 @@ services: image: nginx volumes: - ./web:/usr/share/nginx/html + healthcheck: + test: + ["CMD", "curl", "--fail", "http://localhost/motion-tween-example.gif"] + interval: 3s + timeout: 5s + retries: 3 mc: depends_on: - - web + web: + condition: service_healthy image: ${IMAGE_TO_TEST:-itzg/minecraft-server} environment: EULA: "true" diff --git a/tests/setuponlytests/icon-png-atscale/docker-compose.yml b/tests/setuponlytests/icon-png-atscale/docker-compose.yml index 7ac20b6d..b179d581 100644 --- a/tests/setuponlytests/icon-png-atscale/docker-compose.yml +++ b/tests/setuponlytests/icon-png-atscale/docker-compose.yml @@ -3,9 +3,21 @@ services: image: nginx volumes: - ./web:/usr/share/nginx/html + healthcheck: + test: + [ + "CMD", + "curl", + "--fail", + "http://localhost/4737386_minecraft_squircle_icon.png", + ] + interval: 3s + timeout: 5s + retries: 3 mc: depends_on: - - web + web: + condition: service_healthy image: ${IMAGE_TO_TEST:-itzg/minecraft-server} environment: EULA: "true" diff --git a/tests/setuponlytests/icon-png-scale/docker-compose.yml b/tests/setuponlytests/icon-png-scale/docker-compose.yml index 7ac20b6d..b179d581 100644 --- a/tests/setuponlytests/icon-png-scale/docker-compose.yml +++ b/tests/setuponlytests/icon-png-scale/docker-compose.yml @@ -3,9 +3,21 @@ services: image: nginx volumes: - ./web:/usr/share/nginx/html + healthcheck: + test: + [ + "CMD", + "curl", + "--fail", + "http://localhost/4737386_minecraft_squircle_icon.png", + ] + interval: 3s + timeout: 5s + retries: 3 mc: depends_on: - - web + web: + condition: service_healthy image: ${IMAGE_TO_TEST:-itzg/minecraft-server} environment: EULA: "true" diff --git a/tests/setuponlytests/packwiz/docker-compose.yml b/tests/setuponlytests/packwiz/docker-compose.yml index d4ff5016..63a85b80 100644 --- a/tests/setuponlytests/packwiz/docker-compose.yml +++ b/tests/setuponlytests/packwiz/docker-compose.yml @@ -3,9 +3,15 @@ services: image: nginx volumes: - ./web:/usr/share/nginx/html + healthcheck: + test: ["CMD", "curl", "--fail", "http://localhost/pack.toml"] + interval: 3s + timeout: 5s + retries: 3 mc: depends_on: - - web + web: + condition: service_healthy image: ${IMAGE_TO_TEST:-itzg/minecraft-server} environment: EULA: "true"