|
Lines 1640-1646
subtest "Bug 13841 - Do not create new 0 amount fines" => sub {
Link Here
|
| 1640 |
}; |
1640 |
}; |
| 1641 |
|
1641 |
|
| 1642 |
subtest "AllowRenewalIfOtherItemsAvailable tests" => sub { |
1642 |
subtest "AllowRenewalIfOtherItemsAvailable tests" => sub { |
| 1643 |
plan tests => 13; |
1643 |
plan tests => 12; |
| 1644 |
my $biblio = $builder->build_sample_biblio(); |
1644 |
my $biblio = $builder->build_sample_biblio(); |
| 1645 |
my $item_1 = $builder->build_sample_item( |
1645 |
my $item_1 = $builder->build_sample_item( |
| 1646 |
{ |
1646 |
{ |
|
Lines 1816-1822
subtest "AllowRenewalIfOtherItemsAvailable tests" => sub {
Link Here
|
| 1816 |
|
1816 |
|
| 1817 |
my $mock_circ = Test::MockModule->new("C4::Circulation"); |
1817 |
my $mock_circ = Test::MockModule->new("C4::Circulation"); |
| 1818 |
$mock_circ->mock( CanItemBeReserved => sub { |
1818 |
$mock_circ->mock( CanItemBeReserved => sub { |
| 1819 |
warn "Checked"; |
|
|
| 1820 |
return { status => 'no' } |
1819 |
return { status => 'no' } |
| 1821 |
} ); |
1820 |
} ); |
| 1822 |
|
1821 |
|
|
Lines 1824-1832
subtest "AllowRenewalIfOtherItemsAvailable tests" => sub {
Link Here
|
| 1824 |
$item_3->delete(); |
1823 |
$item_3->delete(); |
| 1825 |
# Two items total, one item available, one issued, two holds on record |
1824 |
# Two items total, one item available, one issued, two holds on record |
| 1826 |
|
1825 |
|
| 1827 |
warnings_are{ |
1826 |
( $renewokay, $error ) = CanBookBeRenewed( $borrowernumber1, $item_1->itemnumber ); |
| 1828 |
( $renewokay, $error ) = CanBookBeRenewed( $borrowernumber1, $item_1->itemnumber ); |
|
|
| 1829 |
} [], "CanItemBeReserved not called when there are more possible holds than available items"; |
| 1830 |
is( $renewokay, 0, 'Borrower cannot renew when there are more holds than available items' ); |
1827 |
is( $renewokay, 0, 'Borrower cannot renew when there are more holds than available items' ); |
| 1831 |
|
1828 |
|
| 1832 |
$item_3 = $builder->build_sample_item( |
1829 |
$item_3 = $builder->build_sample_item( |
|
Lines 1848-1853
subtest "AllowRenewalIfOtherItemsAvailable tests" => sub {
Link Here
|
| 1848 |
} |
1845 |
} |
| 1849 |
); |
1846 |
); |
| 1850 |
|
1847 |
|
|
|
1848 |
$mock_circ->mock( CanItemBeReserved => sub { |
| 1849 |
warn "Checked"; |
| 1850 |
return { status => 'no' } |
| 1851 |
} ); |
| 1852 |
|
| 1851 |
warnings_are{ |
1853 |
warnings_are{ |
| 1852 |
( $renewokay, $error ) = CanBookBeRenewed( $borrowernumber1, $item_1->itemnumber ); |
1854 |
( $renewokay, $error ) = CanBookBeRenewed( $borrowernumber1, $item_1->itemnumber ); |
| 1853 |
} ["Checked","Checked"], "CanItemBeReserved only called once per available item if it returns a negative result for all items for a borrower"; |
1855 |
} ["Checked","Checked"], "CanItemBeReserved only called once per available item if it returns a negative result for all items for a borrower"; |
| 1854 |
- |
|
|