Skip to content

Add option for generated enum types#31

Open
gslaszlo wants to merge 7 commits intografana:mainfrom
gslaszlo:main
Open

Add option for generated enum types#31
gslaszlo wants to merge 7 commits intografana:mainfrom
gslaszlo:main

Conversation

@gslaszlo
Copy link
Copy Markdown

Add support for generated enum types, as in orval docs.

@gslaszlo gslaszlo requested a review from a team as a code owner July 25, 2025 08:14
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jul 25, 2025

CLA assistant check
All committers have signed the CLA.

@DefCon-007
Copy link
Copy Markdown
Collaborator

@gslaszlo Thanks a lot for the changes. The changes look good. I will try to test it locally once, meanwhile, can you add some unit tests for this? We can add some functional tests like this, https://github.com/grafana/openapi-to-k6/blob/main/tests/functional-tests/test-generator/generator.test.ts, we could take a single schema and test it out for all the supported enum types.

@gslaszlo
Copy link
Copy Markdown
Author

gslaszlo commented Aug 7, 2025

@DefCon-007 Thanks for pointing it out, I've added tests for all 3 cases. I didn't want to do a lot of edits in the generator.test.ts so I added 3 fixtures with different expected_sdk params, but if you prefer to add more code instead of fixtures, I can do that as well.

@DefCon-007
Copy link
Copy Markdown
Collaborator

@gslaszlo Could you fix the conflicts? We are good to merge once they are fixed.

@gslaszlo
Copy link
Copy Markdown
Author

gslaszlo commented Sep 3, 2025

@DefCon-007 Sorry for the delay, fixed the conflicts.

@facundobatista facundobatista removed their request for review February 9, 2026 11:09
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.

3 participants