-
-
Notifications
You must be signed in to change notification settings - Fork 7k
[BUG][SCALAZ] tests are failing with generator 4.2.0 #4072
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
Comments
The issue was that this was never one of our "popular" generators, and so wasn't regenerated and checked on every PR/commit. CI does verify compilation, but the compilation is only against the outputs of the last time the generator was modified. There were issues with the generator (i.e. not handling defaulted values for things like arrays), and issues with other changes (new previously unsupported functionality added to the shared openapi YAML file, Java-only imports in DefaultCodegen). The workaround here would be for users to use openapi-generator 3.0.0 until my fix in #4508 is released. |
I've merged #4508. Can you re-evaluate latest master in your changes? |
I am currently trying to update the branches and fix all the new bugs which were introduced. As soon as the tests are working again I will test it against scala. Thank you for the fix so far |
The fix for this was merged and released in 4.2.2 (see 10f1787) I've updated the milestone back to 4.2.2 because this bug was already documented in our 4.2.2 Release Notes. |
Thank you jimschubert |
Uh oh!
There was an error while loading. Please reload this page.
Bug Report Checklist
Latest master (4.2.0-SNAPSHOT)
Description
Scalaz is not building against the current openapi-generator. The golden files are from openapi-generator-3.0.0 which means that the scalaz tests are testing against invalid code...
Suggest a fix
Fix tests or remove tests / golden files to no longer test this generator in CircleCI
References:
CircleCI run: https://circleci.com/gh/OpenAPITools/openapi-generator/9310?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link
Tested in Pull request #4071 (comment)
The text was updated successfully, but these errors were encountered: