From 95db387463a41b2279f3da54a6224f2b0d6da24c Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Thu, 28 Mar 2024 06:36:25 -0400 Subject: [PATCH] Bug 36159: Add unit test Content-Type: text/plain; charset=utf-8 Signed-off-by: Marcel de Rooy --- t/db_dependent/Koha/Patrons.t | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/t/db_dependent/Koha/Patrons.t b/t/db_dependent/Koha/Patrons.t index 85a1159fcd..ca186941b7 100755 --- a/t/db_dependent/Koha/Patrons.t +++ b/t/db_dependent/Koha/Patrons.t @@ -1608,7 +1608,7 @@ $retrieved_patron_1->delete; is( Koha::Patrons->search->count, $nb_of_patrons - 1, 'Delete should have deleted the patron' ); subtest 'BorrowersLog tests' => sub { - plan tests => 4; + plan tests => 5; t::lib::Mocks::mock_preference( 'BorrowersLog', 1 ); my $patron = $builder->build_object( { class => 'Koha::Patrons' } ); @@ -1627,6 +1627,13 @@ subtest 'BorrowersLog tests' => sub { $patron->update_lastseen('connection'); @logs = $schema->resultset('ActionLog')->search( { module => 'MEMBERS', action => 'MODIFY', object => $patron->borrowernumber } ); is( scalar @logs, 1, 'With BorrowerLogs and TrackLastPatronActivityTriggers we should not spam the logs'); + + Koha::ActionLogs->search()->delete(); + $patron->get_from_storage(); + $patron->set( { debarred => "" }); + $patron->store; + my $log = Koha::ActionLogs->search( { module => 'MEMBERS', action => 'MODIFY', object => $patron->borrowernumber } )->next; + isnt( defined $log, "No action log generated where incoming changed column is empty string and value in storage is NULL" ); }; $schema->storage->txn_rollback; -- 2.30.2