#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 , 15 years ago
milestone: | → 1.2 |
---|---|
Owner: | changed from | to
Status: | new → assigned |
Triage Stage: | Unreviewed → Accepted |
Version: | 1.1 → SVN |
comment:2 by , 15 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.