Reworked URI params trait spec/tests.

jdiegodcp / ramlfications

Python parser for RAML

https://ramlfications.readthedocs.org

Apache License 2.0

234 stars 50 forks source link

Reworked URI params trait spec/tests. #57

Closed matiasb closed 8 years ago

matiasb commented 8 years ago

While looking at #37, I ran into this situation that I think shouldn't be allowed. According to RAML spec, http://raml.org/spec.html#uri-parameters, URI parameters should be present in the URI but their description via uriParameters property is optional (what leads me to #56). If that makes sense, I think the complete-valid-example spec should be changed to be really valid? :-) Updating spec and tests here.

econchick commented 8 years ago

My apologies for being so late on this! Thank you so much for this patch. I'm also working on #37 and #56 today as well, just an FYI.

matiasb commented 8 years ago

Great, no problem, and thanks a lot for looking at (all) this!