@@ -, +, @@ $ DB_IMAGE=mysqli:8.0 ktd up -d # wait until it finished: $ ktd --logs $ ktd --shell k$ prove t/db_dependent/api/v1/ill_requests.t (repeat a few times) --- t/db_dependent/api/v1/ill_requests.t | 29 ++++++++++++++++++++++++---- 1 file changed, 25 insertions(+), 4 deletions(-) --- a/t/db_dependent/api/v1/ill_requests.t +++ a/t/db_dependent/api/v1/ill_requests.t @@ -139,9 +139,30 @@ subtest 'list() tests' => sub { }; # Create some ILL requests - my $req_1 = $builder->build_object({ class => 'Koha::Illrequests', value => { borrowernumber => $patron->borrowernumber, status => $request_status->{code}, backend =>$backend->name } }); - my $req_2 = $builder->build_object({ class => 'Koha::Illrequests', value => { status => $request_status->{code}, backend =>$backend->name , status_alias => $av->authorised_value } } ); - my $ret = $builder->build_object({ class => 'Koha::Illrequests', value => { status => 'RET' } } ); + my $req_1 = $builder->build_object( + { + class => 'Koha::Illrequests', + value => { + borrowernumber => $patron->borrowernumber, + status => $request_status->{code}, + backend => $backend->name, + notesstaff => '1' + } + } + ); + my $req_2 = $builder->build_object( + { + class => 'Koha::Illrequests', + value => { + status => $request_status->{code}, + backend => $backend->name, + status_alias => $av->authorised_value, + notesstaff => '2' + } + + } + ); + my $ret = $builder->build_object({ class => 'Koha::Illrequests', value => { status => 'RET' } }); # Three requests exist, expect all three to be returned $t->get_ok("//$userid:$password@/api/v1/ill/requests") @@ -209,7 +230,7 @@ subtest 'list() tests' => sub { # Hide 'RET', expect to return 2 'REQ' t::lib::Mocks::mock_preference( 'ILLHiddenRequestStatuses', 'RET' ); - $t->get_ok( "//$userid:$password@/api/v1/ill/requests" ) + $t->get_ok( "//$userid:$password@/api/v1/ill/requests?_order_by=staff_notes" ) ->status_is(200) ->json_is( [ $req_1->to_api, $req_2->to_api ]); --