Opened 15 years ago

Closed 13 years ago

#14478 closed Bug (fixed)

Couple of tests make assumptions about configuration

Reported by: Travis Swicegood Owned by: nobody
Component: Contrib apps Version: dev
Severity: Normal Keywords: tests
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

django.contrib.messages and django.contrib.sitemaps both run tests with certain assumptions that are not guaranteed. I've fixed both on ​my GitHub fork. They are available at:

The patches are against master, but I have tested the same changes against 1.2.3 and they work there as well.

Attachments (1)

14478.diff​ (3.6 KB ) - added by Claude Paroz 14 years ago.
Improved tests in current trunk

Download all attachments as: .zip

Change History (6)

comment:1 by Gabriel Hurley, 15 years ago

Triage Stage: Unreviewed → Accepted

The fix for sitemaps looks fine, but my inclination is that the messages test should restore settings.TEMPLATE_CONTEXT_PROCESSORS in the tearDown method the way other tests do for maximum isolation.

comment:2 by Julien Phalip, 15 years ago

Severity: → Normal
Type: → Bug

by Claude Paroz, 14 years ago

Attachment: 14478.diff​ added

Improved tests in current trunk

comment:3 by Claude Paroz, 14 years ago

Easy pickings: unset
Has patch: set
UI/UX: unset
Version: 1.2 → SVN

The USE_I18N setting in sitemaps has been addressed in #14833

comment:4 by Claude Paroz <claude@…>, 13 years ago

In [7676d6e764c47c9d33a755c3b861034b32de77ac]:

Made sitemaps tests use override_settings. Refs #14478

comment:5 by Claude Paroz <claude@…>, 13 years ago

Resolution: → fixed
Status: new → closed

In [6522283a712af96a98e99d675b69ca15aadbace8]:

Fixed #14478 -- Isolated messages tests from custom TEMPLATE_CONTEXT_PROCESSORS

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