Issue 4867 - Add global get_pages_metadata to template converters

tests/test_page_outputs.py

Index: tests/test_page_outputs.py
===================================================================
--- a/tests/test_page_outputs.py
+++ b/tests/test_page_outputs.py
@@ -65,13 +65,14 @@
 
 
 @pytest.mark.parametrize('filename,expected_output', expected_outputs)
 def test_dynamic(dynamic_server, filename, expected_output):
     response = urllib2.urlopen(dynamic_server + filename)
     assert response.read() == expected_output
 
 
-def test_revision_arg(revision, output_pages):
+def test_revision_arg(revision, temp_site):

[Vasily Kuznetsov, 2017/06/23 14:15:24]

Is this test related to `get_pages_metadata` global?

[Jon Sonesen, 2017/06/26 07:22:43]

Yeah, so I actually shouldn't have changed this. Did a nono, and changed the
tests to pass with the new code. But basically since I changed the metadata
parsing to be a generator I had to change the 'page_data' instantiation to
happen before calling the generator. This caused the page added by the revision
arg's test bookmark to get added to the page rendering on the filterless
get_pages_metadata calls, even though there was no metadata there the dict with
the key 'page': page name was getting through the filter. I fixy.

+    exists = os.path.isfile(os.path.join(temp_site, 'templates', 'bar.md'))
     if revision is None:
-        assert 'bar' in output_pages
-    else:
-        assert 'bar' not in output_pages
+        assert exists
+    elif revision is 'test':
+        assert not exists