| Summary: | All list endpoints should have the x-koha-request-id parameter definition | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Tomás Cohen Arazi (tcohen) <tomascohen> |
| Component: | REST API | Assignee: | Bugs List <koha-bugs> |
| Status: | NEW --- | QA Contact: | Testopia <testopia> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | tomascohen |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: | Bug 39817: Add some GET routes tests | ||
|
Description
Tomás Cohen Arazi (tcohen)
2025-05-02 13:18:02 UTC
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> |