|
Lines 17-23
Link Here
|
| 17 |
|
17 |
|
| 18 |
use Modern::Perl; |
18 |
use Modern::Perl; |
| 19 |
|
19 |
|
| 20 |
use Test::More tests => 69; |
20 |
use Test::More tests => 81; |
| 21 |
use Test::MockModule; |
21 |
use Test::MockModule; |
| 22 |
use Test::Warn; |
22 |
use Test::Warn; |
| 23 |
|
23 |
|
|
Lines 62-67
my $frameworkcode = q//;
Link Here
|
| 62 |
|
62 |
|
| 63 |
|
63 |
|
| 64 |
t::lib::Mocks::mock_preference('ReservesNeedReturns', 1); |
64 |
t::lib::Mocks::mock_preference('ReservesNeedReturns', 1); |
|
|
65 |
t::lib::Mocks::mock_preference( 'IndependentBranchesTransfers', 0 ); |
| 65 |
|
66 |
|
| 66 |
# Somewhat arbitrary field chosen for age restriction unit tests. Must be added to db before the framework is cached |
67 |
# Somewhat arbitrary field chosen for age restriction unit tests. Must be added to db before the framework is cached |
| 67 |
$dbh->do("update marc_subfield_structure set kohafield='biblioitems.agerestriction' where tagfield='521' and tagsubfield='a' and frameworkcode=?", undef, $frameworkcode); |
68 |
$dbh->do("update marc_subfield_structure set kohafield='biblioitems.agerestriction' where tagfield='521' and tagsubfield='a' and frameworkcode=?", undef, $frameworkcode); |
|
Lines 128-133
ok(exists($reserve->{reserve_id}), 'CheckReserves() include reserve_id in its re
Link Here
|
| 128 |
($status, $reserve, $all_reserves) = CheckReserves( $item ); |
129 |
($status, $reserve, $all_reserves) = CheckReserves( $item ); |
| 129 |
is($status, "Reserved", "CheckReserves Test 2"); |
130 |
is($status, "Reserved", "CheckReserves Test 2"); |
| 130 |
|
131 |
|
|
|
132 |
# Set the preference 'IndependentBranchesTransfers' is set to 'yes' |
| 133 |
# the userenv branch and the branche code are not the same holds should be filtered |
| 134 |
t::lib::Mocks::mock_preference( 'IndependentBranchesTransfers', 1 ); |
| 135 |
($status, $reserve, $all_reserves) = CheckReserves( $item ); |
| 136 |
is( $status, "", 'Reserves is filtered'); |
| 137 |
|
| 138 |
# Set the userenv branchcode to be the same to the item branchcode. |
| 139 |
t::lib::Mocks::mock_userenv({ branchcode => $branchcode }); |
| 140 |
($status, $reserve, $all_reserves) = CheckReserves( $item ); |
| 141 |
is( $status, "Reserved", 'Reserves should not be filtered'); |
| 142 |
|
| 143 |
t::lib::Mocks::mock_userenv({ branchcode => $branch_1 }); |
| 144 |
t::lib::Mocks::mock_preference( 'IndependentBranchesTransfers', 0 ); |
| 145 |
my $ReservesControlBranch = C4::Context->preference('ReservesControlBranch'); |
| 146 |
t::lib::Mocks::mock_preference( 'ReservesControlBranch', 'ItemHomeLibrary' ); |
| 147 |
ok( |
| 148 |
$item->homebranch eq Koha::Policy::Holds->holds_control_library( $item, $patron ), |
| 149 |
"Koha::Policy::Holds->holds_control_library returns item home branch when set to ItemHomeLibrary" |
| 150 |
); |
| 151 |
t::lib::Mocks::mock_preference( 'ReservesControlBranch', 'PatronLibrary' ); |
| 152 |
ok( |
| 153 |
$patron->branchcode eq Koha::Policy::Holds->holds_control_library( $item, $patron ), |
| 154 |
"Koha::Policy::Holds->holds_control_library returns patron home branch when set to PatronLibrary" |
| 155 |
); |
| 156 |
t::lib::Mocks::mock_preference( 'ReservesControlBranch', $ReservesControlBranch ); |
| 157 |
|
| 131 |
### |
158 |
### |
| 132 |
### Regression test for bug 10272 |
159 |
### Regression test for bug 10272 |
| 133 |
### |
160 |
### |
| 134 |
- |
|
|