From de787111b9acee292489c38c66e86aac07d23052 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Tue, 9 Nov 2021 11:38:06 +0000 Subject: [PATCH] Bug 24295: Replace mock in t/db_dependent/Search.t Replace the mock of GetTransfers in this test with a mock of transfers data Test plan 1/ Run t/db_dependent/Search.t 2/ Confirm it passes 3/ Apply patch 4/ Repeat steps 1-2 --- t/db_dependent/Search.t | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/t/db_dependent/Search.t b/t/db_dependent/Search.t index 6a1abe54a3..d2076f6d2d 100755 --- a/t/db_dependent/Search.t +++ b/t/db_dependent/Search.t @@ -31,6 +31,7 @@ use Test::MockModule; use Test::Warn; use t::lib::Mocks; use t::lib::Mocks::Zebra; +use t::lib::TestBuilder; use Koha::Caches; @@ -678,14 +679,18 @@ ok(MARC::Record::new_from_xml($results_hashref->{biblioserver}->{RECORDS}->[0],' # make one of the test items appear to be in transit my $circ_module = Test::MockModule->new('C4::Circulation'); - $circ_module->mock('GetTransfers', sub { - my $itemnumber = shift // -1; - if ($itemnumber == 11) { - return ('2013-07-19', 'MPL', 'CPL'); - } else { - return; + my $builder = t::lib::TestBuilder->new; + my $transfer = $builder->build( + { + source => 'Branchtransfer', + value => { + itemnumber => 11, + frombranch => 'MPL', + tobranch => 'CPL', + datesent => \'NOW()' + } } - }); + ); ($error, $results_hashref, $facets_loop) = getRecords("TEST12121212","TEST12121212",[ ], [ 'biblioserver' ],20,0,\%branches,\%itemtypes,$query_type,0); @newresults = searchResults({'interface'=>'intranet'}, $query_desc, $results_hashref->{'biblioserver'}->{'hits'}, 17, 0, 0, -- 2.20.1