diff --git a/conservancy_beancount/reports/core.py b/conservancy_beancount/reports/core.py index be4b65d9f236a963064724e022a0128461c1cbf2..c692cf36d07de7252b011bf05b705f145afb2e8a 100644 --- a/conservancy_beancount/reports/core.py +++ b/conservancy_beancount/reports/core.py @@ -60,6 +60,13 @@ class Balance(Mapping[str, data.Amount]): def __repr__(self) -> str: return f"{type(self).__name__}({self._currency_map!r})" + def __str__(self) -> str: + amounts = [amount for amount in self.values() if amount.number] + if not amounts: + return "Zero balance" + amounts.sort(key=lambda amt: abs(amt.number), reverse=True) + return ', '.join(str(amount) for amount in amounts) + def __getitem__(self, key: str) -> data.Amount: return data.Amount(self._currency_map[key], key) diff --git a/tests/test_reports_balance.py b/tests/test_reports_balance.py index 68cb98cf03d6df08aff008b8b86601952cad549d..f3af572854b3b481d72a1414d302c01583b3340c 100644 --- a/tests/test_reports_balance.py +++ b/tests/test_reports_balance.py @@ -66,3 +66,14 @@ def test_mixed_balance(): assert len(balance) == 2 assert not balance.is_zero() assert all(balance[key] == amt for key, amt in amounts.items()) + +@pytest.mark.parametrize('balance_map_kwargs,expected', [ + ({}, "Zero balance"), + ({'JPY': 0, 'BRL': 0}, "Zero balance"), + ({'USD': '20.00'}, "20.00 USD"), + ({'EUR': '50.00', 'GBP': '80.00'}, "80.00 GBP, 50.00 EUR"), + ({'JPY': '-55.00', 'BRL': '-85.00'}, "-85.00 BRL, -55.00 JPY"), +]) +def test_str(balance_map_kwargs, expected): + amounts = testutil.balance_map(**balance_map_kwargs) + assert str(core.Balance(amounts.items())) == expected