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.

Sign up for GitHub

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

Jump to bottom

Fix Status Code of Type-Level $evaluate-measure Operation #512

Closed
alexanderkiel opened this issue Oct 29, 2021 · 1 comment
Closed

Fix Status Code of Type-Level $evaluate-measure Operation #512

alexanderkiel opened this issue Oct 29, 2021 · 1 comment
Assignees
@allentiak
Labels
bug Something isn't working good first issue Good for newcomers module:evaluate-measure
Milestone
v0.30.0

Comments

@alexanderkiel
Copy link
Member

alexanderkiel commented Oct 29, 2021 •
edited
Loading

A request to [base]/Measure/$evaluate-measure returns a 404 if the measure was not found. However it should return a 422 Unprocessable Entity on POST requests and a 400 Bad Request on GET requests because the resource itself exists.
@alexanderkiel alexanderkiel added bug Something isn't working good first issue Good for newcomers labels Oct 29, 2021
@allentiak allentiak self-assigned this Aug 19, 2024
@alexanderkiel alexanderkiel added this to the v0.30.0 milestone Aug 29, 2024
@allentiak allentiak mentioned this issue Sep 3, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Sep 3, 2024
@allentiak @alexanderkiel
Make sure we return 422 on non-existing measure POST requests (Fixes #…
bedf203 
…512)

module: operation-evaluate-measure
* apply parinfer-required formatting
* add failing POST regression test for #512
* major refactoring: tests: separate handler instance tests from handler
type tests (WIP)
* add missing test for "with missing measure parameter" POST requests
* other minor refactorings
@allentiak
Copy link
Member

Fixed by bedf203.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@allentiak allentiak

Labels
bug Something isn't working good first issue Good for newcomers module:evaluate-measure
Projects
None yet
Milestone
v0.30.0
Development

No branches or pull requests
2 participants
@alexanderkiel @allentiak