Posted by xjm on July 5, 2012 at 4:20pm
|Issue tags:||needs backport to D7|
- Misuse of the method
DrupalTestBase::randomString()is frequently responsible for random, hard-to-debug test failures, because it generates a list of random characters that includes special characters like
#, *, /, \, ?, etc.
- Developers often do not consider the implications of using this for values that are not user input or that have already been sanitized, nor do they necessarily know to look for
DrupalTestBase::randomName()(which is the "safe" variant).
- Improve the documentation to clarify when each method should be used.
- Patch needs review.