8b6ecc727d
Testy / Testy warstwy logicznej (silnik) (pull_request) Successful in 10m44s
Testy / Testy warstwy prezentacji (dostęp do baz) (pull_request) Successful in 9m28s
Testy / Testy warstwy bazodanowej (ochrona baz) (pull_request) Successful in 9m28s
Testy / Build obrazu silnika B (swisseph) (pull_request) Successful in 20s
Testy / Kontrola składni wszystkich warstw (pull_request) Successful in 8s
build / build (push) Successful in 23s
Testy / Testy warstwy logicznej (silnik) (push) Successful in 10m38s
Testy / Testy warstwy prezentacji (dostęp do baz) (push) Successful in 9m28s
Testy / Testy warstwy bazodanowej (ochrona baz) (push) Successful in 9m28s
Testy / Build obrazu silnika B (swisseph) (push) Successful in 17s
Testy / Kontrola składni wszystkich warstw (push) Successful in 9s
ETAP 5, część pierwsza: assign_house pod wyrocznią (swe_house_pos). Osobny rodzaj błędu niż same cuspy — i od razu jeden znalazł. BŁĄD: assign_house szło ZAWSZE do przodu. Przy dużych szerokościach systemy dzielące koła wielkie mają kolejność domów ODWRÓCONĄ (przy φ=−84,3° cusp domu I wypada na 174,2°, a domu II na 165,3°) — co nie jest usterką, bo wyrocznia zwraca dokładnie te same wartości. Suma przeskoków „do przodu" wychodziła 3960° zamiast 360°, czyli każdy krok obchodził koło dookoła. Planety lądowały w złych domach dla regiomontanusa, campanusa i topocentrica: 9,7-12,5% przypadków. Błąd cichy — wykres wyglądał bez zarzutu. Kierunek bierzemy teraz z samych cuspów. TOPOCENTRIC MA JEDNAK GRANICĘ DZIEDZINY — korekta tego, co pisałem wcześniej. Cuspy są poprawne wszędzie (zgodne z wyrocznią co do zera), ale powyżej koła podbiegunowego przestają DZIELIĆ OKRĄG: cusp VII (= I + 180°) wypada przed cuspem VI i domy nachodzą na siebie. Przypisanie planety traci wtedy sens — co potwierdza sama wyrocznia, której swe_house_pos przeczy tam własnym cuspom (100% zgodności do 62°, 83,9% przy 66°, ok. 50% przy 72°; regiomontanus 100% w tych samych punktach). Odmawiamy, z jawnym fallbackiem jak Placidus i Koch. Próg jest WYPROWADZONY z warunku „dwanaście cuspów sumuje się do 360°", nie dobrany pod wynik testu — i wypada na kole podbiegunowym (zmierzone: 100% podziałów do 65°, 78% w pasie 66-67°). To inny rodzaj granicy niż u Placidusa i Kocha: tam nie istnieją same cuspy, tu istnieją, tylko nie tworzą podziału. Framework dostał pojęcie dziedziny WĘŻSZEJ niż wyroczni (NARROWER_THAN_ORACLE), zamiast wyjątku „bo topocentric": skoro wyrocznia przeczy sama sobie, nie może rozstrzygać, więc tam nie porównujemy — a nasze przypisanie jest w tym obszarze sprawdzane testem samospójności z cuspami, bez swissepha. UKŁAD STRONY — zmierzony na żywej stronie, nie na oko: - tabela porównania przy 13 systemach miała 14 kolumn i 1863 px, a stała w rodzicu bez overflow-x, więc ROZPYCHAŁA CAŁY DOKUMENT: 1713 px przy oknie 1280 px, poziomy pasek na body. Teraz ma własny kontener przewijany (dokument 1265 px, nie przewija się), a numer domu jest przyklejony do lewej, bo inaczej po przewinięciu nie wiadomo, który to wiersz. - przypis „* nie działa za kołem podbiegunowym" siedział WEWNĄTRZ <label> selektora, łamał się na dwie linie i rozciągał wiersz siatki ze 66 do 108 px, rozjeżdżając go z sąsiednim polem. Wyjaśnienie stoi teraz raz, przy checkboxach z gwiazdkami; wiersz wrócił do 66 px. - 13 checkboxów na flexie zawijało się w poszarpane wiersze — jest siatka o stałej szerokości kolumny (auto-fill, więc na wąskim ekranie kolumn mniej). Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
210 lines
9.5 KiB
Python
210 lines
9.5 KiB
Python
"""Porównanie systemów domów naraz (PRE-05).
|
|
|
|
JS-a tu nie ma — sprawdzamy strukturalnie: formularz pozwala zaznaczyć kilka
|
|
systemów, klient przekazuje je do logiki, a szablon pokazuje kuspy obok siebie.
|
|
"""
|
|
import pathlib
|
|
|
|
import pytest
|
|
|
|
APP = pathlib.Path(__file__).resolve().parents[1] / "app"
|
|
CHART = (APP / "templates" / "chart.html").read_text(encoding="utf-8")
|
|
COMPILE = (APP / "templates" / "compile.html").read_text(encoding="utf-8")
|
|
OPTS = (APP / "templates" / "_form_options.html").read_text(encoding="utf-8") # wspólne opcje
|
|
TABLES = (APP / "templates" / "_result_tables.html").read_text(encoding="utf-8") # wspólne tabele
|
|
MAIN = (APP / "main.py").read_text(encoding="utf-8")
|
|
|
|
from app.house_systems import HOUSE_SYSTEMS, LIMITED, label as house_label # noqa: E402
|
|
CLIENT = (APP / "clients" / "logic_client.py").read_text(encoding="utf-8")
|
|
|
|
|
|
def test_both_forms_use_shared_option_partial():
|
|
"""Opcje w jednym pliku — oba formularze go dołączają (koniec duplikacji)."""
|
|
assert '{% include "_form_options.html" %}' in CHART
|
|
assert '{% include "_form_options.html" %}' in COMPILE
|
|
|
|
|
|
def _render(name: str, **ctx) -> str:
|
|
"""Renderuje szablon z tymi samymi globalsami, co produkcja.
|
|
|
|
Szablony budują listę systemów PĘTLĄ po katalogu, więc sprawdzanie ich
|
|
surowego tekstu nic już nie mówi — trzeba je wykonać. Przy okazji test
|
|
łapie literówki w Jinja, których grep po źródle nigdy nie wyłapał."""
|
|
from jinja2 import Environment, FileSystemLoader
|
|
|
|
env = Environment(loader=FileSystemLoader(str(APP / "templates")), autoescape=True)
|
|
env.globals.update(HOUSE_SYSTEMS=HOUSE_SYSTEMS, HOUSE_LIMITED=LIMITED,
|
|
house_label=house_label, static=lambda p: p)
|
|
return env.get_template(name).render(**ctx)
|
|
|
|
|
|
def test_form_offers_every_system_the_engine_advertises():
|
|
"""Każdy system z katalogu ma checkbox. Katalog jest jednym źródłem nazw,
|
|
więc dopisanie systemu bez odświeżenia UI zapali się tutaj."""
|
|
html = _render("_form_options.html", form={})
|
|
for value, name in HOUSE_SYSTEMS:
|
|
assert f'name="house_systems" value="{value}"' in html, f"brak checkboxa {value}"
|
|
assert name in html, f"brak nazwy {name}"
|
|
|
|
|
|
def test_form_marks_systems_with_a_domain_limit():
|
|
"""Placidus i Koch nie istnieją za kołem podbiegunowym — UI ma uprzedzać
|
|
ZANIM ktoś policzy horoskop dla Tromsø, nie dopiero w ostrzeżeniu wyniku."""
|
|
html = _render("_form_options.html", form={})
|
|
assert LIMITED, "katalog nie oznacza żadnego systemu jako ograniczonego"
|
|
for value, name in HOUSE_SYSTEMS:
|
|
starred = f"{name} *" in html
|
|
assert starred == (value in LIMITED), f"{value}: gwiazdka={starred}"
|
|
|
|
|
|
def test_handler_accepts_and_passes_house_systems():
|
|
assert "house_systems: list[str] = Form([])" in MAIN
|
|
assert "house_systems=house_systems" in MAIN # przekazane do logiki
|
|
|
|
|
|
def test_client_forwards_house_systems_to_logic():
|
|
assert '"house_systems": house_systems' in CLIENT
|
|
|
|
|
|
def test_comparison_table_renders_only_for_multiple_systems():
|
|
"""Bez dodatkowych systemów nie pokazujemy pustej tabeli porównania."""
|
|
assert "result.house_systems | length > 1" in TABLES
|
|
assert "Porównanie systemów domów" in TABLES
|
|
|
|
|
|
def test_comparison_table_shows_cusp_per_system():
|
|
"""Tabela iteruje po systemach i po 12 domach — kusp w każdej komórce."""
|
|
assert "for hs in result.house_systems" in TABLES
|
|
assert "hs.cusps[i].in_sign" in TABLES
|
|
|
|
|
|
def test_checkbox_state_survives_submit():
|
|
"""Zaznaczone systemy zostają zaznaczone po przeliczeniu."""
|
|
html = _render("_form_options.html", form={"house_systems": ["koch", "campanus"]})
|
|
for value, _ in HOUSE_SYSTEMS:
|
|
marker = f'value="{value}" checked'
|
|
assert (marker in html) == (value in ("koch", "campanus")), value
|
|
|
|
|
|
def test_fallback_notice_is_shown_prominently():
|
|
"""Po fallbacku kosmogram wygląda bezbłędnie — ostrzeżenie jest jedynym
|
|
sygnałem, że podział domów jest z innego systemu."""
|
|
html = _render("_result_tables.html",
|
|
result={"house_warnings": ["UWAGA: system domów nie ma definicji"]})
|
|
assert "house-warning" in html
|
|
assert "UWAGA: system domów nie ma definicji" in html
|
|
|
|
|
|
def test_no_notice_when_nothing_was_substituted():
|
|
html = _render("_result_tables.html", result={"house_warnings": []})
|
|
assert "house-warning" not in html
|
|
|
|
|
|
# ── ścieżka PRZEZ handlery, nie obok nich ───────────────────────────────
|
|
# Dwa błędy przeszły przez komplet testów szablonowych, bo żaden nie wywołał
|
|
# POST-a: odwołanie do nieistniejącej zmiennej w handlerze strony głównej oraz
|
|
# użycie parametru formularza, którego w sygnaturze nie było. Oba dają 500 na
|
|
# żywo i oba łapie dopiero prawdziwe żądanie.
|
|
|
|
def _client_with_stub_logic(monkeypatch, chart):
|
|
import os
|
|
|
|
os.environ.pop("APP_PASSWORD", None)
|
|
from starlette.testclient import TestClient
|
|
|
|
from app.main import app, logic
|
|
|
|
monkeypatch.setattr(logic, "positions", lambda **kw: chart)
|
|
return TestClient(app)
|
|
|
|
|
|
def _minimal_chart():
|
|
cusps = [{"house": i + 1, "sign": "Aries", "in_sign": "0°00'00''",
|
|
"decimal": float(i * 30), "sign_glyph": "♈"} for i in range(12)]
|
|
ang = {k: {"name": k, "sign": "Aries", "in_sign": "0°00'00''", "decimal": 0.0,
|
|
"sign_glyph": "♈"} for k in ("Asc", "MC", "Dsc", "IC")}
|
|
return {"engine": "test", "positions": [], "cusps": cusps, "angles": ang,
|
|
"sign_glyphs": [{"sign": "Aries", "glyph": "♈"}],
|
|
"house_system": "porphyry", "house_system_requested": "placidus",
|
|
"house_warnings": ["UWAGA: system domów „placidus” nie ma definicji"]}
|
|
|
|
|
|
FORM = {"date": "1984-04-30", "time": "11:20", "tz_offset": "2",
|
|
"lat": "50.06", "lon": "19.94", "house_system": "placidus"}
|
|
|
|
|
|
def test_chart_page_survives_a_real_post(monkeypatch):
|
|
c = _client_with_stub_logic(monkeypatch, _minimal_chart())
|
|
r = c.post("/", data=FORM)
|
|
assert r.status_code == 200, r.text[:400]
|
|
assert "nie ma definicji" in r.text, "ostrzeżenie o fallbacku nie dotarło na stronę"
|
|
|
|
|
|
def test_compile_page_survives_a_real_post(monkeypatch):
|
|
c = _client_with_stub_logic(monkeypatch, _minimal_chart())
|
|
r = c.post("/compile", data=FORM)
|
|
assert r.status_code == 200, r.text[:400]
|
|
|
|
|
|
@pytest.mark.parametrize("orientation", ("asc", "aries"))
|
|
def test_wheel_orientation_reaches_the_renderer(monkeypatch, orientation):
|
|
"""Wybór obrotu koła musi dojść z formularza do rysunku — i wrócić
|
|
zaznaczony, żeby nie gubił się przy przeliczeniu."""
|
|
c = _client_with_stub_logic(monkeypatch, _minimal_chart())
|
|
r = c.post("/", data={**FORM, "wheel_orientation": orientation})
|
|
assert r.status_code == 200, r.text[:400]
|
|
assert f'value="{orientation}" checked' in r.text
|
|
|
|
|
|
# ── układ strony przy komplecie systemów ────────────────────────────────
|
|
# Regresja z pomiaru na żywej stronie: przy 13 zaznaczonych systemach tabela
|
|
# porównania miała 14 kolumn i 1863 px szerokości, a że stała w rodzicu bez
|
|
# overflow-x, ROZPYCHAŁA CAŁY DOKUMENT — 1713 px przy oknie 1280 px, czyli
|
|
# poziomy pasek przewijania na body i rozjechany układ.
|
|
|
|
CSS = (APP / "static" / "styles.css").read_text(encoding="utf-8")
|
|
|
|
|
|
def test_comparison_table_scrolls_instead_of_widening_the_page():
|
|
"""Tabela ma własny kontener przewijany — przewija się ona, nie strona."""
|
|
html = _render("_result_tables.html", result={
|
|
"house_systems": [{"system": v, "cusps": [
|
|
{"sign_glyph": "♈", "in_sign": "0°00'00''"} for _ in range(12)]}
|
|
for v, _ in HOUSE_SYSTEMS],
|
|
"house_warnings": []})
|
|
assert '<div class="table-scroll">' in html
|
|
before = html.index('class="table-scroll"')
|
|
assert html.index("<table", before) < html.index("</div>", before), \
|
|
"tabela musi być WEWNĄTRZ kontenera przewijanego"
|
|
|
|
|
|
def test_scroll_container_actually_clips_and_pins_the_house_number():
|
|
"""Sam znacznik nie wystarczy — reguła musi istnieć w arkuszu.
|
|
|
|
Geometrii nie da się sprawdzić bez przeglądarki, więc sprawdzamy kontrakt:
|
|
kontener przewija się w poziomie, a pierwsza kolumna zostaje przyklejona
|
|
(bez tego po przewinięciu w prawo nie wiadomo, który to dom)."""
|
|
assert ".table-scroll { overflow-x: auto;" in CSS
|
|
assert ".table-scroll th:first-child, .table-scroll td:first-child {" in CSS
|
|
assert "position: sticky" in CSS
|
|
|
|
|
|
def test_system_checkboxes_use_a_grid_not_a_ragged_flex_wrap():
|
|
"""13 pozycji na flexie zawijało się w poszarpane wiersze o różnej długości."""
|
|
html = _render("_form_options.html", form={})
|
|
assert 'class="opts systems"' in html
|
|
assert ".opts.systems { display: grid;" in CSS
|
|
|
|
|
|
def test_polar_circle_note_is_explained_once_next_to_the_asterisks():
|
|
"""Przypis siedział WEWNĄTRZ <label> selektora i łamał się na dwie linie,
|
|
przez co wiersz siatki rósł z 66 do 108 px i rozjeżdżał się z sąsiednim
|
|
polem. Wyjaśnienie stoi teraz raz, przy checkboxach z gwiazdkami."""
|
|
opts = _render("_form_options.html", form={})
|
|
assert "kołem podbiegunowym" in opts
|
|
for name in ("chart.html", "compile.html"):
|
|
tpl = (APP / "templates" / name).read_text(encoding="utf-8")
|
|
select = tpl[tpl.index("<select name=\"house_system\""):]
|
|
assert "podbiegunow" not in select[:select.index("</label>")], \
|
|
f"{name}: przypis wrócił do etykiety selektora"
|