feat: Kontouebertraege lassen sich im Eingang buchen

Bisher blieb im Eingang nur, eine Umbuchung als Ausgabe oder Einnahme zu kategorisieren. Sie zaehlte dann als Ausgabe, obwohl kein Geld weg ist – nur verschoben. Jetzt gibt es je Vorschlag den Weg 'Als Uebertrag buchen': Zielkonto waehlen, fertig. Es entsteht EINE Buchung, die beide Salden bewegt und in keiner Auswertung zaehlt. Die Richtung kommt aus dem Vorzeichen – ein Abgang geht von hier weg, ein Zugang kommt vom Gegenkonto her; entsprechend fragt der Dialog 'wohin' oder 'woher'. Bei verschiedenen Waehrungen wird die Seite exakt gesetzt, die wir wirklich kennen, und die andere umgerechnet. Der teuerste Fehler an dieser Stelle waere ein Zielkonto, das selbst an einer Bank haengt: Es meldet den Zugang von allein, eine Gegenbuchung hier haette das Geld zweimal. Der Server lehnt das mit einer Begruendung ab; wer es besser weiss, bestaetigt und kommt durch. Das Wissen bleibt an einer Stelle – die Oberflaeche fragt nicht selbst nach, sie zeigt die Antwort des Servers. Die Betragslogik ist dafuer aus dem Buchungs-Router in einen Dienst gewandert. Zwei Kopien derselben Umrechnung waeren genau die Stelle, an der Salden auseinanderlaufen. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-27 06:31:49 +02:00
parent b9abb912e3
commit a471eba617
9 changed files with 415 additions and 33 deletions
@@ -0,0 +1,144 @@
+"""Eingang: einen Vorschlag als Kontoübertrag buchen.
+
+Ohne diesen Weg bleibt nur, eine Umbuchung als Ausgabe zu kategorisieren – sie
+zählt dann als Ausgabe, obwohl kein Geld weg ist.
+"""
+
+import secrets
+
+
+def _konto(admin, **kw):
+    body = {"name": "Girokonto", "type": "girokonto", "currency": "CHF",
+            "opening_balance_cents": 0}
+    body.update(kw)
+    r = admin.post("/api/accounts", json=body)
+    assert r.status_code == 201, r.text
+    return r.json()
+
+
+def _eingang(admin, konto, betrag_cents, datum="2026-08-20", payee="Umbuchung"):
+    """Ein OFFENER Eintrag – so wie ihn Mail und Bankabruf hinterlassen.
+
+    Nicht ueber die Schnellerfassung: Die bucht sofort, und dann gaebe es
+    nichts mehr als Uebertrag zu buchen.
+    """
+    from datetime import date
+
+    from app.database import SessionLocal
+    from app.models import InboxItem, InboxSource, InboxStatus, User
+
+    db = SessionLocal()
+    try:
+        besitzer = db.query(User).order_by(User.id).first()
+        eintrag = InboxItem(
+            owner_id=besitzer.id,
+            source=InboxSource.mail,
+            status=InboxStatus.pending,
+            account_id=konto["id"],
+            date=date.fromisoformat(datum),
+            amount_cents=betrag_cents,
+            currency=konto["currency"],
+            payee=payee,
+        )
+        db.add(eintrag)
+        db.commit()
+        return {"id": eintrag.id}
+    finally:
+        db.close()
+
+
+def test_eingang_laesst_sich_als_uebertrag_buchen(admin):
+    von = _konto(admin, name=f"Von {secrets.token_hex(3)}", currency="CHF")
+    nach = _konto(admin, name=f"Nach {secrets.token_hex(3)}", currency="CHF")
+    eintrag = _eingang(admin, von, -50000)
+
+    r = admin.post(f"/api/inbox/{eintrag['id']}/transfer",
+                   json={"to_account_id": nach["id"]})
+    assert r.status_code == 200, r.text
+    assert r.json()["status"] == "booked"
+    assert r.json()["category_id"] is None
+
+    txn = admin.get(f"/api/transactions/{r.json()['transaction_id']}").json()
+    assert txn["type"] == "transfer"
+    assert txn["account_id"] == von["id"]
+    assert txn["to_account_id"] == nach["id"]
+    assert txn["amount_cents"] == 50000
+
+    # Das Geld ist verschoben, nicht ausgegeben.
+    assert admin.get(f"/api/accounts/{von['id']}").json()["balance_cents"] == -50000
+    assert admin.get(f"/api/accounts/{nach['id']}").json()["balance_cents"] == 50000
+
+
+def test_zugang_dreht_die_richtung_um(admin):
+    """Ein positiver Betrag heisst: Auf DIESEM Konto kam das Geld an. Die
+    Buchung muss vom Gegenkonto hierher gehen, nicht umgekehrt."""
+    hier = _konto(admin, name=f"Hier {secrets.token_hex(3)}", currency="CHF")
+    dort = _konto(admin, name=f"Dort {secrets.token_hex(3)}", currency="CHF")
+    eintrag = _eingang(admin, hier, 25000)
+
+    r = admin.post(f"/api/inbox/{eintrag['id']}/transfer", json={"to_account_id": dort["id"]})
+    assert r.status_code == 200, r.text
+    txn = admin.get(f"/api/transactions/{r.json()['transaction_id']}").json()
+    assert txn["account_id"] == dort["id"], "der Abgang gehört zum Gegenkonto"
+    assert txn["to_account_id"] == hier["id"]
+    assert admin.get(f"/api/accounts/{hier['id']}").json()["balance_cents"] == 25000
+    assert admin.get(f"/api/accounts/{dort['id']}").json()["balance_cents"] == -25000
+
+
+def test_uebertrag_zaehlt_in_keiner_auswertung(admin):
+    """Der eigentliche Zweck: Ein Übertrag ist keine Ausgabe."""
+    von = _konto(admin, name=f"Budget-Von {secrets.token_hex(3)}", currency="CHF")
+    nach = _konto(admin, name=f"Budget-Nach {secrets.token_hex(3)}", currency="CHF")
+    eintrag = _eingang(admin, von, -30000, datum="2026-08-05")
+    admin.post(f"/api/inbox/{eintrag['id']}/transfer", json={"to_account_id": nach["id"]})
+
+    v = admin.get("/api/budget/2026-08").json()
+    gesamt = sum(z.get("actual_cents") or 0 for z in v.get("rows", []))
+    assert gesamt == 0, "ein Übertrag darf im Budget nicht auftauchen"
+
+
+def _binde_an(konto):
+    """Konto so markieren, als haenge es an einer Bank."""
+    from app.database import SessionLocal
+    from app.models import BankAccountLink, BankApp, BankConnection, User
+
+    db = SessionLocal()
+    try:
+        besitzer = db.query(User).order_by(User.id).first()
+        app = BankApp(owner_id=besitzer.id, application_id="app-x",
+                      private_key_enc="x", base_url="https://x.example")
+        db.add(app)
+        verbindung = BankConnection(owner_id=besitzer.id, aspsp_name="Zielbank",
+                                    aspsp_country="CH")
+        db.add(verbindung)
+        db.flush()
+        db.add(BankAccountLink(
+            owner_id=besitzer.id, connection_id=verbindung.id,
+            remote_uid=f"acc-{secrets.token_hex(4)}", currency=konto["currency"],
+            account_id=konto["id"]))
+        db.commit()
+    finally:
+        db.close()
+
+
+def test_angebundenes_zielkonto_wird_abgefangen(admin):
+    """Der teuerste Fehler an dieser Stelle: Das Zielkonto haengt selbst an
+    einer Bank und meldet den Zugang von allein – eine Gegenbuchung hier
+    bedeutet das Geld zweimal."""
+    ziel = _konto(admin, name=f"Angebunden {secrets.token_hex(3)}", currency="CHF")
+    _binde_an(ziel)
+
+    quelle = _konto(admin, name=f"Quelle {secrets.token_hex(3)}", currency="CHF")
+    eintrag = _eingang(admin, quelle, -10000)
+
+    r = admin.post(f"/api/inbox/{eintrag['id']}/transfer", json={"to_account_id": ziel["id"]})
+    assert r.status_code == 409, r.text
+    assert "angebunden" in r.json()["detail"]
+    # Nichts gebucht, nichts verschoben.
+    assert admin.get(f"/api/accounts/{ziel['id']}").json()["balance_cents"] == 0
+
+    # Wer es besser weiss, kommt trotzdem durch.
+    r = admin.post(f"/api/inbox/{eintrag['id']}/transfer",
+                   json={"to_account_id": ziel["id"], "trotzdem": True})
+    assert r.status_code == 200, r.text
+    assert admin.get(f"/api/accounts/{ziel['id']}").json()["balance_cents"] == 10000