test: money-formatting edge cases (+ comma support in to_cents) — community PR by @MohammedAnasNathani

ledger.py

@@ -67,6 +67,8 @@ def to_cents(amount):
     """Turn a dollar amount (like 19.99 or '19.99') into integer cents."""
     if amount is None or amount == "":
         return 0
+    if isinstance(amount, str):
+        amount = amount.strip().replace(",", "")
     return int(round(float(amount) * 100))
 
 

tests/test_ledger.py

@@ -38,6 +38,20 @@ def test_format_money():
     assert ledger.format_money(0) == "0.00"
 
 
+def test_format_money_edge_cases():
+    assert ledger.format_money(1) == "0.01"  # exactly one cent
+    assert ledger.format_money(-199) == "-1.99"  # negative amount
+    assert ledger.format_money(100000001) == "1,000,000.01"  # over a million
+
+
+def test_to_cents_edge_cases():
+    assert ledger.to_cents(0.01) == 1  # exactly one cent
+    assert ledger.to_cents(-1.99) == -199  # negative amount
+    assert ledger.to_cents("1,234.56") == 123456  # comma thousands separator
+    assert ledger.to_cents("1,000,000.01") == 100000001  # over a million with commas
+    assert ledger.to_cents("") == 0
+
+
 # -------------------------------------------------------------- accounts