NPO-Accounting/conservancy_beancount Changeset - 6aadb740e619

Changeset - 6aadb740e619

Parent rev.

Child rev.

[Not reviewed]

0 2 0

Brett Smith - 4 years ago 2020-05-01 20:18:46
brettcsmith@brettcsmith.org

meta_receipt: Don't check chargebacks. RT#10978.

2 files changed with 23 insertions and 0 deletions:

conservancy_beancount/plugin/meta_receipt.py

tests/test_meta_receipt.py

0 comments (0 inline, 0 general)

conservancy_beancount/plugin/meta_receipt.py

➞

Show inline comments

@@ ... / @@ -40,6 +40,7 @@ class MetaReceipt(core._RequireLinksPostingMetadataHook): @@
             (post.account.is_cash_equivalent() or post.account.is_credit_card())
             and not post.account.is_under('Assets:PayPal')
             and abs(post.units.number) >= self.payment_threshold
             and post.meta.get('tax-implication', '').title() != 'Chargeback'
+        )
     def _run_checking_debit(self, txn: Transaction, post: data.Posting) -> errormod.Iter:

tests/test_meta_receipt.py

➞

Show inline comments

@@ ... @@
     ])
     assert not list(hook.run(txn))
 @pytest.mark.parametrize('test_acct,other_acct', testutil.combine_values(
     ACCOUNTS,
     NOT_REQUIRED_ACCOUNTS,
 ))
 def test_not_required_on_chargeback_txns(hook, test_acct, other_acct):
     if test_acct.required_types & PostType.DEBIT:
         check(hook, test_acct, other_acct, None,
               txn_meta={'tax-implication': 'Chargeback'},
               check_type=PostType.DEBIT,
+        )
 @pytest.mark.parametrize('test_acct,other_acct', testutil.combine_values(
     ACCOUNTS,
     NOT_REQUIRED_ACCOUNTS,
 ))
 def test_not_required_on_chargeback_posts(hook, test_acct, other_acct):
     if test_acct.required_types & PostType.DEBIT:
         check(hook, test_acct, other_acct, None,
               post_meta={'tax-implication': 'Chargeback'},
               check_type=PostType.DEBIT,
+        )
 @pytest.mark.parametrize('test_acct,equity_acct', testutil.combine_values(
     ACCOUNTS,
     testutil.OPENING_EQUITY_ACCOUNTS,

0 comments (0 inline, 0 general)