@@ -, +, @@ --- t/db_dependent/Holds.t | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) --- a/t/db_dependent/Holds.t +++ a/t/db_dependent/Holds.t @@ -7,7 +7,7 @@ use t::lib::TestBuilder; use C4::Context; -use Test::More tests => 57; +use Test::More tests => 58; use MARC::Record; use C4::Biblio; @@ -375,11 +375,16 @@ $biblio = $builder->build_sample_biblio({ itemtype => 'CANNOT' }); is(CanItemBeReserved($borrowernumbers[0], $itemnumber)->{status}, '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' } , $biblio->biblionumber); is(CanItemBeReserved($borrowernumbers[0], $itemnumber)->{status}, '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' } , $biblio->biblionumber); --