From 925843d4562ef947a57952cad100645fcdb3277d Mon Sep 17 00:00:00 2001 From: Fridolin Somers Date: Tue, 22 Jun 2021 14:21:20 +0200 Subject: [PATCH] Revert "Bug 28503: (follow-up) Get rid of tests warnings" This reverts commit e98c8cb707d7a23d6b61cfa3f3960b2fc3b5a026. --- t/db_dependent/Reserves.t | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/t/db_dependent/Reserves.t b/t/db_dependent/Reserves.t index f786cf03bd..ab9f322757 100755 --- a/t/db_dependent/Reserves.t +++ b/t/db_dependent/Reserves.t @@ -1312,15 +1312,11 @@ subtest 'ModReserveAffect logging' => sub { $hold = Koha::Holds->find($reserve_id); is( $hold->timestamp, $previous_timestamp, 'Make sure the previous timestamp has been used' ); - # Avoid warnings - my $reserve_mock = Test::MockModule->new('C4::Reserves'); - $reserve_mock->mock( '_koha_notify_reserve', undef ); - # Mark it waiting ModReserveAffect( $item->itemnumber, $patron->borrowernumber ); - $hold->discard_changes; - ok( $hold->is_waiting, 'Hold has been set waiting' ); + $hold = Koha::Holds->find($reserve_id); + is( $hold->found, 'W', 'Hold has been set waiting' ); isnt( $hold->timestamp, $previous_timestamp, 'The timestamp has been modified' ); my $log = Koha::ActionLogs->search({ module => 'HOLDS', action => 'MODIFY', object => $hold->reserve_id })->next; -- 2.39.5