https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30724

Nick Clemens (kidclamp) <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #30 from Nick Clemens (kidclamp) <[email protected]> ---
(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.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to