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
feat(server): migrate nested paths may be root paths rule to use context object to support OpenAPI 3 #829
feat(server): migrate nested paths may be root paths rule to use context object to support OpenAPI 3 #829
Conversation
…ext object to support OpenAPI 3 (#714)
Codecov Report
@@ Coverage Diff @@
## master #829 +/- ##
============================================
+ Coverage 84.91% 84.95% +0.03%
- Complexity 902 904 +2
============================================
Files 171 171
Lines 2553 2553
Branches 392 392
============================================
+ Hits 2168 2169 +1
Misses 151 151
+ Partials 234 233 -1
Continue to review full report at Codecov.
|
val spec = """ | ||
openapi: 3.0.1 | ||
paths: | ||
pets/: {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing check for /pets/dogs
. This is a legal pattern and should be interpreted as top level resource.
val spec = """ | ||
openapi: 3.0.1 | ||
paths: | ||
"/countries/{country-id}/cities/{city-id}": {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should also check for something like /countries/{country-id}/populated/cities/{city-id}
.
@tkrop @ddufourboivin-zalando please have another look |
👍 |
👍 |
Related to #714