@@ -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.
('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),