Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#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 Karen Tracey, 14 years ago

milestone: 1.2
Owner: changed from nobody to Karen Tracey
Status: newassigned
Triage Stage: UnreviewedAccepted
Version: 1.1SVN

We already have the comment ids, the tests just need to use them.

comment:2 by Karen Tracey, 14 years ago

Resolution: fixed
Status: assignedclosed

(In [12345]) Fixed #12724: Fixed new comment tests to not hardcode comment primary key values. Thanks carljm.

comment:3 by Jacob, 13 years ago

milestone: 1.2

Milestone 1.2 deleted

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