diff --git a/tests/test_meta_approval.py b/tests/test_meta_approval.py index 76179ba52551f7f52f09e662f3ca39481dfc2087..affbc3dbd74d71ad7adf22f0ce6e8fbb44bf53b7 100644 --- a/tests/test_meta_approval.py +++ b/tests/test_meta_approval.py @@ -163,24 +163,14 @@ def test_approval_not_required_to_pay_credit_card(hook): ('Bank-Transfer', 'Assets:Savings'), ('Chargeback', 'Income:Donations'), ]) -def test_approval_not_required_for_asset_transfers(hook, tax_implication, other_acct): +def test_approval_not_required_by_tax_implication(hook, tax_implication, other_acct): txn = testutil.Transaction(postings=[ ('Assets:Checking', -250, {'tax-implication': tax_implication}), - (other_acct, 250), + (other_acct, 245), + ('Expenses:BankingFees', 5), ]) assert not list(hook.run(txn)) -def test_approval_required_for_partial_transfer(hook): - # I'm not sure this ever comes up in reality, but just being thorough - # out of an abundance of precaution. - txn = testutil.Transaction(postings=[ - ('Assets:Checking', -250, {'tax-implication': 'Bank-Transfer'}), - ('Assets:Savings', 225), - ('Expenses:BankingFees', 25), - ]) - actual = {error.message for error in hook.run(txn)} - assert actual == {"Assets:Checking missing {}".format(TEST_KEY)} - def test_not_required_on_flagged(hook): txn = testutil.Transaction(flag='!', postings=[ ('Assets:Checking', -25),