#12724 closed (fixed)
get_comment_permalink tests failing on Postgres
| Reported by: | Carl Meyer | Owned by: | Karen Tracey |
|---|---|---|---|
| Component: | contrib.comments | Version: | dev |
| Severity: | Keywords: | ||
| Cc: | Triage Stage: | Accepted | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
The tests for get_comment_permalink rely on created comments having specific IDs. Postgres transaction rollbacks don't reset sequences (apparently), so the created comments have much higher IDs than expected under Postgres.
Best fix isn't immediately apparent to me: either the actual comment id needs to be stored somewhere the test can get at it, or the test needs to query for the comment to get the id, or the assertion needs to be agnostic about the actual id.
(test_env)carljm@kale:~/projects/django/django/django-hg$ python tests/runtests.py comment_tests
======================================================================
FAIL: testGetCommentPermalink (regressiontests.comment_tests.tests.templatetag_tests.CommentTemplateTagTests)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/home/carljm/projects/django/django/django-hg/tests/regressiontests/comment_tests/tests/templatetag_tests.py", line 75, in testGetCommentPermalink
self.assertEqual(out, "/cr/%s/%s/#c2" % (ct.id, author.id))
AssertionError: u'/cr/13/1/#c71' != '/cr/13/1/#c2'
======================================================================
FAIL: testGetCommentPermalinkFormatted (regressiontests.comment_tests.tests.templatetag_tests.CommentTemplateTagTests)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/home/carljm/projects/django/django/django-hg/tests/regressiontests/comment_tests/tests/templatetag_tests.py", line 84, in testGetCommentPermalinkFormatted
self.assertEqual(out, "/cr/%s/%s/#c2-by-Joe Somebody" % (ct.id, author.id))
AssertionError: u'/cr/13/1/#c75-by-Joe Somebody' != '/cr/13/1/#c2-by-Joe Somebody'
----------------------------------------------------------------------
Ran 81 tests in 3.114s
FAILED (failures=2)
Change History (3)
comment:1 by , 16 years ago
| milestone: | → 1.2 |
|---|---|
| Owner: | changed from to |
| Status: | new → assigned |
| Triage Stage: | Unreviewed → Accepted |
| Version: | 1.1 → SVN |
comment:2 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Note:
See TracTickets
for help on using tickets.
We already have the comment ids, the tests just need to use them.