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

[BUG] [Java] Default Java client does not compile #3094

Closed
rizwansaeed opened this issue Jun 5, 2019 · 1 comment

Comments

Projects
None yet
1 participant
@rizwansaeed
Copy link

commented Jun 5, 2019

Bug Report Checklist

  • Have you provided a full/minimal spec to reproduce the issue?

  • Have you validated the input using an OpenAPI validator (example)?

  • What's the version of OpenAPI Generator used?

  • Have you search for related issues/PRs?

  • What's the actual output vs expected output?
    Generated client successfully compiles, currently fails due to missing imports

Description

Generating a Java client using the default options produces code that does not compile. A sample swagger file is provided that reproduces the issue. It appears that some model classes are not included as required dependencies, manually adding them (as imports to the generated code) resolves the issue. Note that the sample is only a subset of the full swagger file, other model and API classes generate successfully but there appears to be something in this example that causes it not work.

Also note that the C# client generates successfully which suggests that the swagger file itself is ok

openapi-generator version

4.0.0-SNAPSHOT (via Docker image)

OpenAPI declaration file content or url

https://gist.github.com/rizwansaeed/a8413c9dd55715b1a6e71fce93f2f89c

Command line used for generation
docker run --rm -v ${PWD}:/local openapitools/openapi-generator-cli generate \
    -i /local/sample.json \
    -g java \
    -o /local/out/src
Steps to reproduce
  1. Run the command line above
  2. Try to build the project
Suggest a fix

The generated code should include imports for all dependencies

@auto-labeler

This comment has been minimized.

Copy link

commented Jun 5, 2019

👍 Thanks for opening this issue!
🏷 I have applied any labels matching special text in your issue.

The team will review the labels and make any necessary changes.

@Zomzog Zomzog referenced this issue Jun 8, 2019

Merged

Import inner items for map #3123

4 of 4 tasks complete

Zomzog added a commit to Zomzog/openapi-generator that referenced this issue Jun 8, 2019

@wing328 wing328 closed this in #3123 Jun 9, 2019

wing328 added a commit that referenced this issue Jun 9, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.