tests: Use reverse_host() over hardcoded value

Author: rsebille

docs/tests/test_views.py

@@ -5,7 +5,7 @@
 from django.test import SimpleTestCase, TestCase
 from django.urls import reverse, set_urlconf
 from django.utils.translation import activate, gettext as _
-from django_hosts.resolvers import reverse as reverse_with_host
+from django_hosts.resolvers import reverse as reverse_with_host, reverse_host
 
 from djangoproject.urls import www as www_urls
 from releases.models import Release
@@ -31,7 +31,7 @@ def test_team_url(self):
     def test_internals_team(self):
         response = self.client.get(
             "/en/dev/internals/team/",
-            headers={"host": "docs.djangoproject.localhost:8000"},
+            headers={"host": reverse_host("docs")},
         )
         self.assertRedirects(
             response,
@@ -82,7 +82,7 @@ def tearDownClass(cls):
 
     def test_empty_get(self):
         response = self.client.get(
-            "/en/dev/search/", headers={"host": "docs.djangoproject.localhost:8000"}
+            "/en/dev/search/", headers={"host": reverse_host("docs")}
         )
         self.assertEqual(response.status_code, 200)
         # No header item is active.
@@ -93,7 +93,7 @@ def test_empty_get(self):
     def test_search_type_filter_all(self):
         response = self.client.get(
             "/en/5.1/search/?q=generic",
-            headers={"host": "docs.djangoproject.localhost:8000"},
+            headers={"host": reverse_host("docs")},
         )
         self.assertEqual(response.status_code, 200)
         self.assertContains(response, "5 results for <em>generic</em>", html=True)
@@ -105,7 +105,7 @@ def test_search_type_filter_by_doc_types(self):
             with self.subTest(category=category):
                 response = self.client.get(
                     f"/en/5.1/search/?q=generic&category={category.value}",
-                    headers={"host": "docs.djangoproject.localhost:8000"},
+                    headers={"host": reverse_host("docs")},
                 )
                 self.assertEqual(response.status_code, 200)
                 self.assertContains(
@@ -122,7 +122,7 @@ def test_search_type_filter_by_doc_types(self):
     def test_search_category_filter_invalid_doc_categories(self):
         response = self.client.get(
             "/en/5.1/search/?q=generic&category=invalid-so-ignored",
-            headers={"host": "docs.djangoproject.localhost:8000"},
+            headers={"host": reverse_host("docs")},
         )
         self.assertEqual(response.status_code, 200)
         self.assertContains(response, "5 results for <em>generic</em>", html=True)
@@ -132,7 +132,7 @@ def test_search_category_filter_invalid_doc_categories(self):
     def test_search_category_filter_no_results(self):
         response = self.client.get(
             "/en/5.1/search/?q=potato&category=ref",
-            headers={"host": "docs.djangoproject.localhost:8000"},
+            headers={"host": reverse_host("docs")},
         )
         self.assertEqual(response.status_code, 200)
         self.assertContains(response, self.active_filter, count=1)
@@ -267,7 +267,7 @@ def test_code_links(self):
             with self.subTest(query=query):
                 response = self.client.get(
                     f"/en/5.1/search/?q={query}",
-                    headers={"host": "docs.djangoproject.localhost:8000"},
+                    headers={"host": reverse_host("docs")},
                 )
                 self.assertEqual(response.status_code, 200)
                 self.assertContains(
@@ -289,7 +289,7 @@ def tearDownClass(cls):
 
     def test_sitemap_index(self):
         response = self.client.get(
-            "/sitemap.xml", headers={"host": "docs.djangoproject.localhost:8000"}
+            "/sitemap.xml", headers={"host": reverse_host("docs")}
         )
         self.assertContains(response, "<sitemap>", count=2)
         en_sitemap_url = reverse_with_host(
@@ -318,7 +318,7 @@ def test_sitemap(self):
 
     def test_sitemap_404(self):
         response = self.client.get(
-            "/sitemap-xx.xml", headers={"host": "docs.djangoproject.localhost:8000"}
+            "/sitemap-xx.xml", headers={"host": reverse_host("docs")}
         )
         self.assertEqual(response.status_code, 404)
         self.assertEqual(