mirror of
https://github.com/django/django.git
synced 2025-03-31 11:37:06 +00:00
Refs #31747 -- Added more tests for preserving query strings when redirect with APPEND_SLASH in admin.
This commit is contained in:
parent
55bcbd8d17
commit
4338b6526d
@ -8515,6 +8515,23 @@ class AdminSiteFinalCatchAllPatternTests(TestCase):
|
||||
"/test_admin/admin/login/?next=/test_admin/admin/admin_views/article",
|
||||
)
|
||||
|
||||
@override_settings(APPEND_SLASH=True)
|
||||
def test_missing_slash_append_slash_true_non_staff_user_query_string(self):
|
||||
user = User.objects.create_user(
|
||||
username="user",
|
||||
password="secret",
|
||||
email="user@example.com",
|
||||
is_staff=False,
|
||||
)
|
||||
self.client.force_login(user)
|
||||
known_url = reverse("admin:admin_views_article_changelist")
|
||||
response = self.client.get("%s?id=1" % known_url[:-1])
|
||||
self.assertRedirects(
|
||||
response,
|
||||
"/test_admin/admin/login/?next=/test_admin/admin/admin_views/article"
|
||||
"%3Fid%3D1",
|
||||
)
|
||||
|
||||
@override_settings(APPEND_SLASH=False)
|
||||
def test_missing_slash_append_slash_false(self):
|
||||
superuser = User.objects.create_user(
|
||||
@ -8629,6 +8646,24 @@ class AdminSiteFinalCatchAllPatternTests(TestCase):
|
||||
response, known_url, status_code=301, target_status_code=403
|
||||
)
|
||||
|
||||
@override_settings(APPEND_SLASH=True)
|
||||
def test_missing_slash_append_slash_true_query_without_final_catch_all_view(self):
|
||||
superuser = User.objects.create_user(
|
||||
username="staff",
|
||||
password="secret",
|
||||
email="staff@example.com",
|
||||
is_staff=True,
|
||||
)
|
||||
self.client.force_login(superuser)
|
||||
known_url = reverse("admin10:admin_views_article_changelist")
|
||||
response = self.client.get("%s?id=1" % known_url[:-1])
|
||||
self.assertRedirects(
|
||||
response,
|
||||
f"{known_url}?id=1",
|
||||
status_code=301,
|
||||
fetch_redirect_response=False,
|
||||
)
|
||||
|
||||
@override_settings(APPEND_SLASH=False)
|
||||
def test_missing_slash_append_slash_false_without_final_catch_all_view(self):
|
||||
superuser = User.objects.create_user(
|
||||
|
Loading…
x
Reference in New Issue
Block a user