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

Implement OpenAPIVerifier #4215

Merged
merged 5 commits into from
Dec 23, 2024
Merged

Conversation

abdelfetah18
Copy link
Contributor

@abdelfetah18 abdelfetah18 commented Dec 19, 2024

This implementation of OpenAPIVerifier is fairly basic and supports three modes: Exact, AtLeast, and AtMost.

The verify method returns a list of issues. If the list is empty, it indicates a valid verification.

/claim #3645

@adamw
Copy link
Member

adamw commented Dec 20, 2024

Thanks! Can we move the functionality to a new module? We can't (shouldn't) add a circe dependency to the main openapi docs one.

Also, can you add a mention of the feature to the docs? I think a new section in https://tapir.softwaremill.com/en/latest/testing.html#endpoints-verification would be best

@abdelfetah18 abdelfetah18 force-pushed the openapi-verifier branch 2 times, most recently from 1d1c5f2 to d9c49b1 Compare December 20, 2024 19:43
@adamw adamw merged commit b494b1b into softwaremill:master Dec 23, 2024
28 checks passed
@adamw
Copy link
Member

adamw commented Dec 23, 2024

Thanks!

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

Successfully merging this pull request may close these issues.

2 participants