NPO-Accounting/conservancy_beancount Changeset - 677c99b565d8

Changeset - 677c99b565d8

Parent rev.

Child rev.

[Not reviewed]

0 3 0

Brett Smith - 4 years ago 2020-06-03 22:51:48
brettcsmith@brettcsmith.org

accrual: Filter opening balance txn before main reporting.

So far we've been implicitly relying on this by the user passing search
terms that filter out the opening balance transaction. That will stop
happening with the aging report, so we need to do it ourselves.

3 files changed with 22 insertions and 8 deletions:

conservancy_beancount/reports/accrual.py

tests/books/accruals.beancount

tests/test_reports_accrual.py

0 comments (0 inline, 0 general)

conservancy_beancount/reports/accrual.py

➞

Show inline comments

@@ ... / @@ -495,12 +495,13 @@ def main(arglist: Optional[Sequence[str]]=None, @@
         entries = []
         source = {
             'filename': str(config.config_file_path()),
             'lineno': 1,
+        }
         load_errors = [Error(source, "no books to load in configuration", None)]
     filters.remove_opening_balance_txn(entries)
     postings = filter_search(data.Posting.from_entries(entries), args.search_terms)
     groups: PostGroups = dict(AccrualPostings.group_by_meta(postings, 'invoice'))
     groups = AccrualAccount.filter_paid_accruals(groups) or groups
     returncode = 0
     for error in load_errors:
         bc_printer.print_error(error, file=stderr)

tests/books/accruals.beancount

➞

Show inline comments

@@ ... / @@ -2,12 +2,18 @@ @@
 -01-01 open Assets:Receivable:Accounts
 -01-01 open Expenses:FilingFees
 -01-01 open Expenses:Services:Legal
 -01-01 open Expenses:Travel
 -01-01 open Income:Donations
 -01-01 open Liabilities:Payable:Accounts
 -01-01 open Equity:Funds:Opening
 -03-01 * "Opening balances"
   Equity:Funds:Opening  -1000 USD
   Assets:Receivable:Accounts  6000 USD
   Liabilities:Payable:Accounts  -5000 USD
 -03-05 * "EarlyBird" "Payment for receivable from previous FY"
   rt-id: "rt:40"
   invoice: "rt:40/400"
   Assets:Receivable:Accounts  -500 USD
   Assets:Checking  500 USD
@@ ... / @@ -21,12 +27,19 @@ @@
 -03-30 * "EarlyBird" "Travel reimbursement"
   rt-id: "rt:490"
   invoice: "rt:490/4900"
   Liabilities:Payable:Accounts  -75 USD
   Expenses:Travel  75 USD
 -04-30 ! "Vendor" "Travel reimbursement"
   rt-id: "rt:310"
   contract: "rt:310/3100"
   invoice: "FIXME"  ; still waiting on them to send it
   Liabilities:Payable:Accounts  -200 USD
   Expenses:Travel  200 USD
 -05-05 * "DonorA" "Donation pledge"
   rt-id: "rt:505"
   invoice: "rt:505/5050"
   approval: "rt:505/5040"
   Income:Donations  -1000.00 USD
   Assets:Receivable:Accounts  1000.00 USD

tests/test_reports_accrual.py

➞

Show inline comments

@@ ... / @@ -30,16 +30,21 @@ from beancount import loader as bc_loader @@
 from conservancy_beancount import data
 from conservancy_beancount import rtutil
 from conservancy_beancount.reports import accrual
 from conservancy_beancount.reports import core
 _accruals_load = bc_loader.load_file(testutil.test_path('books/accruals.beancount'))
 ACCRUAL_TXNS = [
     entry for entry in _accruals_load[0]
     if hasattr(entry, 'narration')
     and entry.narration != 'Opening balances'
+]
 ACCRUALS_COUNT = sum(
     for entry in _accruals_load[0]
     for post in getattr(entry, 'postings', ())
     for txn in ACCRUAL_TXNS
     for post in txn.postings
     if post.account.startswith(('Assets:Receivable:', 'Liabilities:Payable:'))
+)
 ACCOUNTS = [
     'Assets:Receivable:Accounts',
     'Assets:Receivable:Loans',
@@ ... / @@ -71,20 +76,15 @@ class RTClient(testutil.RTClient): @@
         ],
         '515': [],
         '520': [],
+    }
 @pytest.fixture
 def accrual_entries():
     return copy.deepcopy(_accruals_load[0])
 @pytest.fixture
 def accrual_postings():
     entries = copy.deepcopy(_accruals_load[0])
     return data.Posting.from_entries(entries)
     return data.Posting.from_entries(copy.deepcopy(ACCRUAL_TXNS))
 def check_link_regexp(regexp, match_s, first_link_only=False):
     assert regexp
     assert re.search(regexp, match_s)
     assert re.search(regexp, match_s + ' postlink')
     assert re.search(regexp, match_s + '0') is None

0 comments (0 inline, 0 general)