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

rest-api-specs are out of sync with the code base #27158

Closed
olcbean opened this issue Oct 28, 2017 · 1 comment
Closed

rest-api-specs are out of sync with the code base #27158

olcbean opened this issue Oct 28, 2017 · 1 comment

Comments

@olcbean
Copy link
Contributor

olcbean commented Oct 28, 2017

Currently it is very easy for the rest-api-spec to go out of sync with the code base. For example: the code base is changed, the docs are updated but the rest-api-spec still refers to no-longer-existing parameter names (#27124, d6af6697766 and 51d53791fe7). Or a required path part is still wrongly marked as not required (#25988)

Writing explicit tests is error prone - forgetting to add a new parameter or change an existing one is way too common.

We should have a more robust approach to verify that the rest-api-specs remain valid. It is not possible to gurantee that all parameters are exposed to the rest-api-apec but at least we should attempt to check that every parameter in the rest-api-spec can actually be used.

Has this been discussed before / is there any prior art related to this style of tests?

@jasontedor
Copy link
Member

At some point we intend to generate the specification from the server code. This issue would then be addressed by that effort, and I don't think we are going to make a concerted effort now otherwise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants