TestMailer - Add Headers attribute in order for tests to asserts its contents

[ssmarco]

When running tests and finding that an email has been sent, in SS3 we are able to assert if the emails sent contains Cc/Bcc using customHeaders attribute. This pull request re-adds that feature using Headers attribute.

Reference:

silverstripe-framework/tests/email/EmailTest.php

Line 213 in 88a4e69

public function sendPlain($to, $from, $subject, $plainContent, $attachedFiles = false, $customheaders = false) {

[dhensby]

Wouldn't it be better to return/save all the headers not just these two?

[ssmarco]

Good point @dhensby, added all headers as suggested.

[dhensby]

looks good to me. It would be nice to squash these commits, but not a blocker

[michalkleiner]

Thanks @ssmarco. Please squash the commits and remove the reference to SS3 to avoid confusion. The commit message can be something along the lines of "ENH Save all headers when testing email sending" or similar.

[ssmarco]

Thanks @ssmarco. Please squash the commits and remove the reference to SS3 to avoid confusion. The commit message can be something along the lines of "ENH Save all headers when testing email sending" or similar.

Thanks. Done