Hausmasse und Oeffnungsfrist bekommen eine Oberflaeche
Die Endpunkte standen seit dem Backend-Commit, aber eintragen liess sich
nichts - die Mengen gingen nur per API rein. Damit war "2 EL Mehl" im
Verbrauchen-Dialog zwar vorgesehen, aber fuer niemanden erreichbar.
Zwei Stellen dafuer:
Stammdaten > Masse verwaltet nur die NAMEN ("Essloeffel", "Tasse"). Die
Seite ist absichtlich kurz, denn die Mengen gehoeren nicht an das Mass:
Ein Essloeffel Mehl sind etwa 10 g, einer Zucker etwa 12 g, und eine
Tasse Milch ist Milliliter. Ein globaler Wert waere bei jedem zweiten
Artikel falsch.
Kategorien > "Masse & Frist" ist deshalb der Ort, an dem die Arbeit
passiert. Beides steckt in einem Bedienfeld, weil es dieselbe Mechanik
ist: Der Wert gilt fuer die Kategorie und alles darunter, und der
naechste Vorfahr mit einem Eintrag gewinnt. Leere Felder erben - und
zeigen als Platzhalter, WAS sie erben ("erbt 5 Tage (Kategorie: Mehl &
Zucker)", "10 g" mit Abzeichen "geerbt"). Ohne das muesste man den
Kategorienbaum im Kopf verfolgen, um eine leere Eingabe zu verstehen.
Dafuer zwei kleine Ergaenzungen am Backend: CategoryOut nennt die
geerbte Frist samt Herkunft, und /categories/{id}/measures nimmt
?vererbt=true, um neben den eigenen auch die von oben geerbten Mengen zu
liefern. Die Bearbeitungsansicht ohne Flagge bleibt wie sie war.
Von Hand durchgeklickt - und dabei einen echten Fehler gefunden: Der
API-Aufruf im Browser hat das vererbt-Flag verschluckt, weil die
Funktion nur ein Argument annahm. Die Unterkategorie zeigte dadurch
leere Felder, obwohl der Server die geerbten Werte lieferte. Jetzt
stehen dort "10 g" und "3 g" mit der Quelle im Tooltip.
4 neue Tests, Suite bei 516.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
795e233a1b
commit
f61c76034c
@@ -15,10 +15,15 @@ from app.models import (
|
||||
Product,
|
||||
ProductMeasure,
|
||||
)
|
||||
from app.routers.measures import category_measures
|
||||
from app.seed import ensure_builtin_measures
|
||||
from app.services.conversion import ConversionError, to_base
|
||||
from app.services.stock import check_in, check_out, current_stock
|
||||
from app.services.vererbung import mass_aufloesen, verfuegbare_masse
|
||||
from app.services.vererbung import (
|
||||
geerbte_kategorie_frist,
|
||||
mass_aufloesen,
|
||||
verfuegbare_masse,
|
||||
)
|
||||
|
||||
|
||||
@pytest.fixture()
|
||||
@@ -253,3 +258,54 @@ def test_artikel_ohne_kategorie_hat_keine_masse(db, masse):
|
||||
|
||||
assert verfuegbare_masse(db, product) == []
|
||||
assert mass_aufloesen(db, product, "EL") is None
|
||||
|
||||
|
||||
# --------------------------------------------------------------------------
|
||||
# Bearbeitungsansicht: eigene und geerbte Mengen
|
||||
# --------------------------------------------------------------------------
|
||||
def test_kategorie_sieht_ihre_eigenen_mengen(db, masse):
|
||||
"""Ohne Vererbung ist die Liste die reine Bearbeitungsansicht."""
|
||||
ober = _kategorie(db, "Mehl & Zucker")
|
||||
_mass_an_kategorie(db, masse["Esslöffel"], ober, 10)
|
||||
|
||||
liste = category_measures(ober.id, db=db, _=None)
|
||||
|
||||
assert [(e.name, e.amount, e.herkunft) for e in liste] == [("Esslöffel", 10, "Kategorie")]
|
||||
|
||||
|
||||
def test_unterkategorie_zeigt_das_geerbte_als_solches(db, masse):
|
||||
"""Der Platzhalter in der Maske: Was gilt, wenn ich das Feld leer lasse?"""
|
||||
ober = _kategorie(db, "Mehl & Zucker")
|
||||
unter = _kategorie(db, "Backzutaten", parent=ober)
|
||||
_mass_an_kategorie(db, masse["Esslöffel"], ober, 10)
|
||||
|
||||
ohne = category_measures(unter.id, db=db, _=None)
|
||||
mit = category_measures(unter.id, vererbt=True, db=db, _=None)
|
||||
|
||||
assert ohne == []
|
||||
assert [(e.name, e.amount, e.herkunft) for e in mit] == [
|
||||
("Esslöffel", 10, "Kategorie: Mehl & Zucker"),
|
||||
]
|
||||
|
||||
|
||||
def test_eigener_wert_verdeckt_den_geerbten_in_der_liste(db, masse):
|
||||
"""Sonst stünde dieselbe Zeile zweimal da, mit verschiedenen Zahlen."""
|
||||
ober = _kategorie(db, "Mehl & Zucker")
|
||||
unter = _kategorie(db, "Backzutaten", parent=ober)
|
||||
_mass_an_kategorie(db, masse["Esslöffel"], ober, 10)
|
||||
_mass_an_kategorie(db, masse["Esslöffel"], unter, 12)
|
||||
|
||||
liste = category_measures(unter.id, vererbt=True, db=db, _=None)
|
||||
|
||||
assert [(e.amount, e.herkunft) for e in liste] == [(12, "Kategorie")]
|
||||
|
||||
|
||||
def test_geerbte_frist_nennt_die_kategorie_die_sie_setzt(db):
|
||||
ober = _kategorie(db, "Molkerei")
|
||||
unter = _kategorie(db, "Frischmilch", parent=ober)
|
||||
ober.open_days = 5
|
||||
db.commit()
|
||||
|
||||
assert geerbte_kategorie_frist(db, unter) == (5, "Kategorie: Molkerei")
|
||||
# Die Kategorie, die den Wert selbst setzt, erbt ihn NICHT von sich.
|
||||
assert geerbte_kategorie_frist(db, ober)[1] == "Einstellungen"
|
||||
|
||||
Reference in New Issue
Block a user