Opened 15 years ago
Closed 15 years ago
#10818 closed (invalid)
auth tests fail for projects which override registration templates
| Reported by: | Chris Beaven | Owned by: | Chris Beaven |
|---|---|---|---|
| Component: | contrib.auth | Version: | dev |
| Severity: | Keywords: | ||
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
The problem is due to the now loudly failing {% url %} tag. If you override a registration template with one which uses a URL tag, you'll end up with your project tests failing loudly for the contrib.auth tests.
The solution is to put default templates in the test directory (and make sure all the tests use them)
Attachments (1)
Change History (4)
by , 15 years ago
| Attachment: | 10818.diff added |
|---|
comment:1 by , 15 years ago
| Summary: | auth tests fail due to overridden templates → auth tests fail for projects which override registration templates |
|---|---|
| Triage Stage: | Unreviewed → Ready for checkin |
comment:2 by , 15 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:3 by , 15 years ago
| Resolution: | → invalid |
|---|---|
| Status: | assigned → closed |
Yawn... I should really svn up before I start working on things, fixed as a side effect by #10747
Note:
See TracTickets
for help on using tickets.
(preview doesn't show that the new files actually contain
{{ form.as_ul }})This issue has bugged me on multiple projects, and this simple fix makes it work across the board.
I'll just be brash and push this direct to ready for checkin. Feel free to slap me round if it's not, committers :P