From 6d0c5dd546e317d11a286a0f8dbca426fe36a0b8 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Mon, 24 Dec 2018 15:59:22 +0000 Subject: [PATCH] Bug 17978: Unit tests for changes Signed-off-by: Liz Rea --- t/db_dependent/Holds.t | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/t/db_dependent/Holds.t b/t/db_dependent/Holds.t index e2066c7a88..cae1ec9f0d 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 => 57; +use Test::More tests => 58; use MARC::Record; use C4::Biblio; @@ -413,6 +413,13 @@ my $res_id = AddReserve( $branch_1, $borrowernumbers[0], $biblio->biblionumber, is( CanItemBeReserved( $borrowernumbers[0], $itemnumber )->{status}, 'tooManyReserves', 'Patron cannot reserve item with hold limit of 1, 1 bib level hold placed' ); + #results should be the same for both ReservesControlBranch settings +t::lib::Mocks::mock_preference( 'ReservesControlBranch', 'ItemHomeLibrary' ); +is( CanItemBeReserved( $borrowernumbers[0], $itemnumber )->{status}, + 'tooManyReserves', 'Patron cannot reserve item with hold limit of 1, 1 bib level hold placed' ); +#reset for further tests +t::lib::Mocks::mock_preference( 'ReservesControlBranch', 'PatronLibrary' ); + subtest 'Test max_holds per library/patron category' => sub { plan tests => 6; @@ -533,7 +540,7 @@ $schema->storage->txn_rollback; subtest 'CanItemBeReserved / holds_per_day tests' => sub { - plan tests => 9; + plan tests => 10; $schema->storage->txn_begin; @@ -653,6 +660,13 @@ subtest 'CanItemBeReserved / holds_per_day tests' => sub { { status => 'tooManyReserves', limit => 3 }, 'Unlimited daily holds, but reached reservesallowed' ); + #results should be the same for both ReservesControlBranch settings + t::lib::Mocks::mock_preference('ReservesControlBranch', 'ItemHomeLibrary'); + is_deeply( + CanItemBeReserved( $patron->borrowernumber, $itemnumber_3 ), + { status => 'tooManyReserves', limit => 3 }, + 'Unlimited daily holds, but reached reservesallowed' + ); $schema->storage->txn_rollback; }; -- 2.11.0