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

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.patch 2.58 KB
Testbed results
tests.patchpassedPassed: 7256 passes, 0 fails, 0 exceptions Detailed results

#4

catch - August 25, 2008 - 12:47
Title:TestingParty08: default site mail settings» Tests needed mail.inc
Status:active» 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.

#6

Rob Loach - November 7, 2008 - 23:48

#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

System Message - November 16, 2008 - 21:45
Status:needs review» needs work

The last submitted patch failed testing.

#8

lilou - November 17, 2008 - 14:18

#9

catch - November 19, 2008 - 01:15
Status:needs review» needs work

We no longer have phpdoc for setUp(), getInfo() or tearDown()
http://drupal.org/node/325974

#10

Rob Loach - December 7, 2008 - 23:10
Status:needs work» postponed

Postponed until #331180: fix pluggable smtp/mail framework is in.

#11

Jax - September 5, 2009 - 13:36
Component:tests» base system

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?

 
 

Drupal is a registered trademark of Dries Buytaert.