Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#14990 closed (fixed)

Custom sitemap unittests failing for contrib.sitemaps in 1.3 Beta 1

Reported by: eKIK Owned by: nobody
Component: Contrib apps Version: 1.3-beta
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:



From looking at ticket #14041 I can see that two custom sitemap xml files have been added:

  • django/contrib/sitemaps/tests/templates/custom_sitemap_index.xml
  • django/contrib/sitemaps/tests/templates/custom_sitemap.xml

The unittests using those xml files are test_simple_sitemap_custom_index and test_simple_custom_sitemap (they live in django/contrib/sitemaps/tests/

When downloading Django 1.3 Beta 1 the two unittests are there, but the xml files are not, which gives us two failing tests.

How to reproduce

  • Download and install Django 1.3 Beta 1
  • Create a project and enable the sitemaps application per instructions given in the documentation
  • Run python test

This should produce the two failing tests.

Proposed fix

Include custom_sitemap_index.xml and custom_sitemap.xml in the Django tarball.

Change History (3)

comment:1 Changed 6 years ago by Jannis Leidel

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedAccepted
Version: 1.3-beta

comment:2 Changed 6 years ago by Jannis Leidel

Resolution: fixed
Status: newclosed

(In [15114]) Fixed #14990 -- Added sitemaps tests templates to manifest template.

comment:3 Changed 5 years ago by Jacob

milestone: 1.3

Milestone 1.3 deleted

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