Opened 17 years ago

Last modified 17 years ago

#5538 closed

assertRedirects should assume http://testserver if it isn't provided — at Initial Version

Reported by: Russell Keith-Magee Owned by: nobody
Component: Testing framework Version: dev
Severity: Keywords: assertRedirects testserver
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

[6164] fixed #987, a problem with redirects allowing relative URIs. The fix is correct, but it makes the testing API slightly unwieldy; all calls to assertRedirects() now require that 'http://testserver' be part of the asserted URL.

The vast majority of redirection tests will be internal to the site, so requiring the full dummy host prefix only serves to muddy the tests. This is made worse by the fact that urlresolvers.reverse() doesn't include the testserver component, so assertRedirects(reverse('some_name')) doesn't work.

assertRedirects should be modified to inspect the test URL; if no ':' doesn't occur in the test url, 'http://testserver' should be prepended for test purposes. i.e.,

assertRedirects('/some/url') is equivalent to assertRedirects('http://testserver/some/url')
assertRedirects('http://otherserver/some/url') would be untouched

Change History (0)

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