Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test for GET /op/definition that provides a jsonPathQuery parameter #1537

Closed
cd-rite opened this issue Feb 20, 2025 · 1 comment
Closed
Assignees
Labels

Comments

@cd-rite
Copy link
Collaborator

cd-rite commented Feb 20, 2025

PR #1531 includes an update to JsonPath module, but I noticed we do not have coverage in the tests for that endpoint when provided with a jsonPathQuery param. The only test expects the default response.

opId: getDefinition

Pass a JsonPathQuery string that refines the response to that portion of the api schema (for example: $.components.schemas.RoleId)
Check that endpoint responds successfully , with proper portion of spec

Note, I informally tested with the new JsonPath update, and it does work, so I'll merge that PR

@cd-rite cd-rite added the tests label Feb 20, 2025
@cd-rite cd-rite self-assigned this Feb 20, 2025
@cd-rite
Copy link
Collaborator Author

cd-rite commented Feb 24, 2025

resolved by: #1539

@cd-rite cd-rite closed this as completed Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant