When using logme feature the brute force detection on login level might not work correctly.
As the parameter names are defined in Login plugin I don't think it would
make sense to move it. Also testing might only make sense by adding more UI
tests or maybe some kind of integration tests. Only testing that method
won't help if the parameter name can be changed somewhere else (eg in logme
method or login action form)
I guess we could potentially add assertions to an existing login/logme test (for example, "assert that a new entry was added to the brute force log"). I'll merge this for now, if needed we can modify the test later.