#2713: Sitemap Index with Cache gives a NoReverseMatch exception
-----------------------------------+----------------------------------------
Reporter: JayKlehr | Owner: stavros
Status: new | Milestone:
Component: Contrib apps | Version: SVN
Resolution: | Keywords:
Stage: Accepted | Has_patch: 1
Needs_docs: 0 | Needs_tests: 1
Needs_better_patch: 0 |
-----------------------------------+----------------------------------------
Comment (by russellm):
Replying to [comment:12 stavros]:
> Any idea where the regression tests should go? There are no preexisting
tests.
Then now is the time to write some.
> Does this code even need tests? It's pretty straightforward (I'm pretty
sure this will come back to bite me in the ass).
Yes, tests are required. Tests are not optional. You need to provide a
very compelling reason that tests are either impractical or impossible
before you get a pass on writing tests.
--
Ticket URL: <http://code.djangoproject.com/ticket/2713#comment:13>
Django <http://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com
To unsubscribe from this group, send email to
django-updates+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
