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

Added validation for duplicate operationId. #192

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Aribros
Copy link

@Aribros Aribros commented Aug 1, 2023

Added validation for duplicate operationId.

Fix #181

@Aribros Aribros changed the title Added a failing test Added validation for duplicate operationId. Aug 2, 2023
@charjr
Copy link

charjr commented Sep 1, 2023

@Aribros What happens if there are duplicate operationIds in two separate paths?

Unique string used to identify the operation. The id MUST be unique among all operations described in the API.

@Aribros
Copy link
Author

Aribros commented Sep 8, 2023

@Aribros What happens if there are duplicate operationIds in two separate paths?

Unique string used to identify the operation. The id MUST be unique among all operations described in the API.

This has been taken care of. More tests were added to cover duplicate IDs in separate paths .

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

Successfully merging this pull request may close these issues.

Duplicate OperationIds should be considered invalid
2 participants