NPO-Accounting/conservancy_beancount Changeset - 89aaae821b30

Changeset - 89aaae821b30

Parent rev.

Child rev.

[Not reviewed]

0 2 0

Brett Smith - 4 years ago 2020-03-18 20:25:37
brettcsmith@brettcsmith.org

data.Account: Exclude PrepaidVacation from "real" assets.

Same rationale as PrepaidExpenses.

2 files changed with 2 insertions and 0 deletions:

conservancy_beancount/data.py

tests/test_data_account.py

0 comments (0 inline, 0 general)

conservancy_beancount/data.py

➞

Show inline comments

@@ ... / @@ -36,12 +36,13 @@ class Account(str): @@
     SEP = bc_account.sep
     def is_real_asset(self) -> bool:
         return (
             self.is_under('Assets:')
             and not self.is_under('Assets:PrepaidExpenses')
             and not self.is_under('Assets:PrepaidVacation')
+        )
     def is_under(self, acct_s: str) -> bool:
         return self.startswith(acct_s) and (
             acct_s.endswith(self.SEP)
             or self == acct_s

tests/test_data_account.py

➞

Show inline comments

@@ ... / @@ -34,12 +34,13 @@ def test_is_under(acct_name, under_arg, expected): @@
 @pytest.mark.parametrize('acct_name,expected', [
     ('Accrued:AccountsPayable', False),
     ('Accrued:AccountsReceivable', False),
     ('Assets:Cash', True),
     ('Assets:Cash:EUR', True),
     ('Assets:PrepaidExpenses', False),
     ('Assets:PrepaidVacation', False),
     ('Assets:Bank:Checking', True),
     ('Expenses:General', False),
     ('Income:Donations', False),
 ])
 def test_is_real_asset(acct_name, expected):
     assert data.Account(acct_name).is_real_asset() == expected

0 comments (0 inline, 0 general)