Fixing up more bugs.

Author: tim-schilling

docs/models.py

@@ -310,7 +310,7 @@ def _clean_document_path(path):
 
 
 def document_url(doc):
-    if doc.metadata["parents"] == DocumentationCategory.WEBSITE.value:
+    if doc.metadata.get("parents") == DocumentationCategory.WEBSITE.value:
         return doc.path
     elif doc.path:
         kwargs = {

docs/tests/test_models.py

@@ -632,7 +632,7 @@ def setUpTestData(cls):
             },
             # I'm not sure if this is valid or not.
             {
-                "metadata": {"parents": "topics"},
+                "metadata": {},
                 "path": "",
                 "release": cls.release,
                 "title": "Index",

docs/tests/test_views.py

@@ -91,7 +91,7 @@ def test_search_type_filter_all(self):
         )
         self.assertEqual(response.status_code, 200)
         self.assertContains(
-            response, "4 results for <em>generic</em> in version 5.1", html=True
+            response, "5 results for <em>generic</em> in version 5.1", html=True
         )
         self.assertContains(response, self.active_filter, count=1)
         self.assertContains(response, f"{self.active_filter}All</a>", html=True)
@@ -122,7 +122,7 @@ def test_search_category_filter_invalid_doc_categories(self):
         )
         self.assertEqual(response.status_code, 200)
         self.assertContains(
-            response, "4 results for <em>generic</em> in version 5.1", html=True
+            response, "5 results for <em>generic</em> in version 5.1", html=True
         )
         self.assertContains(response, self.active_filter, count=1)
         self.assertContains(response, f"{self.active_filter}All</a>", html=True)