[Koha-bugs] [Bug 11013] Tests broken by new crypto stuff

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11013

Bernardo Gonzalez Kriegel <bgkriegel at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #21867|0                           |1
        is obsolete|                            |

--- Comment #4 from Bernardo Gonzalez Kriegel <bgkriegel at gmail.com> ---
Created attachment 21873
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21873&action=edit
[SIGNED-OFF] Bug 11013 - move hashing functions into their own module

As the hashing functions were centralised into C4::Auth, this required
more modules (C4::Members in particular) to use it. However, C4::Auth
uses the database, and so anything that used C4::Members failed to load
in the 00-load.t test. This patch moves the hashing into its own small
module so this isn't an issue.

Test plan:
* Ensure that password related functions work:
** Creating a user account
** Logging in
** Changing a password (OPAC and staff client)
** LDAP logins still work like they should
* Ensure that tests pass without a database server present

Note: there are testing notes for LDAP on bug 9611 that'll bypass the
need to set up an LDAP server.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel at gmail.com>
All Works. No koha-qa errors

Test
1) patch applied
2) new user created
3) change pass from opac/staff
4) change pass using LDAP (sort of, same as on Bug 9611)

NOTE: Robin, you just sent an email about copyright headers,
KohaPassword.pm's reads "...either version 2 of the License,...",
coding guidelines suggests other wording and version 3 of GPL.

-- 
You are receiving this mail because:
You are watching all bug changes.