From fa63dee8f99fe7654c5e7db719c1e72c4dfc18ed Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Thu, 21 Feb 2019 18:03:21 -0300 Subject: [PATCH] Bug 22049: Do not pass Koha::Patron to AddIssue The tests are not supposed to pass with such obvious mistake (?) Signed-off-by: Jonathan Druart --- t/db_dependent/Circulation/MarkIssueReturned.t | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/t/db_dependent/Circulation/MarkIssueReturned.t b/t/db_dependent/Circulation/MarkIssueReturned.t index 8b9a6f9ea6..70924bca39 100644 --- a/t/db_dependent/Circulation/MarkIssueReturned.t +++ b/t/db_dependent/Circulation/MarkIssueReturned.t @@ -69,7 +69,7 @@ subtest 'Failure tests' => sub { is( $issue_id, undef, 'No issue_id returned' ); # In the next call we return the item and try it another time - $issue = C4::Circulation::AddIssue( $patron, $item->barcode ); + $issue = C4::Circulation::AddIssue( $patron->unblessed, $item->barcode ); eval { $issue_id = C4::Circulation::MarkIssueReturned( $patron->borrowernumber, $item->itemnumber, undef, 0 ) }; is( $issue_id, $issue->issue_id, "Item has been returned (issue $issue_id)" ); eval { $issue_id = C4::Circulation::MarkIssueReturned( $patron->borrowernumber, $item->itemnumber, undef, 0 ) }; @@ -172,7 +172,7 @@ subtest 'Manually pass a return date' => sub { my $old_checkout = Koha::Old::Checkouts->find( $issue_id ); is( $old_checkout->returndate, '2018-12-25 00:00:00', 'Manually passed date stored correctly' ); - $issue = C4::Circulation::AddIssue( $patron, $item->barcode ); + $issue = C4::Circulation::AddIssue( $patron->unblessed, $item->barcode ); throws_ok { $issue_id = C4::Circulation::MarkIssueReturned( $patron->borrowernumber, $item->itemnumber, 'bad_date', 0 ); } -- 2.11.0