Posted by catch on August 23, 2008 at 8:46pm
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | base system |
| Category: | task |
| Priority: | normal |
| Assigned: | p.brouwers |
| Status: | needs work |
Issue Summary
Need to test one line e-mail handling in mail.inc
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | base system |
| Category: | task |
| Priority: | normal |
| Assigned: | p.brouwers |
| Status: | needs work |
Need to test one line e-mail handling in mail.inc
Comments
#1
Assigned!
#2
Sorry, withdrew myself. Too busy for now. If I've got more time I will try to pick up more tests.
#3
made a small test for it
#4
<?php+ $this->assertTrue(strpos($message['body'], '\n')===FALSE, t('Testing a one line mail'));
?>
Need spaces on either side of ===
#5
Sorry for my ignorance, but why should there be a specific test for single-line e-mails (what specifically should be tested)?
Pending patch has some code style errors, and with the advent of #296001: Capture e-mails sent during tests and add e-mail assertions / API it can probably be simplified a lot.
#6
Moving this out of the critical bugs queue - see #607038: Meta issue: fix gaps in code coverage.
#7
Changed the patch to use the new email-capture system and fixed the code style
#8
The last submitted patch failed testing.