Comment 10 for bug 1013786

Revision history for this message
Galen Charlton (gmc) wrote :

Kathy asked me to comment on the use of needstest tag here. An automated regression test would be a good thing to have -- and I suggest that Jeff or somebody else consider a follow-up to move the password strength check via regex to a self-contained routine (say in OpenILS::WWW::EGCatLoader::Util) that would be reachable to a direct Test::More test.

However, given the age of the patch and the user testing that has taken place so far, for this one it would be sufficient to just meet the new requirement that a written test plan be supplied. This could be done as a new commit message or just via a comment in this bug by one of the testers or by Jeff.