As show in the second list there is a "Reviewed" column. I plan to review each of the tests, clean them up, and ensure that they test the necessary functionality.
This patch cleans up the code, adds comment blocks were necessary.
The test previously did not pass HEAD since it used the following two e-mail addresses as examples of invalid e-mails.
'invalid@site', 'invalid@site.'
The e-mails are invalid, but the core considers them fine. There are sever issues in the core intended to change the code that check's e-mails. Once that code is committed I will uncomment those two addresses.
| Comment | File | Size | Author |
|---|---|---|---|
| simpletest_review_contact.patch | 10.76 KB | boombatower |
Comments
Comment #1
boombatower commentedAs instructed I have committed the cleanup.
Comment #2
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.