gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 29021: (QA follow-up) Fix for new parameter, add unit test 

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit d1bd38d853)
Signed-off-by: Jacob O'Mara <jacob.omara@ptfs-europe.com>
This commit is contained in:
Kyle Hall 2023-01-27 11:58:45 -05:00 committed by Jacob O'Mara
parent 2c9ad0a59b
commit bce3f40d59
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
Koha/Account/Line.pm
View File

@ -1035,6 +1035,7 @@ sub renew_item {
$self->{branchcode},
undef,
undef,
undef,
0
);
return {

4
t/db_dependent/Koha/Account/Line.t
View File

@ -568,10 +568,10 @@ subtest 'Renewal related tests' => sub {
);
my $called = 0;
my $module = Test::MockModule->new('C4::Circulation');
$module->mock('AddRenewal', sub { $called = 1; });
$module->mock('CanBookBeRenewed', sub { return 1; });
$line->renew_item;
is( $called, 1, 'Attempt to renew succeeds when conditions are met' );
my $r = Koha::Checkouts::Renewals->find({ checkout_id => $issue->id });
is( $r->seen, 0, "RenewAccruingItemWhenPaid triggers an unseen renewal" );
$schema->storage->txn_rollback;
};