From 31f850cb58d72846abd8b37454e17fa9d21c49f5 Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Fri, 22 Sep 2023 11:32:19 -0300 Subject: [PATCH] Bug 32496: (follow-up) Fix t/db_dependent/SIP/Message.t Signed-off-by: Tomas Cohen Arazi --- t/db_dependent/SIP/Message.t | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/t/db_dependent/SIP/Message.t b/t/db_dependent/SIP/Message.t index a8b4ef70bb..2a105189d3 100755 --- a/t/db_dependent/SIP/Message.t +++ b/t/db_dependent/SIP/Message.t @@ -246,13 +246,13 @@ subtest "Test patron_status_string" => sub { my $builder = t::lib::TestBuilder->new(); my $branchcode = $builder->build({ source => 'Branch' })->{branchcode}; - my $patron = $builder->build({ - source => 'Borrower', + my $patron = $builder->build_object({ + class => 'Koha::Patrons', value => { branchcode => $branchcode, }, }); - my $sip_patron = C4::SIP::ILS::Patron->new( $patron->{cardnumber} ); + my $sip_patron = C4::SIP::ILS::Patron->new( $patron->cardnumber ); t::lib::Mocks::mock_userenv({ branchcode => $branchcode }); @@ -282,12 +282,21 @@ subtest "Test patron_status_string" => sub { ); AddIssue( $patron, $item2->barcode ); - is( Koha::Checkouts->search({ borrowernumber => $patron->{borrowernumber} })->count, 2, "Found 2 checkouts for this patron" ); + is( + Koha::Checkouts->search( { borrowernumber => $patron->borrowernumber } )->count, 2, + "Found 2 checkouts for this patron" + ); $item1->itemlost(1)->store(); $item2->itemlost(2)->store(); - is( Koha::Checkouts->search({ borrowernumber => $patron->{borrowernumber}, 'itemlost' => { '>', 0 } }, { join => 'item'} )->count, 2, "Found 2 lost checkouts for this patron" ); + is( + Koha::Checkouts->search( + { borrowernumber => $patron->borrowernumber, 'itemlost' => { '>', 0 } }, { join => 'item' } + )->count, + 2, + "Found 2 lost checkouts for this patron" + ); my $server->{account}->{lost_block_checkout} = undef; my $patron_status_string = C4::SIP::Sip::MsgType::patron_status_string( $sip_patron, $server ); -- 2.39.2