|
Lines 392-399
subtest 'get_lostreturn_policy() tests' => sub {
Link Here
|
| 392 |
branchcode => undef, |
392 |
branchcode => undef, |
| 393 |
categorycode => undef, |
393 |
categorycode => undef, |
| 394 |
itemtype => undef, |
394 |
itemtype => undef, |
| 395 |
rule_name => 'refund', |
395 |
rule_name => 'lostreturn', |
| 396 |
rule_value => 1 |
396 |
rule_value => 'charge' |
| 397 |
} |
397 |
} |
| 398 |
} |
398 |
} |
| 399 |
); |
399 |
); |
|
Lines 405-425
subtest 'get_lostreturn_policy() tests' => sub {
Link Here
|
| 405 |
branchcode => $branchcode, |
405 |
branchcode => $branchcode, |
| 406 |
categorycode => undef, |
406 |
categorycode => undef, |
| 407 |
itemtype => undef, |
407 |
itemtype => undef, |
| 408 |
rule_name => 'refund', |
408 |
rule_name => 'lostreturn', |
| 409 |
rule_value => 0 |
409 |
rule_value => 0 |
| 410 |
} |
410 |
} |
| 411 |
} |
411 |
} |
| 412 |
); |
412 |
); |
| 413 |
my $branchcode2 = $builder->build( { source => 'Branch' } )->{branchcode}; |
413 |
my $branchcode2 = $builder->build( { source => 'Branch' } )->{branchcode}; |
| 414 |
my $specific_rule_true = $builder->build( |
414 |
my $specific_rule_lostreturn = $builder->build( |
| 415 |
{ |
415 |
{ |
| 416 |
source => 'CirculationRule', |
416 |
source => 'CirculationRule', |
| 417 |
value => { |
417 |
value => { |
| 418 |
branchcode => $branchcode2, |
418 |
branchcode => $branchcode2, |
| 419 |
categorycode => undef, |
419 |
categorycode => undef, |
| 420 |
itemtype => undef, |
420 |
itemtype => undef, |
| 421 |
rule_name => 'refund', |
421 |
rule_name => 'lostreturn', |
| 422 |
rule_value => 1 |
422 |
rule_value => 'refund' |
| 423 |
} |
423 |
} |
| 424 |
} |
424 |
} |
| 425 |
); |
425 |
); |
|
Lines 432-438
subtest 'get_lostreturn_policy() tests' => sub {
Link Here
|
| 432 |
branchcode => $branchcode3, |
432 |
branchcode => $branchcode3, |
| 433 |
categorycode => undef, |
433 |
categorycode => undef, |
| 434 |
itemtype => undef, |
434 |
itemtype => undef, |
| 435 |
rule_name => 'refund', |
435 |
rule_name => 'lostreturn', |
|
|
436 |
rule_value => 'restore' |
| 436 |
} |
437 |
} |
| 437 |
} |
438 |
} |
| 438 |
); |
439 |
); |
|
Lines 443-449
subtest 'get_lostreturn_policy() tests' => sub {
Link Here
|
| 443 |
branchcode => $branch_without_rule, |
444 |
branchcode => $branch_without_rule, |
| 444 |
categorycode => undef, |
445 |
categorycode => undef, |
| 445 |
itemtype => undef, |
446 |
itemtype => undef, |
| 446 |
rule_name => 'refund' |
447 |
rule_name => 'lostreturn', |
|
|
448 |
rule_value => 'restore' |
| 447 |
} |
449 |
} |
| 448 |
) |
450 |
) |
| 449 |
->next |
451 |
->next |
|
Lines 451-486
subtest 'get_lostreturn_policy() tests' => sub {
Link Here
|
| 451 |
|
453 |
|
| 452 |
my $item = $builder->build_sample_item( |
454 |
my $item = $builder->build_sample_item( |
| 453 |
{ |
455 |
{ |
| 454 |
homebranch => $specific_rule_true->{branchcode}, |
456 |
homebranch => $specific_rule_lostreturn->{branchcode}, |
| 455 |
holdingbranch => $specific_rule_false->{branchcode} |
457 |
holdingbranch => $specific_rule_false->{branchcode} |
| 456 |
} |
458 |
} |
| 457 |
); |
459 |
); |
| 458 |
my $params = { |
460 |
my $params = { |
| 459 |
return_branch => $specific_rule_true->{ branchcode }, |
461 |
return_branch => $specific_rule_lostreturn->{ branchcode }, |
| 460 |
item => $item |
462 |
item => $item |
| 461 |
}; |
463 |
}; |
| 462 |
|
464 |
|
| 463 |
# Specific rules |
465 |
# Specific rules |
| 464 |
t::lib::Mocks::mock_preference( 'RefundLostOnReturnControl', 'CheckinLibrary' ); |
466 |
t::lib::Mocks::mock_preference( 'RefundLostOnReturnControl', 'CheckinLibrary' ); |
| 465 |
is( Koha::CirculationRules->get_lostreturn_policy( $params ), |
467 |
is( Koha::CirculationRules->get_lostreturn_policy( $params ), |
| 466 |
1,'Specific rule for checkin branch is applied (true)'); |
468 |
'refund','Specific rule for checkin branch is applied (refund)'); |
| 467 |
|
469 |
|
| 468 |
t::lib::Mocks::mock_preference( 'RefundLostOnReturnControl', 'ItemHomeBranch' ); |
470 |
t::lib::Mocks::mock_preference( 'RefundLostOnReturnControl', 'ItemHomeBranch' ); |
| 469 |
is( Koha::CirculationRules->get_lostreturn_policy( $params ), |
471 |
is( Koha::CirculationRules->get_lostreturn_policy( $params ), |
| 470 |
1,'Specific rule for home branch is applied (true)'); |
472 |
'refund','Specific rule for home branch is applied (refund)'); |
| 471 |
|
473 |
|
| 472 |
t::lib::Mocks::mock_preference( 'RefundLostOnReturnControl', 'ItemHoldingBranch' ); |
474 |
t::lib::Mocks::mock_preference( 'RefundLostOnReturnControl', 'ItemHoldingBranch' ); |
| 473 |
is( Koha::CirculationRules->get_lostreturn_policy( $params ), |
475 |
is( Koha::CirculationRules->get_lostreturn_policy( $params ), |
| 474 |
0,'Specific rule for holoding branch is applied (false)'); |
476 |
0,'Specific rule for holding branch is applied (false)'); |
| 475 |
|
477 |
|
| 476 |
# Default rule check |
478 |
# Default rule check |
| 477 |
t::lib::Mocks::mock_preference( 'RefundLostOnReturnControl', 'CheckinLibrary' ); |
479 |
t::lib::Mocks::mock_preference( 'RefundLostOnReturnControl', 'CheckinLibrary' ); |
| 478 |
$params->{return_branch} = $branch_without_rule; |
480 |
$params->{return_branch} = $branch_without_rule; |
| 479 |
is( Koha::CirculationRules->get_lostreturn_policy( $params ), |
481 |
is( Koha::CirculationRules->get_lostreturn_policy( $params ), |
| 480 |
1,'No rule for branch, global rule applied (true)'); |
482 |
'charge','No rule for branch, global rule applied (charge)'); |
| 481 |
|
483 |
|
| 482 |
# Change the default value just to try |
484 |
# Change the default value just to try |
| 483 |
Koha::CirculationRules->search({ branchcode => undef, rule_name => 'refund' })->next->rule_value(0)->store; |
485 |
Koha::CirculationRules->search({ branchcode => undef, rule_name => 'lostreturn' })->next->rule_value(0)->store; |
| 484 |
is( Koha::CirculationRules->get_lostreturn_policy( $params ), |
486 |
is( Koha::CirculationRules->get_lostreturn_policy( $params ), |
| 485 |
0,'No rule for branch, global rule applied (false)'); |
487 |
0,'No rule for branch, global rule applied (false)'); |
| 486 |
|
488 |
|
|
Lines 491-503
subtest 'get_lostreturn_policy() tests' => sub {
Link Here
|
| 491 |
branchcode => undef, |
493 |
branchcode => undef, |
| 492 |
categorycode => undef, |
494 |
categorycode => undef, |
| 493 |
itemtype => undef, |
495 |
itemtype => undef, |
| 494 |
rule_name => 'refund' |
496 |
rule_name => 'lostreturn' |
| 495 |
} |
497 |
} |
| 496 |
) |
498 |
) |
| 497 |
->next |
499 |
->next |
| 498 |
->delete; |
500 |
->delete; |
| 499 |
is( Koha::CirculationRules->get_lostreturn_policy( $params ), |
501 |
is( Koha::CirculationRules->get_lostreturn_policy( $params ), |
| 500 |
1,'No rule for branch, no default rule, fallback default (true)'); |
502 |
'refund','No rule for branch, no default rule, fallback default (refund)'); |
| 501 |
|
503 |
|
| 502 |
$schema->storage->txn_rollback; |
504 |
$schema->storage->txn_rollback; |
| 503 |
}; |
505 |
}; |