gitea
fd79513ce2
Testy / Testy warstwy logicznej (silnik) (pull_request) Failing after 4s
Testy / Testy warstwy prezentacji (dostęp do baz) (pull_request) Failing after 3s
Testy / Testy warstwy bazodanowej (ochrona baz) (pull_request) Failing after 3s
Testy / Testy astrodemo (pull_request) Failing after 3s
Testy / Build obrazu silnika B (swisseph) (pull_request) Failing after 2s
Testy / Kontrola składni wszystkich warstw (pull_request) Failing after 3s
Testy / Testy warstwy logicznej (silnik) (push) Failing after 4s
Testy / Testy warstwy prezentacji (dostęp do baz) (push) Failing after 3s
Testy / Testy warstwy bazodanowej (ochrona baz) (push) Failing after 3s
Testy / Testy astrodemo (push) Failing after 3s
Testy / Build obrazu silnika B (swisseph) (push) Failing after 2s
Testy / Kontrola składni wszystkich warstw (push) Failing after 3s
build / build (push) Successful in 9s
Trzeci z pięciu kroków. Zmieniłem jego zakres wobec planu i warto wiedzieć dlaczego: pierwotnie miała to być deduplikacja pięciu kopii link_crypto.py, ale po zrobieniu PR 2 widać, że astroklienta blokuje co innego — ekrany, których nie ma mieć, siedzą wewnątrz jednego main.py. Deduplikacja kryptografii jest realnym długiem, ale niczego nie blokuje. PODZIAŁ. main.py (1023 linie) rozpadł się na podstawa.py (wspólne obiekty i pomocnicy), jedenaście modułów w app/ekrany/ i main.py, który jest już samym ZŁOŻENIEM: lista importów JEST definicją produktu. Podział zrobiony mechanicznie, z osobnym sprawdzeniem, że żadna sekcja nie wołała pomocnika z innej (nie wołała). KATALOG Z REJESTRACJI. Dotąd wszystkie funkcje były wypisane w features.py, więc obraz produktu, który części z nich nie ma, i tak niósł ich nazwy — spis funkcji, których nie ma jak włączyć. Teraz ekran zgłasza siebie, swoje trasy i swoje zasoby przy imporcie własnego modułu, a features.py nie wymienia ani jednego ekranu. Kolejność w nawigacji jest jawna (`kolejnosc`), żeby nie rządziła nią kolejność importów. To samo dotyczy nawigacji administratora: odsyłacz do ekranu kont był wpisany na sztywno w base.html, więc w węższym produkcie zostawał martwy link i nazwa ekranu, którego nie ma. EKSPORT JAKO MODUŁ. Zgodnie z ustaleniem eksport jest funkcją administracyjną, więc musi dać się usunąć. app/moduly/eksport/ zabiera arkusz, trasę PDF-a i akcję formularza. „Można, ale nie temu kontu" i „nie ma takiej możliwości" to dwie różne gwarancje, a eksport wynosi najwięcej treści baz naraz. MOST ODKRYWA MODUŁY. Skoro modułów jest więcej niż jeden, most nie może ich znać z nazwy — nazwa nieobecnego modułu jechałaby do obrazu, w którym go nie ma. Przechodzi więc po podkatalogach app/moduly/ i pyta każdy, co wnosi. Katalog generowania przeniesiony z app/dodatki na app/moduly/dodatki. ZNALEZIONE PRZY OKAZJI. Po wydzieleniu eksportu okazało się, że jego ścieżki SZCZĘŚLIWEJ nie sprawdzał żaden test — badano wyłącznie odmowę dla konta bez uprawnienia. Moduł dostaje zależności z wywołania montującego, więc brak jednej z nich wyszedłby dopiero przy pierwszym kliknięciu. Dopisany test funkcjonalny (realny arkusz, sprawdzany aż do nagłówka ZIP-a) i brakujące zależności. Cztery komentarze w plikach współdzielonych wymieniały zakładkę „Skompiluj", w tym wheelzoom.js ze wzmianką o „przyszłej zakładce" — to samo zgłoszenie, które audyt podnosił wcześniej. TEST ZŁOŻENIA. Buduje węższy produkt NAPRAWDĘ: kopiuje drzewo, usuwa cztery ekrany i oba moduły, uruchamia aplikację w OSOBNYM PROCESIE (importy są zapamiętywane, więc sprawdzanie tego w procesie, który moduł już zaimportował, dawałoby wynik fałszywie pozytywny) i sprawdza, że wstaje, że zachowane ekrany oddają 200, że usunięte oddają 404 (nie 403 i nie 500), że katalog opisuje ten obraz, i że w nawigacji nie ma martwych odsyłaczy. Napisałem najpierw ostrzejszy test — „nazwa ekranu nie pada poza jego modułem" — i go wyrzuciłem: zgłaszał wzmianki o Horoskopie w plikach współdzielonych, choć astroklient Horoskop MA. Ślad ma znaczenie wyłącznie wobec konkretnego złożenia, więc sprawdzenie należy do produktu, nie do mechanizmu. Testy: presentation 368, logic 342, data 37, render 41. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
96 lines
3.6 KiB
Python
96 lines
3.6 KiB
Python
"""Zapamiętane predykcje okresowe (PRE-22).
|
|
|
|
Magazyn żyje w JS (localStorage), więc znów pilnujemy go strukturalnie. Sedno:
|
|
predykcje mają PRZEŻYĆ przejście na inną zakładkę i trafić do raportu, a lista
|
|
nie może puchnąć od powtórzonych prób na tym samym okresie.
|
|
"""
|
|
import pathlib
|
|
|
|
APP = pathlib.Path(__file__).resolve().parents[1] / "app"
|
|
# Skrypty generowania mieszkają w module `app/ai/` — katalogu, którego
|
|
# w obrazie bez tej funkcji po prostu nie ma.
|
|
JS = (APP / "moduly" / "dodatki" / "static" / "predictions.js").read_text(encoding="utf-8")
|
|
PROGRESS = (APP / "moduly" / "dodatki" / "static" / "progress.js").read_text(encoding="utf-8")
|
|
from tests.skladanie import zloz
|
|
|
|
TIMELINE = zloz("timeline.html")
|
|
|
|
|
|
# ─────────────────────────────── podpięcie ───────────────────────────────
|
|
|
|
def test_script_loaded_on_calendar_tab():
|
|
assert "predictions.js" in TIMELINE
|
|
|
|
|
|
def test_loaded_after_progress_script():
|
|
"""Nasłuchuje zdarzenia, które wysyła progress.js — musi być załadowany po nim,
|
|
inaczej przegapi pierwsze wywołanie."""
|
|
assert TIMELINE.index("progress.js") < TIMELINE.index("predictions.js")
|
|
|
|
|
|
def test_calendar_has_container_for_the_list():
|
|
assert 'id="savedPredictions"' in TIMELINE
|
|
|
|
|
|
def test_progress_announces_finished_horoscope():
|
|
"""Okno postępu ogłasza gotowy horoskop zdarzeniem, zamiast samo go zapisywać
|
|
— to okno postępu, a nie magazyn."""
|
|
assert "astrololo:horoscope" in PROGRESS
|
|
assert "profile" in PROGRESS.split("astrololo:horoscope")[1][:200]
|
|
|
|
|
|
# ─────────────────────────────── zachowanie ──────────────────────────────
|
|
|
|
def test_only_period_horoscopes_are_stored():
|
|
"""Interpretacja natalna ma NIE trafiać na listę predykcji okresowych."""
|
|
assert "'period'" in JS
|
|
|
|
|
|
def test_same_period_replaces_instead_of_duplicating():
|
|
"""Ponowne policzenie tego samego zakresu podmienia wpis. Inaczej lista
|
|
puchłaby przy każdej próbie z innym modelem."""
|
|
assert "function upsert" in JS
|
|
assert "p.from_date === entry.from_date" in JS
|
|
assert "p.to_date === entry.to_date" in JS
|
|
|
|
|
|
def test_entry_carries_period_and_text():
|
|
for field in ("from_date", "to_date", "text", "saved_at"):
|
|
assert field in JS, f"wpis nie niesie pola {field}"
|
|
|
|
|
|
def test_incomplete_result_is_not_stored():
|
|
"""Bez tekstu albo bez dat nie ma czego zapamiętać — nie zapisujemy śmieci."""
|
|
assert "if (!text || !from || !to) return null;" in JS
|
|
|
|
|
|
def test_list_can_be_pruned():
|
|
assert "function remove" in JS
|
|
assert "data-drop" in JS
|
|
|
|
|
|
def test_full_storage_is_reported_not_swallowed():
|
|
"""Horoskopy bywają długie. Gdy magazyn się przepełni, użytkownik musi się
|
|
o tym dowiedzieć, zamiast po cichu stracić wynik."""
|
|
assert "pełny" in JS or "pelny" in JS
|
|
assert "return false;" in JS
|
|
|
|
|
|
def test_corrupt_storage_does_not_break_the_page():
|
|
assert "catch" in JS and "Array.isArray" in JS
|
|
|
|
|
|
def test_storage_is_exposed_for_the_compile_tab():
|
|
"""Zakładka „Skompiluj" (PRE-23) ma czytać to samo miejsce — jedno źródło prawdy."""
|
|
assert "window.astrololoPredictions" in JS
|
|
|
|
|
|
def test_does_nothing_on_other_tabs():
|
|
assert "if (!document.getElementById('savedPredictions')) return;" in JS
|
|
|
|
|
|
def test_reload_does_not_duplicate():
|
|
"""Wariant bez strumienia zapisuje przy wczytaniu strony — klucz to okres,
|
|
więc odświeżenie nie mnoży wpisów."""
|
|
assert "id: from + '..' + to" in JS
|