36 resulted in adding Debt.living_expense (and renaming Debt.reduction_rate to Debt.savings_rate). It also resulted in adding companion methods Debt.payment and Debt.payment_from_savings.
We should extend __init__ tests to ensure the the new property is being constructed correctly. We should also add tests for the new methods. Be sure to test for out-of-range values (e.g. negative available, available greater than max_inflow).
Consider also adding tests for Debt accounts with a non-default living_expense in both Debt tests and DebtPaymentStrategy tests.
36 resulted in adding
Debt.living_expense
(and renamingDebt.reduction_rate
toDebt.savings_rate
). It also resulted in adding companion methodsDebt.payment
andDebt.payment_from_savings
.We should extend
__init__
tests to ensure the the new property is being constructed correctly. We should also add tests for the new methods. Be sure to test for out-of-range values (e.g. negativeavailable
,available
greater thanmax_inflow
).Consider also adding tests for
Debt
accounts with a non-defaultliving_expense
in bothDebt
tests andDebtPaymentStrategy
tests.