Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 34916: Fix wrong borrowernumber in ArticleRequests.t

Browse source 
Simple fix: Send patron to userenv instead of library only.

Test plan:
Do not apply patch.
Remove borrowernumber 51.
Run t/db_dependent/Koha/ArticleRequests.t. (FAIL)
Apply patch, rerun test (PASS)

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Marcel de Rooy 2023-09-26 13:45:35 +00:00 committed by Tomas Cohen Arazi
parent ff00e77f46
commit 7d93d18548
Signed by: tomascohen
GPG key ID: 0A272EA1B2F3C15F
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
t/db_dependent/Koha/ArticleRequests.t
View file

@ -40,7 +40,7 @@ subtest 'requested() tests' => sub {
my $library_2 = $builder->build_object( { class => 'Koha::Libraries' } );
my $patron = $builder->build_object( { class => 'Koha::Patrons', value => { branchcode => $library_1->id, flags => 1 } } );
t::lib::Mocks::mock_userenv( { branchcode => $library_1 } );
t::lib::Mocks::mock_userenv( { patron => $patron } );
# FIXME: we moved past this pattern. This method should be refactored
# as ->filter_by_requested