https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30724 Nick Clemens (kidclamp) <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nick@bywatersolutions.com --- Comment #30 from Nick Clemens (kidclamp) <nick@bywatersolutions.com> --- (In reply to Jonathan Druart from comment #25)
Created attachment 187964 [details] [review] Bug 30724: Fix selenium tests - info => success class
I don't think this is correct. We only want success after we enabled 2FA
I am not sure of the problem here - you check for alert-success - but only expect it not blank when 2FA enabled, unless I misread (In reply to Jonathan Druart from comment #26)
This third patch fixes the selenium tests but they are not correct IMO.
We still need new tests to cover changes from this bug report.
More tests are always good, opened bug 41038 -- You are receiving this mail because: You are watching all bug changes.