From 487dbb8fd35ab29a73cb61ae007c0aa1c94dabbe Mon Sep 17 00:00:00 2001 From: migatu Date: Wed, 22 Jul 2026 17:29:27 +0200 Subject: [PATCH] fix(ci): smoke test silnika B bez kontenera w tle i bez sieci MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Job „Build obrazu silnika B" padal na kazdym przebiegu po pierwszym: Conflict. The container name "/swe" is already in use Dwie wady starego kroku, obie moje: 1. startowal kontener w tle (docker run -d --name swe) i NIGDY go nie usuwal, wiec nazwa zostawala zajeta na runnerze i kolejne przebiegi sie wywalaly; 2. pukal curl-em w localhost:8003, podczas gdy job Gitea Actions sam dziala w kontenerze, a -p publikuje port na HOSCIE — to nie ten sam localhost, wiec health-check i tak nie mial prawa dojsc. Teraz test biegnie WEWNATRZ obrazu (docker run --rm ... python -), wolajac funkcje endpointow wprost. Omija oba problemy, nie zostawia niczego po sobie, a sprawdza to samo i wiecej: obraz sie zbudowal, pyswisseph liczy, komplet 13 obiektow, Slonce w oczekiwanym zakresie, SN = NN + 180. Dodany krok sprzatajacy osierocony kontener „swe" ze starych przebiegow. Zweryfikowane lokalnie na realnym pyswisseph: Sun=40.2102, NN=68.1530, 13 obiektow — zgodnie z wyrocznia. Co-Authored-By: Claude Opus 4.8 --- .gitea/workflows/tests.yml | 56 ++++++++++++++++++++++++++++---------- 1 file changed, 41 insertions(+), 15 deletions(-) diff --git a/.gitea/workflows/tests.yml b/.gitea/workflows/tests.yml index 29d6d71..c1b358f 100644 --- a/.gitea/workflows/tests.yml +++ b/.gitea/workflows/tests.yml @@ -82,25 +82,51 @@ jobs: - name: docker build run: docker build -t astrololo/engine-swisseph:ci services/engine-swisseph - - name: Smoke test (health + pozycje) + # Test biegnie WEWNĄTRZ obrazu, bez sieci i bez kontenera w tle. Poprzednia + # wersja startowała kontener w tle (--name swe) i pukała curl-em w + # localhost:8003 — co miało dwie wady: + # 1. nie sprzątała kontenera, więc każdy kolejny przebieg padał na + # konflikcie nazwy (Conflict. The container name "/swe" is already in use), + # 2. job Gitea Actions sam działa w kontenerze, a -p publikuje port na + # HOŚCIE — więc localhost joba to nie ten sam localhost. + # Wywołanie funkcji endpointów wprost omija oba problemy, a sprawdza to samo: + # obraz się zbudował, pyswisseph liczy, kontrakt /positions się zgadza. + # --rm gwarantuje, że nic nie zostaje po przebiegu. + - name: Smoke test (health + pozycje) wewnątrz obrazu run: | - docker run -d --name swe -p 8003:8003 astrololo/engine-swisseph:ci - for i in $(seq 1 30); do - curl -fsS http://localhost:8003/health >/dev/null 2>&1 && break - sleep 1 - done - curl -fsS http://localhost:8003/health - echo + docker run --rm astrololo/engine-swisseph:ci python - <<'PY' + from datetime import datetime, timezone + + from app.main import DEFAULT_OBJECTS, PositionsRequest, health, positions + + h = health() + assert h["status"] == "ok", h + print("health:", h) + # Horoskop referencyjny (30.04.1984) — ten sam, na którym opieramy testy # silnika własnego; sprawdzamy, że silnik B faktycznie liczy. - curl -fsS -X POST http://localhost:8003/positions \ - -H 'Content-Type: application/json' \ - -d '{"when_utc":"1984-04-30T09:20:00Z","lat":50.0647,"lon":19.9450}' - echo + req = PositionsRequest(when_utc=datetime(1984, 4, 30, 9, 20, tzinfo=timezone.utc), + lat=50.0647, lon=19.9450) + out = positions(req) + by = {p["name"]: p for p in out["positions"]} - - name: Logi kontenera (gdy coś padło) - if: failure() - run: docker logs swe || true + assert out["engine"] == "swisseph", out["engine"] + assert len(by) == len(DEFAULT_OBJECTS), sorted(by) + sun = by["Sun"]["longitude"] + assert 39.5 < sun < 41.0, f"Slonce poza oczekiwanym zakresem: {sun}" + nn, sn = by["North Node"]["longitude"], by["South Node"]["longitude"] + assert abs(((sn - nn) % 360.0) - 180.0) < 1e-6, (nn, sn) + + print(f"Sun={sun:.4f} NN={nn:.4f} obiektow={len(by)}") + print("SMOKE OK") + PY + + # Sprzątanie po POPRZEDNICH przebiegach starej wersji workflow, która + # zostawiała kontener „swe" na runnerze i blokowała nazwę. Nowa wersja + # kontenera w tle nie tworzy, więc to tylko jednorazowe uprzątnięcie. + - name: Usuń osierocony kontener ze starych przebiegów + if: always() + run: docker rm -f swe 2>/dev/null || true compile-all: name: Kontrola składni wszystkich warstw -- 2.52.0