|
Lines 19-29
Link Here
|
| 19 |
|
19 |
|
| 20 |
use Modern::Perl; |
20 |
use Modern::Perl; |
| 21 |
|
21 |
|
| 22 |
use Test::More tests => 3; |
22 |
use Test::More tests => 4; |
| 23 |
|
23 |
|
| 24 |
use Test::Exception; |
24 |
use Test::Exception; |
| 25 |
use Test::MockModule; |
25 |
use Test::MockModule; |
|
|
26 |
use Test::Warn; |
| 26 |
|
27 |
|
|
|
28 |
use t::lib::Mocks; |
| 27 |
use t::lib::TestBuilder; |
29 |
use t::lib::TestBuilder; |
| 28 |
|
30 |
|
| 29 |
use Koha::Libraries; |
31 |
use Koha::Libraries; |
|
Lines 198-200
subtest 'is_pickup_location_valid() tests' => sub {
Link Here
|
| 198 |
|
200 |
|
| 199 |
$schema->storage->txn_rollback; |
201 |
$schema->storage->txn_rollback; |
| 200 |
}; |
202 |
}; |
| 201 |
- |
203 |
|
|
|
204 |
subtest 'cancel() tests' => sub { |
| 205 |
|
| 206 |
plan tests => 12; |
| 207 |
|
| 208 |
$schema->storage->txn_begin; |
| 209 |
|
| 210 |
my $get_prepared_letter_called; |
| 211 |
|
| 212 |
# Mock GetPreparedLetter so it raises a warning we can look for |
| 213 |
# and returns undef, so no call to EnqueueLetter happens |
| 214 |
my $mocked_letter = Test::MockModule->new("C4::Letters"); |
| 215 |
$mocked_letter->mock( 'GetPreparedLetter', sub { |
| 216 |
$get_prepared_letter_called = 1; |
| 217 |
return; |
| 218 |
}); |
| 219 |
|
| 220 |
my $hold = $builder->build_object( |
| 221 |
{ |
| 222 |
class => 'Koha::Holds', |
| 223 |
value => { |
| 224 |
cancellationdate => undef, |
| 225 |
priority => 1, |
| 226 |
cancellation_reason => undef, |
| 227 |
} |
| 228 |
} |
| 229 |
); |
| 230 |
|
| 231 |
# leave this things out of the test |
| 232 |
t::lib::Mocks::mock_preference( 'ExpireReservesMaxPickUpDelayCharge', 0 ); |
| 233 |
t::lib::Mocks::mock_preference( 'HoldsLog', 0 ); |
| 234 |
|
| 235 |
$hold = $builder->build_object( |
| 236 |
{ |
| 237 |
class => 'Koha::Holds', |
| 238 |
value => { |
| 239 |
cancellationdate => undef, |
| 240 |
priority => 1, |
| 241 |
cancellation_reason => undef, |
| 242 |
} |
| 243 |
} |
| 244 |
); |
| 245 |
|
| 246 |
$get_prepared_letter_called = 0; |
| 247 |
$hold->cancel({ cancellation_reason => 'Some reason' }); |
| 248 |
ok( !$get_prepared_letter_called, 'GetPreparedLetter not called' ); |
| 249 |
|
| 250 |
isnt( $hold->cancellationdate, undef, 'cancellationdate gets set to the passed value' ); |
| 251 |
is( $hold->priority, 0, 'priority gets set to 0' ); |
| 252 |
is( $hold->cancellation_reason, 'Some reason', 'cancellation_reason is set to the passed value' ); |
| 253 |
|
| 254 |
$hold = $builder->build_object( |
| 255 |
{ |
| 256 |
class => 'Koha::Holds', |
| 257 |
value => { |
| 258 |
cancellationdate => undef, |
| 259 |
priority => 1, |
| 260 |
cancellation_reason => undef, |
| 261 |
} |
| 262 |
} |
| 263 |
); |
| 264 |
|
| 265 |
$get_prepared_letter_called = 0; |
| 266 |
$hold->cancel({ cancellation_reason => 'Some reason', notify_patron => 1 }); |
| 267 |
ok( $get_prepared_letter_called, 'GetPreparedLetter called if notify_patron and cancellation_reason passed' ); |
| 268 |
|
| 269 |
isnt( $hold->cancellationdate, undef, 'cancellationdate gets set to the passed value' ); |
| 270 |
is( $hold->priority, 0, 'priority gets set to 0' ); |
| 271 |
is( $hold->cancellation_reason, 'Some reason', 'cancellation_reason is set to the passed value' ); |
| 272 |
|
| 273 |
$hold = $builder->build_object( |
| 274 |
{ |
| 275 |
class => 'Koha::Holds', |
| 276 |
value => { |
| 277 |
cancellationdate => undef, |
| 278 |
priority => 1, |
| 279 |
cancellation_reason => undef, |
| 280 |
} |
| 281 |
} |
| 282 |
); |
| 283 |
|
| 284 |
$get_prepared_letter_called = 0; |
| 285 |
$hold->cancel({ notify_patron => 1 }); |
| 286 |
ok( $get_prepared_letter_called, 'GetPreparedLetter called if notify_patron passed' ); |
| 287 |
|
| 288 |
isnt( $hold->cancellationdate, undef, 'cancellationdate gets set to the passed value' ); |
| 289 |
is( $hold->priority, 0, 'priority gets set to 0' ); |
| 290 |
is( $hold->cancellation_reason, undef, 'cancellation_reason not passed' ); |
| 291 |
|
| 292 |
$schema->storage->txn_rollback; |
| 293 |
}; |