refactor(service): remove set_entry_notes, superseded by update_entry (#21)
This commit is contained in:
parent
1c525f0202
commit
fe419fe802
2 changed files with 0 additions and 21 deletions
|
|
@ -115,17 +115,6 @@ def _clean_notes(raw: str | None) -> str | None:
|
||||||
return stripped if stripped else None
|
return stripped if stripped else None
|
||||||
|
|
||||||
|
|
||||||
def set_entry_notes(
|
|
||||||
db: Session, entry_id: int, notes: str | None
|
|
||||||
) -> Entry | None:
|
|
||||||
entry = db.get(Entry, entry_id)
|
|
||||||
if entry is None:
|
|
||||||
return None
|
|
||||||
entry.notes = _clean_notes(notes)
|
|
||||||
db.commit()
|
|
||||||
db.refresh(entry)
|
|
||||||
return entry
|
|
||||||
|
|
||||||
|
|
||||||
def update_entry(
|
def update_entry(
|
||||||
db: Session,
|
db: Session,
|
||||||
|
|
|
||||||
|
|
@ -24,16 +24,6 @@ def test_add_entry_strips_and_nulls_empty_notes(db):
|
||||||
assert entry.notes is None
|
assert entry.notes is None
|
||||||
|
|
||||||
|
|
||||||
def test_set_entry_notes_updates(db):
|
|
||||||
entry = service.add_entry(db, Section.food, "Groceries", Decimal("400.00"))
|
|
||||||
updated = service.set_entry_notes(db, entry.id, "weekly Costco run")
|
|
||||||
assert updated is not None
|
|
||||||
assert updated.notes == "weekly Costco run"
|
|
||||||
|
|
||||||
|
|
||||||
def test_set_entry_notes_missing_returns_none(db):
|
|
||||||
assert service.set_entry_notes(db, 9999, "oops") is None
|
|
||||||
|
|
||||||
|
|
||||||
def test_snapshot_copies_notes(db):
|
def test_snapshot_copies_notes(db):
|
||||||
service.add_entry(
|
service.add_entry(
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue