Test configuration of the comment block + display of the comment block.

CommentFileSizeAuthor
#1 296482_1_recent-comments-block.patch1.32 KBmr.baileys
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mr.baileys’s picture

Component: tests » comment.module
FileSize
1.32 KB

The following cases seem to be already covered by the current comment.test (in testRecentCommentBlock):

  • Verify that the block is available
  • Test setting the block title and comment count
  • Verify that users with "access comments" permission can see the block
  • Verify that users without the "access comments" permission cannot see the block
  • Verify that the number of comments shown is <= than the comment count setting

The only case missing IMO is verifying that all comments are shown if number of comments >= comment count setting. Patch attached to add this test case.

mr.baileys’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch failed testing.

mr.baileys’s picture

Status: Needs work » Needs review

re-test.

Status: Needs review » Needs work

The last submitted patch failed testing.

mr.baileys’s picture

Status: Needs work » Needs review

At first sight, this change is unrelated to the failed test (GD image library), and patch passed tests succesfully ealier, so re-testing to see if we can make the bot happy.

catch’s picture

Status: Needs review » Reviewed & tested by the community

Looks good, tests pass. RTBC.

sun’s picture

Issue tags: +Quick fix, +Simpletest

.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix, -Simpletest

Automatically closed -- issue fixed for 2 weeks with no activity.