Contact Module Test

boombatower - March 11, 2008 - 05:05
Project:SimpleTest
Version:6.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:boombatower
Status:closed
Description

I have create a test for the contact module that checks just about everything.

The only question that I have is to the validity of the valid_email_address function in common.inc

It returns the following e-mail addresses as valid:

  • invalid@site
  • invalid@site.

Both of which I don't think are valid, but maybe I am mistaken.

A possible solution could be to change the regular expression, maybe something like: ^[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,4}$.
Found at: http://www.regular-expressions.info/email.html.

AttachmentSize
contact_module.test_.txt10.34 KB

#1

boombatower - March 15, 2008 - 21:46
Status:patch (code needs review)» fixed

Committed.

I will leave the two e-mail address in there in hopes that the e-mail validation is fix: http://drupal.org/node/214114.

If not than this test can easily be patched.

#2

Anonymous (not verified) - March 29, 2008 - 21:52
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.