From cef46729454872b19bf24a4a0bfcb7d7210130b1 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Fri, 3 May 2024 09:07:17 +0000 Subject: [PATCH] Bug 34972: (QA follow-up) Remove second transfer from Circulation.t Content-Type: text/plain; charset=utf-8 Removing the manual transfer and rightaway doing the Reserve transfer. One test description was misleading too. Signed-off-by: Marcel de Rooy --- t/db_dependent/Circulation.t | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/t/db_dependent/Circulation.t b/t/db_dependent/Circulation.t index 5170744e6e..766000718f 100755 --- a/t/db_dependent/Circulation.t +++ b/t/db_dependent/Circulation.t @@ -5974,12 +5974,8 @@ subtest 'Checkout should correctly terminate a transfer' => sub { my $do_transfer = 1; ModItemTransfer( $item->itemnumber, $library_1->branchcode, - $library_2->branchcode, 'Manual' ); + $library_2->branchcode, 'Reserve' ); ModReserveAffect( $item->itemnumber, undef, $do_transfer, $reserve_id ); - ModItemTransfer( - $item->itemnumber, $library_1->branchcode, - $library_2->branchcode, 'Reserve' - ); # To put the Reason, it's what does returns.pl... my $hold = Koha::Holds->find($reserve_id); is( $hold->found, 'T', 'Hold is in transit' ); my $transfer = $item->get_transfer; @@ -5992,7 +5988,7 @@ subtest 'Checkout should correctly terminate a transfer' => sub { $transfer = $transfer->get_from_storage; isnt( $transfer->datearrived, undef ); $hold = $hold->get_from_storage; - is( $hold->found, undef, 'Hold is waiting' ); + is( $hold->found, undef, 'Hold is not waiting or transit' ); is( $hold->priority, 1, ); }; -- 2.30.2