Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.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.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | signature_test-296009-5.patch | 11.71 KB | floretan |
| #4 | signature.test | 12.45 KB | catch |
| #3 | signature.test | 12.3 KB | catch |
| #2 | signature.test | 12.73 KB | catch |
| #1 | signaturetest.patch | 3.79 KB | catch |
Comments
Comment #1
catchThis is pretty ugly, postComment probably needs to go into DrupalWebTestCase for a start, and it needs finishing up. But still, pancakes!
Comment #2
catchHere 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.
Comment #3
catchslight tidy up.
Comment #4
catchComment #5
floretan CreditAttribution: floretan commentedPatch 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.
Comment #6
floretan CreditAttribution: floretan commentedforgot patch
Comment #7
chrisshattuck CreditAttribution: chrisshattuck commentedPatch had 4 failures and several successes. Looks like it needs to be updated.
Comment #8
catchFixing the critical / pending bugs queues to reflect things which are really bugs or release critical.