From da0872e0378289f5c9503f94c4141ac70f5acb79 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Wed, 30 May 2018 17:29:57 +0000 Subject: [PATCH] Bug 20837: Unit tests To test: prove -v t/db_dependent/Holds.t --- t/db_dependent/Holds.t | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/t/db_dependent/Holds.t b/t/db_dependent/Holds.t index 1e6c234..d52c3f8 100755 --- a/t/db_dependent/Holds.t +++ b/t/db_dependent/Holds.t @@ -7,7 +7,7 @@ use t::lib::TestBuilder; use C4::Context; -use Test::More tests => 55; +use Test::More tests => 56; use MARC::Record; use C4::Biblio; use C4::Items; @@ -373,11 +373,16 @@ $dbh->do(q{ is(CanItemBeReserved($borrowernumbers[0], $itemnumber), 'notReservable', "CanItemBeReserved should return 'notReservable'"); +t::lib::Mocks::mock_preference( 'ReservesControlBranch', 'PatronLibrary' ); ($item_bibnum, $item_bibitemnum, $itemnumber) = AddItem( { homebranch => $branch_2, holdingbranch => $branch_1, itype => 'CAN' } , $bibnum); is(CanItemBeReserved($borrowernumbers[0], $itemnumber), 'cannotReserveFromOtherBranches', - "CanItemBeReserved should return 'cannotReserveFromOtherBranches'"); + "CanItemBeReserved should use PatronLibrary rule when ReservesControlBranch set to 'PatronLibrary'"); +t::lib::Mocks::mock_preference( 'ReservesControlBranch', 'ItemHomeLibrary' ); +is(CanItemBeReserved($borrowernumbers[0], $itemnumber), + 'OK', + "CanItemBeReserved should use item home library rule when RsservesControlBranch set to 'ItemsHomeLibrary'"); ($item_bibnum, $item_bibitemnum, $itemnumber) = AddItem( { homebranch => $branch_1, holdingbranch => $branch_1, itype => 'CAN' } , $bibnum); -- 2.1.4