TestingParty08: Comment signatures

cwgordon7 - August 16, 2008 - 02:18
Project:Drupal
Version:7.x-dev
Component:base system
Category:task
Priority:normal
Assigned:Unassigned
Status:needs work
Description

We need to test the ability for users to set default comment signatures.

Suggested assertions:
- Make sure user-set signature appears after comment is submitted.
- Make sure comments' signature changes after the user's signature is changed.

#1

catch - August 28, 2008 - 08:44
Status:active» needs work

This is pretty ugly, postComment probably needs to go into DrupalWebTestCase for a start, and it needs finishing up. But still, pancakes!

AttachmentSize
signaturetest.patch 3.79 KB

#2

catch - August 28, 2008 - 13:56
Status:needs work» needs review

Here goes. postComment is moved out of comment.test and becomes drupalPostComment in drupalWebTestCase() - otherwise unchanged.

Finished off the test, which passes. Kyle Matthews was my partner for the testing party, so should get credit on commit.

AttachmentSize
signature.test 12.73 KB

#3

catch - August 29, 2008 - 11:42

slight tidy up.

AttachmentSize
signature.test 12.3 KB

#4

catch - August 29, 2008 - 11:46
AttachmentSize
signature.test 12.45 KB

#5

flobruit - August 29, 2008 - 13:58

Patch looks good, just some formatting fixes.

I understand that unlike other drupalCreate*() methods, drupalPostComment() actually posts the information rather than creating an item programmatically. However, for consistency, shouldn't the method be named drupalCreateComment()?

Otherwise RTBC.

#6

flobruit - August 29, 2008 - 13:58

forgot patch

AttachmentSize
signature_test-296009-5.patch 11.71 KB

#7

chrisshattuck - October 12, 2008 - 02:34
Status:needs review» needs work

Patch had 4 failures and several successes. Looks like it needs to be updated.

#8

catch - October 17, 2009 - 07:18
Component:tests» base system
Category:bug report» task
Priority:critical» normal

Fixing the critical / pending bugs queues to reflect things which are really bugs or release critical.

 
 

Drupal is a registered trademark of Dries Buytaert.