Lines 19-25
Link Here
|
19 |
|
19 |
|
20 |
use Modern::Perl; |
20 |
use Modern::Perl; |
21 |
|
21 |
|
22 |
use Test::More tests => 4; |
22 |
use Test::More tests => 5; |
23 |
|
23 |
|
24 |
use Test::Exception; |
24 |
use Test::Exception; |
25 |
use Test::MockModule; |
25 |
use Test::MockModule; |
Lines 27-38
use Test::MockModule;
Link Here
|
27 |
use t::lib::Mocks; |
27 |
use t::lib::Mocks; |
28 |
use t::lib::TestBuilder; |
28 |
use t::lib::TestBuilder; |
29 |
|
29 |
|
|
|
30 |
use Koha::ActionLogs; |
31 |
use Koha::Holds; |
30 |
use Koha::Libraries; |
32 |
use Koha::Libraries; |
31 |
use Koha::Old::Holds; |
33 |
use Koha::Old::Holds; |
32 |
|
34 |
|
33 |
my $schema = Koha::Database->new->schema; |
35 |
my $schema = Koha::Database->new->schema; |
34 |
my $builder = t::lib::TestBuilder->new; |
36 |
my $builder = t::lib::TestBuilder->new; |
35 |
|
37 |
|
|
|
38 |
subtest 'fill() tests' => sub { |
39 |
|
40 |
plan tests => 11; |
41 |
|
42 |
$schema->storage->txn_begin; |
43 |
|
44 |
my $fee = 15; |
45 |
|
46 |
my $category = $builder->build_object( |
47 |
{ |
48 |
class => 'Koha::Patron::Categories', |
49 |
value => { reservefee => $fee } |
50 |
} |
51 |
); |
52 |
my $patron = $builder->build_object( |
53 |
{ |
54 |
class => 'Koha::Patrons', |
55 |
value => { categorycode => $category->id } |
56 |
} |
57 |
); |
58 |
my $manager = $builder->build_object( { class => 'Koha::Patrons' } ); |
59 |
|
60 |
my $title = 'Do what you want'; |
61 |
my $biblio = $builder->build_sample_biblio( { title => $title } ); |
62 |
my $item = $builder->build_sample_item( { biblionumber => $biblio->id } ); |
63 |
my $hold = $builder->build_object( |
64 |
{ |
65 |
class => 'Koha::Holds', |
66 |
value => { |
67 |
biblionumber => $biblio->id, |
68 |
borrowernumber => $patron->id, |
69 |
itemnumber => $item->id, |
70 |
priority => 10, |
71 |
} |
72 |
} |
73 |
); |
74 |
|
75 |
t::lib::Mocks::mock_preference( 'HoldFeeMode', 'any_time_is_collected' ); |
76 |
t::lib::Mocks::mock_preference( 'HoldsLog', 1 ); |
77 |
t::lib::Mocks::mock_userenv( |
78 |
{ patron => $manager, branchcode => $manager->branchcode } ); |
79 |
|
80 |
my $interface = 'api'; |
81 |
C4::Context->interface($interface); |
82 |
|
83 |
my $ret = $hold->fill; |
84 |
|
85 |
is( ref($ret), 'Koha::Hold', '->fill returns the object type' ); |
86 |
is( $ret->id, $hold->id, '->fill returns the object' ); |
87 |
|
88 |
is( Koha::Holds->find($hold->id), undef, 'Hold no longer current' ); |
89 |
my $old_hold = Koha::Old::Holds->find( $hold->id ); |
90 |
|
91 |
is( $old_hold->id, $hold->id, 'reserve_id retained' ); |
92 |
is( $old_hold->priority, 0, 'priority set to 0' ); |
93 |
is( $old_hold->found, 'F', 'found set to F' ); |
94 |
|
95 |
subtest 'fee applied tests' => sub { |
96 |
|
97 |
plan tests => 9; |
98 |
|
99 |
my $account = $patron->account; |
100 |
is( $account->balance, $fee, 'Charge applied correctly' ); |
101 |
|
102 |
my $debits = $account->outstanding_debits; |
103 |
is( $debits->count, 1, 'Only one fee charged' ); |
104 |
|
105 |
my $fee_debit = $debits->next; |
106 |
is( $fee_debit->amount * 1, $fee, 'Fee amount stored correctly' ); |
107 |
is( $fee_debit->description, $title, |
108 |
'Fee description stored correctly' ); |
109 |
is( $fee_debit->manager_id, $manager->id, |
110 |
'Fee manager_id stored correctly' ); |
111 |
is( $fee_debit->branchcode, $manager->branchcode, |
112 |
'Fee branchcode stored correctly' ); |
113 |
is( $fee_debit->interface, $interface, |
114 |
'Fee interface stored correctly' ); |
115 |
is( $fee_debit->debit_type_code, |
116 |
'RESERVE', 'Fee debit_type_code stored correctly' ); |
117 |
is( $fee_debit->itemnumber, $item->id, |
118 |
'Fee itemnumber stored correctly' ); |
119 |
}; |
120 |
|
121 |
my $logs = Koha::ActionLogs->search( |
122 |
{ |
123 |
action => 'FILL', |
124 |
module => 'HOLDS', |
125 |
object => $hold->id |
126 |
} |
127 |
); |
128 |
|
129 |
is( $logs->count, 1, '1 log line added' ); |
130 |
|
131 |
# Set HoldFeeMode to something other than any_time_is_collected |
132 |
t::lib::Mocks::mock_preference( 'HoldFeeMode', 'not_always' ); |
133 |
# Disable logging |
134 |
t::lib::Mocks::mock_preference( 'HoldsLog', 0 ); |
135 |
|
136 |
$hold = $builder->build_object( |
137 |
{ |
138 |
class => 'Koha::Holds', |
139 |
value => { |
140 |
biblionumber => $biblio->id, |
141 |
borrowernumber => $patron->id, |
142 |
itemnumber => $item->id, |
143 |
priority => 10, |
144 |
} |
145 |
} |
146 |
); |
147 |
|
148 |
$hold->fill; |
149 |
|
150 |
my $account = $patron->account; |
151 |
is( $account->balance, $fee, 'No new charge applied' ); |
152 |
|
153 |
my $debits = $account->outstanding_debits; |
154 |
is( $debits->count, 1, 'Only one fee charged, because of HoldFeeMode' ); |
155 |
|
156 |
$logs = Koha::ActionLogs->search( |
157 |
{ |
158 |
action => 'FILL', |
159 |
module => 'HOLDS', |
160 |
object => $hold->id |
161 |
} |
162 |
); |
163 |
|
164 |
is( $logs->count, 0, 'HoldsLog disabled, no logs added' ); |
165 |
|
166 |
$schema->storage->txn_rollback; |
167 |
}; |
168 |
|
36 |
subtest 'patron() tests' => sub { |
169 |
subtest 'patron() tests' => sub { |
37 |
|
170 |
|
38 |
plan tests => 2; |
171 |
plan tests => 2; |
39 |
- |
|
|