fix: request bodies should take required property into account

[AllieJonsson]

Status

READY

Description

Fixes #1111

Takes the required property of request bodies into account. We till make it required by default, but setting it to required: false will make it optional.

Todos

• Tests
• Documentation
• Changelog Entry (unreleased)

Steps to Test or Reproduce

1. yarn generate:swr
2. check tests/generated/swr/optional-request-body/endpoints.ts. Endpoints without a required property has the request body required. Endpoints with a required: false property has the request body as optional.

[soartec-lab]

@AffeJonsson
Hi, Thank you for great PR, I commented on one consultation, so please check it.

[soartec-lab]

We will continue to maintain the tests, so we would like to keep them to a minimum as much as possible. Is it possible to slim down to a minimized case that only checks the request body?

[AllieJonsson]

Absolutely, I'll fix!

[anymaniax]

I understand that in the specification it’s optional per default but 99% if you have a request body you want it required no?

[AllieJonsson]

That's probably true. Maybe it would be better if it's required as default. I'll change that, but at least with this change it'll take the required prop into account

[soartec-lab]

Thank you.