From 3dfe2669455b34f9a062f8c739f1a6b3988be9f9 Mon Sep 17 00:00:00 2001
From: Brett Smith <brettcsmith@brettcsmith.org>
Date: Tue, 7 Apr 2020 15:31:58 -0400
Subject: [PATCH] meta_payable_documentation: Bugfix which metadata we're
 checking.

Checking approval/contract was in the original specification,
looking at invoice instead of approval was a pure brain fart.
---
 conservancy_beancount/plugin/meta_payable_documentation.py | 2 +-
 tests/test_meta_payable_documentation.py                   | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/conservancy_beancount/plugin/meta_payable_documentation.py b/conservancy_beancount/plugin/meta_payable_documentation.py
index b100ff5..68b2f08 100644
--- a/conservancy_beancount/plugin/meta_payable_documentation.py
+++ b/conservancy_beancount/plugin/meta_payable_documentation.py
@@ -23,7 +23,7 @@ from ..beancount_types import (
 )
 
 class MetaPayableDocumentation(core._RequireLinksPostingMetadataHook):
-    CHECKED_METADATA = ['invoice', 'contract']
+    CHECKED_METADATA = ['approval', 'contract']
 
     def _run_on_post(self, txn: Transaction, post: data.Posting) -> bool:
         if post.account.is_under('Liabilities:Payable'):
diff --git a/tests/test_meta_payable_documentation.py b/tests/test_meta_payable_documentation.py
index 425e5d9..5add32b 100644
--- a/tests/test_meta_payable_documentation.py
+++ b/tests/test_meta_payable_documentation.py
@@ -29,20 +29,20 @@ ACCOUNTS = [
 ]
 
 SUPPORTING_METADATA = [
+    'approval',
     'contract',
-    'invoice',
 ]
 
 NON_SUPPORTING_METADATA = [
-    'approval',
     'check',
+    'invoice',
     'purchase-order',
     'receipt',
     'statement',
 ]
 
 OTHER_ACCT = 'Expenses:Other'
-MISSING_MSG = f"{{}} missing invoice/contract".format
+MISSING_MSG = f"{{}} missing approval/contract".format
 
 # for supporting links, use the lists from testutil