From 6f2f6d098ec2b09221fa043dfe7991d9d8d73144 Mon Sep 17 00:00:00 2001 From: Matt Westcott Date: Wed, 3 Apr 2024 21:22:57 +0100 Subject: [PATCH] Revert "Add test for redirects without trailing slash being redirected in one go" This reverts commit e58b0e0b1a250145266f3b8aa6e345a480feb01f. --- wagtail/contrib/redirects/tests/test_redirects.py | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/wagtail/contrib/redirects/tests/test_redirects.py b/wagtail/contrib/redirects/tests/test_redirects.py index 4cb3b0d0b40f..d6e8729ea6cc 100644 --- a/wagtail/contrib/redirects/tests/test_redirects.py +++ b/wagtail/contrib/redirects/tests/test_redirects.py @@ -200,21 +200,6 @@ def test_temporary_redirect(self): response, "/redirectto", status_code=302, fetch_redirect_response=False ) - def test_redirect_without_trailing_slash(self): - # Create a redirect - redirect = models.Redirect(old_path="/redirectme", redirect_link="/redirectto") - redirect.save() - - response = self.client.get("/redirectme") - # Request should be picked up by RedirectMiddleware, not CommonMiddleware - # (which would redirect to /redirectme/ instead). - # Before Django 4.2, CommonMiddleware performed the 'add trailing slash' test - # during the initial request processing, which took precedence over RedirectMiddleware - # and caused a double redirect (/redirectme -> /redirectme/ -> /redirectto). - self.assertRedirects( - response, "/redirectto", status_code=301, fetch_redirect_response=False - ) - def test_redirect_stripping_query_string(self): # Create a redirect which includes a query string redirect_with_query_string = models.Redirect(