Opened 18 months ago

Last modified 18 months ago

#34240 closed Bug

assertRedirects() doesn't preserve headers set in RequestFactory/Client methods. — at Initial Version

Reported by: Mariusz Felisiak Owned by: nobody
Component: Testing framework Version: dev
Severity: Release blocker Keywords:
Cc: David Wobrock, Adam Johnson Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Similar to the #28337, HTTP headers are no longer in extra so they are no longer preserved in assertRedirects().

  • tests/test_client_regress/tests.py 

    diff --git a/tests/test_client_regress/tests.py b/tests/test_client_regress/tests.py
index aaf25e2ec4..53cdc3bf07 100644
    a b class AssertRedirectsTests(SimpleTestCase):  
    613613                    status_code=302,
    614614                    target_status_code=302,
    615615                )
     616                response = req_method(
     617                    "/redirect_based_on_extra_headers_1/",
     618                    follow=False,
     619                    headers={"redirect": "val"},
     620                )
     621                self.assertRedirects(
     622                    response,
     623                    "/redirect_based_on_extra_headers_2/",
     624                    fetch_redirect_response=True,
     625                    status_code=302,
     626                    target_status_code=302,
     627                )
    616628
    617629
    618630@override_settings(ROOT_URLCONF="test_client_regress.urls")

Bug in 67da22f08e05018ea968fcacbac9ac37ea925d85.

Change History (0)

Note: See TracTickets for help on using tickets.
Back to Top