https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23816
Jonathan Druart <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA CC| |[email protected] | |ommunity.org --- Comment #41 from Jonathan Druart <[email protected]> --- Hola Agustin, 1. The first patch ("Set default value in tests") is dealing with default values we want to apply for all tests. It replaces your last patch. It's the way to go when you face such situations, easier than update all test scripts ;) 2. I am not sure we should modify misc/admin/set_password.pl It's an sysop script, and sysops (are supposed to) know what they are doing. It's a behaviour change that is not advertised. (for discussion) 3. It seems that we need make categorycode param mandatory. The only call to adjust is from the onboarding tool 4. I don't think we need a new test file, t/AuthUtils.t can be reused easily. The only thing you need is to mock the categories table (search for 'fixtures_ok' in t) -- 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/
