From d392d582efe74df5547a14cf8d7cebc6225efde7 Mon Sep 17 00:00:00 2001
From: Shi Yao Wang <shi-yao.wang@inlibro.com>
Date: Tue, 17 May 2022 11:37:21 -0400
Subject: [PATCH] Bug 30730: Unit tests

Added an entry in holds for an item with negative notforloan value to
test if get_items_that_can_fill returns the right items.

Signed-off-by: David Nind <david@davidnind.com>
---
 t/db_dependent/Koha/Holds.t | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/t/db_dependent/Koha/Holds.t b/t/db_dependent/Koha/Holds.t
index 36040cbb3b..d9f8b666a7 100755
--- a/t/db_dependent/Koha/Holds.t
+++ b/t/db_dependent/Koha/Holds.t
@@ -426,7 +426,7 @@ subtest 'get_items_that_can_fill' => sub {
     my $item_5 = $builder->build_sample_item( { biblionumber => $biblio->biblionumber, itype => $itype_2->itemtype } );
     my $lost       = $builder->build_sample_item( { biblionumber => $biblio->biblionumber, itemlost => 1 } );
     my $withdrawn  = $builder->build_sample_item( { biblionumber => $biblio->biblionumber, withdrawn => 1 } );
-    my $notforloan = $builder->build_sample_item( { biblionumber => $biblio->biblionumber, notforloan => 1 } );
+    my $notforloan = $builder->build_sample_item( { biblionumber => $biblio->biblionumber, notforloan => -1 } );
 
     my $patron_1 = $builder->build_object( { class => 'Koha::Patrons' } );
     my $patron_2 = $builder->build_object( { class => 'Koha::Patrons' } );
@@ -466,6 +466,15 @@ subtest 'get_items_that_can_fill' => sub {
         }
     );
 
+    my $notforloan_reserve_id = C4::Reserves::AddReserve(
+        {
+            borrowernumber => $patron_2->borrowernumber,
+            biblionumber   => $biblio->biblionumber,
+            priority       => 0,
+            itemnumber     => $notforloan->itemnumber,
+        }
+    );
+
     # item 3 is on loan
     AddIssue( $patron_3->unblessed, $item_3->barcode );
 
@@ -483,7 +492,7 @@ subtest 'get_items_that_can_fill' => sub {
 
     $holds = Koha::Holds->search(
         {
-            reserve_id => [ $reserve_id_1, $reserve_id_2, $waiting_reserve_id, ]
+            reserve_id => [ $reserve_id_1, $reserve_id_2, $waiting_reserve_id, $notforloan_reserve_id, ]
         }
     );
 
-- 
2.30.2