Line 0
Link Here
|
0 |
- |
1 |
#!/usr/bin/perl |
|
|
2 |
|
3 |
# This script includes tests for GetReserveFee and ChargeReserveFee |
4 |
|
5 |
# Copyright 2015 Rijksmuseum |
6 |
# |
7 |
# This file is part of Koha. |
8 |
# |
9 |
# Koha is free software; you can redistribute it and/or modify it |
10 |
# under the terms of the GNU General Public License as published by |
11 |
# the Free Software Foundation; either version 3 of the License, or |
12 |
# (at your option) any later version. |
13 |
# |
14 |
# Koha is distributed in the hope that it will be useful, but |
15 |
# WITHOUT ANY WARRANTY; without even the implied warranty of |
16 |
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
17 |
# GNU General Public License for more details. |
18 |
# |
19 |
# You should have received a copy of the GNU General Public License |
20 |
# along with Koha; if not, see <http://www.gnu.org/licenses>. |
21 |
|
22 |
use Modern::Perl; |
23 |
use Test::More tests => 5; |
24 |
use Test::MockModule; |
25 |
|
26 |
use C4::Circulation; |
27 |
use C4::Reserves qw|AddReserve|; |
28 |
use t::lib::TestBuilder; |
29 |
|
30 |
my $mContext = new Test::MockModule('C4::Context'); |
31 |
$mContext->mock( 'userenv', sub { |
32 |
return { branch => 'CPL' }; |
33 |
}); |
34 |
|
35 |
my $builder = t::lib::TestBuilder->new(); |
36 |
my $dbh = C4::Context->dbh; # after start transaction of testbuilder |
37 |
|
38 |
# Category with hold fee, two patrons |
39 |
$builder->build({ |
40 |
source => 'Category', |
41 |
value => { |
42 |
categorycode => 'XYZ1', |
43 |
reservefee => 2.5, |
44 |
}, |
45 |
}); |
46 |
my $patron1 = $builder->build({ |
47 |
source => 'Borrower', |
48 |
value => { |
49 |
categorycode => 'XYZ1', |
50 |
}, |
51 |
}); |
52 |
my $patron2 = $builder->build({ |
53 |
source => 'Borrower', |
54 |
value => { |
55 |
categorycode => 'XYZ1', |
56 |
}, |
57 |
}); |
58 |
|
59 |
# One biblio and two items |
60 |
my $biblio = $builder->build({ |
61 |
source => 'Biblio', |
62 |
value => { |
63 |
title => 'Title 1', |
64 |
}, |
65 |
}); |
66 |
my $item1 = $builder->build({ |
67 |
source => 'Item', |
68 |
value => { |
69 |
biblionumber => $biblio->{biblionumber}, |
70 |
}, |
71 |
}); |
72 |
my $item2 = $builder->build({ |
73 |
source => 'Item', |
74 |
value => { |
75 |
biblionumber => $biblio->{biblionumber}, |
76 |
}, |
77 |
}); |
78 |
|
79 |
|
80 |
# Actual testing starts here! |
81 |
# Add reserve for patron1, no fee expected |
82 |
# Note: AddReserve calls GetReserveFee and ChargeReserveFee |
83 |
my $acc1 = acctlines( $patron1->{borrowernumber} ); |
84 |
my $res1 = addreserve( $patron1->{borrowernumber} ); |
85 |
is( acctlines( $patron1->{borrowernumber} ), $acc1, 'No fee charged for patron 1' ); |
86 |
|
87 |
# Issue item1 to patron1. Since there is still a reserve too, we should |
88 |
# expect a charge for patron2. |
89 |
C4::Circulation::AddIssue( $patron1, $item1->{barcode}, '2015-12-31', 0, undef, 0, {} ); # the date does not really matter |
90 |
my $acc2 = acctlines( $patron2->{borrowernumber} ); |
91 |
my $fee = C4::Reserves::GetReserveFee( $patron2->{borrowernumber}, $biblio->{biblionumber} ); |
92 |
is( $fee > 0, 1, 'Patron 2 should be charged cf GetReserveFee' ); |
93 |
C4::Reserves::ChargeReserveFee( $patron2->{borrowernumber}, $fee, $biblio->{title} ); |
94 |
is( acctlines( $patron2->{borrowernumber} ), $acc2 + 1, 'Patron 2 has been charged by ChargeReserveFee' ); |
95 |
|
96 |
# If we delete the reserve, there should be no charge |
97 |
$dbh->do( "DELETE FROM reserves WHERE reserve_id=?", undef, ( $res1 ) ); |
98 |
$fee = C4::Reserves::GetReserveFee( $patron2->{borrowernumber}, $biblio->{biblionumber} ); |
99 |
is( $fee, 0, 'Patron 2 will not be charged now' ); |
100 |
|
101 |
# If we delete the second item, there should be a charge |
102 |
$dbh->do( "DELETE FROM items WHERE itemnumber=?", undef, ( $item2->{itemnumber} ) ); |
103 |
$fee = C4::Reserves::GetReserveFee( $patron2->{borrowernumber}, $biblio->{biblionumber} ); |
104 |
is( $fee > 0, 1, 'Patron 2 should be charged again this time' ); |
105 |
# End of tests |
106 |
|
107 |
|
108 |
sub acctlines { #calculate number of accountlines for a patron |
109 |
my @temp = $dbh->selectrow_array( "SELECT COUNT(*) FROM accountlines WHERE borrowernumber=?", undef, ( $_[0] ) ); |
110 |
return $temp[0]; |
111 |
} |
112 |
|
113 |
sub addreserve { |
114 |
return AddReserve( |
115 |
'CPL', |
116 |
$_[0], |
117 |
$biblio->{biblionumber}, |
118 |
undef, |
119 |
'1', |
120 |
undef, |
121 |
undef, |
122 |
'', |
123 |
$biblio->{title}, |
124 |
undef, |
125 |
'' |
126 |
); |
127 |
} |