Comment 35 for bug 1728031

Revision history for this message
Robie Basak (racb) wrote :

This seems fine, but I don't see any mention of inverse tests in the test plan. Are we sure that password changes will continue to fail in scenarios where it is expected to fail, such as in admin password incorrect and user not authorized cases? If there are already automated tests that exercise the code being changed and verify these scenarios then that's fine. If not, then please add such tests to the test plan.