Bug 40409: Make Overdues.t fail

The test is wrong, the item is not correctly checked in, it should fail.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit 5c743feec7)
Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de>
This commit is contained in:
Jonathan Druart 2025-07-16 12:32:52 +02:00 committed by Paul Derscheid
parent 1f2dffb43f
commit 9f3c5d2019
No known key found for this signature in database
GPG Key ID: C25F9FD44CED595B
1 changed files with 2 additions and 4 deletions

View File

@ -549,10 +549,8 @@ subtest 'UpdateFine tests' => sub {
{ order_by => { '-asc' => 'accountlines_id' } }
);
is( $fines->count, 4, "New amount should be 0 so no fine added" );
ok(
C4::Circulation::AddReturn( $item_1->barcode, $item_1->homebranch, 1 ),
"Returning the item and forgiving fines succeeds"
);
my @r = C4::Circulation::AddReturn( $item_1->barcode, $item_1->homebranch, 1 );
is( $r[1]->{WasReturned}, 1, "Returning the item and forgiving fines succeeds" );
t::lib::Mocks::mock_preference( 'MaxFine', 0 );