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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cburschka’s picture

cburschka
they
CreditAttribution: cburschka commented
FileSize
contact-test-example-285184-1.patch1.19 KB

Patch failed to upload yet again.

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Status: Needs review » Reviewed & tested by the community

Agreed.

boombatower’s picture

boombatower CreditAttribution: boombatower commented
Component: contact.module » tests

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

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

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

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