From 2d449bbdc8331cb34ee736b6e1b6429f6adc63f3 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Tue, 26 Sep 2023 13:45:35 +0000 Subject: [PATCH] Bug 34916: Fix wrong borrowernumber in ArticleRequests.t Content-Type: text/plain; charset=utf-8 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 --- t/db_dependent/Koha/ArticleRequests.t | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/t/db_dependent/Koha/ArticleRequests.t b/t/db_dependent/Koha/ArticleRequests.t index 98b92a3d40..1b240b6524 100755 --- a/t/db_dependent/Koha/ArticleRequests.t +++ b/t/db_dependent/Koha/ArticleRequests.t @@ -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 -- 2.30.2