This is mandatory for using them with our `kohaTable` datatables wrapper. I spotted the `/acquisitions/baskets` endpoint doesn't have it. We should have a test for it.
Created attachment 181869 [details] [review] Bug 39817: Add some GET routes tests This patch adds tests for to-be mandatory parameters for plural endpoints: * match * order_by * page * per_page * q_param * q_body * request_id_header Exceptions should be added if found in the wild, but the tests are needed. Another check it adds is GET requests `operationId` should start with: * list * get To test: 1. Run: $ ktd --shell k$ prove xt/api.t => FAIL: Tests pass! But our spec is bad 2. Apply this patch 3. Repeat 1 => SUCCESS: All mistakes make the tests fail! 4. Sign off :-D Note: a follow-up patch/set of patches should make this tests pass! Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>