Tests needed mail.inc

cwgordon7 - June 29, 2008 - 18:31
Project:Drupal
Version:7.x-dev
Component:tests
Category:bug report
Priority:critical
Assigned:Unassigned
Status:patch (code needs review)
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

p.brouwers - July 2, 2008 - 07:14
Assigned to:Anonymous» p.brouwers

I'm interested in doing these tests.

#2

catch - August 23, 2008 - 20:49
Title:Tests needed: mail.inc» TestingParty08: default site mail settings
Assigned to:p.brouwers» Anonymous

Opening up for the testing party.

#3

p.brouwers - August 25, 2008 - 12:24

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?

AttachmentSize
tests.patch2.58 KB

#4

catch - August 25, 2008 - 12:47
Title:TestingParty08: default site mail settings» Tests needed mail.inc
Status:active» patch (code needs review)

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

p.brouwers - August 27, 2008 - 09:30

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.

 
 

Drupal is a registered trademark of Dries Buytaert.