Created by: jacob-pro
This removes the unmaintained samples and specs for the Rust client which are no longer automatically built.
The 2 test cases, TypeTesting and fileResponse have been merged into the main Rust spec (modules/openapi-generator/src/test/resources/3_0/rust/petstore.yaml
).
See slack thread: https://openapi-generator.slack.com/archives/CLSB0U0R5/p1660567811900359?thread_ts=1660508979.486299&cid=CLSB0U0R5
The rust-reqwest-petstore-async
and rust-reqwest-petstore-awsv4signature
have also been changed to use the rust specific spec.
I have added a unit test for the TypeTesting case, currently in reqwest/petstore
. Of the 4 different samples I am not sure if this is the correct one to put this test in?
An obsevation concerning the fileResponse test, it looks to me like the generated code for file download is completely broken, for both reqwest and hyper, since they are both attempting to deserialize the response as JSON! So this will need to be fixed separately. Edit: already reported here: https://github.com/OpenAPITools/openapi-generator/pull/9021
Another observation is that despite being configured with supportAsync: "false"
, the hyper client generator is producing asynchronous code (using futures)... Two issues here: Hyper doesn't have a blocking API, so this combination shouldn't be allowed. The generated code should be using the more modern async syntax rather than directly returning Futures
cc: @wing328 @spacether