The latest backport still has a D7 test called "SimpleTestMailCaptureTestCase" which has a bunch of functions that break under D6. Since AssertMail is new to Drupal 7, and not listed here:

http://drupal.org/node/265828

There shouldn't be any tests that depend on it. I think we can likely remove it?

CommentFileSizeAuthor
#2 573822-mail.patch4.36 KBboombatower

Comments

boombatower’s picture

boombatower commented

The point of the backport is the provide any new features...like assert mail. I am not sure how feasible it is, but I am aware of the issue. If it becomes a blocker I will remove before release, but I would like to try and support the feature.

boombatower’s picture

boombatower commented
Title: Remove SimpleTestMailCaptureTestCase from simpletest.test » Update SimpleTestMailCaptureTestCase from simpletest.test
Assigned: Unassigned » boombatower
Status: Active » Needs review
StatusFileSize
new 573822-mail.patch4.36 KB

Mail facility works in D6!

boombatower’s picture

boombatower commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.