contact.test creates three email addresses whose domain is test.com.

test.com is registered to "Test Central, Inc.", who I am sure would not care for even the tiniest possibility that one of a million Drupal sites accidentally spams them in the process of a unit test.

It is advisable to use example.com, which ICANN has reserved for exactly this purpose.

This patch replaces test.com with example.com in contact.test.

CommentFileSizeAuthor
#1 contact-test-example-285184-1.patch1.19 KBcburschka

Comments

cburschka’s picture

cburschka
they
commented
StatusFileSize
new contact-test-example-285184-1.patch1.19 KB

Patch failed to upload yet again.

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Reviewed & tested by the community

Agreed.

boombatower’s picture

boombatower commented
Component: contact.module » tests

Looks good, I haven't run tests against patch though.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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