Tests needed mail.inc
cwgordon7 - June 29, 2008 - 18:31
| Project: | Drupal |
| Version: | 7.x-dev |
| Component: | base system |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | postponed |
Description
This information is from the code coverage report (see http://coverage.cwgordon.com/coverage).
We need to test:
Default site mail-from setting.
#1
I'm interested in doing these tests.
#2
Opening up for the testing party.
#3
I was busy writing the 4 mail tests in one .test file, but I see you've divided it into 4 different tests.
I've cleanup the files and left only this test in it.
I named it 'mail', I hope that's correct.
Should the other mail tests be in another file?
#4
Hi p.brouwers The split is pretty arbitrary due to the testing party at Drupalcon next week. If you've got a full test for this, please post it here and we can close out the other issues.
#5
ah ok.
I still need to write a test for CustomBackends. I'm not really familiar with that, so I don't know how long that'll take.
#6
#331180: fix pluggable smtp/mail framework introduces a nice pluggable mail system as well as a test for it, should this issue be merged in with it?
#7
The last submitted patch failed testing.
#8
See: #335122: Test clean HEAD after every commit and http://pastebin.ca/1258476
#9
We no longer have phpdoc for setUp(), getInfo() or tearDown()
http://drupal.org/node/325974
#10
Postponed until #331180: fix pluggable smtp/mail framework is in.
#11
Referenced issue was committed. A mail.test file has already been created. I'm not sure if this is still relevant. Can someone confirm this can be closed?