Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 21431: Add unit test

Browse source 
Test plan:
Run t/db_dependent/Koha/Patrons.t

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Marcel de Rooy 2023-11-02 12:59:03 +00:00 committed by Tomas Cohen Arazi
parent a08dd73621
commit d2ace878de
Signed by: tomascohen
GPG key ID: 0A272EA1B2F3C15F
1 changed files with 19 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
t/db_dependent/Koha/Patrons.t
View file

@ -1921,7 +1921,7 @@ subtest '->store' => sub {
subtest '->set_password' => sub {
plan tests => 16;
plan tests => 17;
t::lib::Mocks::mock_preference( 'EmailFieldPrecedence', 'emailpro|email' );
@ -1994,10 +1994,25 @@ subtest '->set_password' => sub {
# Enable logging password changes
t::lib::Mocks::mock_preference( 'BorrowersLog', 1 );
$patron->set_password({ password => 'abcd b' });
$number_of_logs = $schema->resultset('ActionLog')->search( { module => 'MEMBERS', action => 'CHANGE PASS', object => $patron->borrowernumber } )->count;
$patron->set_password( { password => 'abcd b' } );
$number_of_logs = $schema->resultset('ActionLog')->search(
{
module => 'MEMBERS',
action => 'CHANGE PASS',
object => $patron->borrowernumber
}
)->count;
is( $number_of_logs, 1, 'With BorrowerLogs, Koha::Patron->set_password does log password changes' );
# add other action name
$patron->set_password( { password => 'abcd b2', action => 'RESET PASS' } );
$number_of_logs = $schema->resultset('ActionLog')->search(
{
module => 'MEMBERS',
action => 'RESET PASS',
object => $patron->borrowernumber
}
)->count;
is( $number_of_logs, 1, 'set_password allows another action name' );
# Enable notifying patrons of password changes
t::lib::Mocks::mock_preference( 'NotifyPasswordChange', 1 );