Skip to content
This repository has been archived by the owner on Nov 19, 2023. It is now read-only.

requestBody schema testing #291

Open
darduf opened this issue Dec 1, 2022 · 4 comments
Open

requestBody schema testing #291

darduf opened this issue Dec 1, 2022 · 4 comments

Comments

@darduf
Copy link

darduf commented Dec 1, 2022

We noticed in our OAS testing that the requestBody schema is not tested. For example if the post.requestBody is removed from a path (that expects a requestBody) in the OAS, there is no schema checking here and tests pass. Similarly if the requestBody is incorrect, the tests also still pass.

Is there scope to include such a test in this package?

Thanks!

@sondrelg
Copy link
Member

sondrelg commented Dec 1, 2022

Yes, that sounds like it should happen. Since I'm no longer using this package myself, would you want to help maintain the package doing forward @darduf? I could give you push access, and I could help review changes, but let you add features like this, if you're interested?

@darduf
Copy link
Author

darduf commented Dec 1, 2022

Thanks for the offer @sondrelg. I don't currently have the bandwidth to maintain the package, but would be open to contribute to the package as previously done on fixes.

Since I'm no longer using this package myself,

Out of interest, what are you using for this type of OAS testing?

@sondrelg
Copy link
Member

sondrelg commented Dec 2, 2022

I'm no longer working on an API first Django backend, otherwise I likely would be 👍 Otherwise I think schemathesis might provide similar benefits

@maticardenas
Copy link
Contributor

Hi @sondrelg @darduf, as I'm currently using this for my project I also found useful to have requestBody validated within the client.
I decided to add the feature and created a PR for it, would be great if you could give it a check :)

Thanks!

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

No branches or pull requests

3 participants