Wrong result for JAX-RS Response with Generic Collection of enum about jaxrs-analyzer HOT 9 CLOSED

Generic List<class> of classes creates correct Result ( case TypeRepresentation.ConcreteTypeRepresentation )
But if Generic List<enum> holds "enum" , result is wrong.
If Rest Services returns just "enum" Type and not a generic Collection of "enum" like List<enum>, result is correct.
It looks like the case with TypeRepresentation.EnumTypeRepresentation and generic Collection of "enum" is not working.

from jaxrs-analyzer.

Using javax.ws.rs.core.GenericEntity<List<enum>> for JAX-RS Service Response also produce wrong Result, just empty reference... in swagger yaml/json

from jaxrs-analyzer.

jars-analyzer version 0.16, running with -b swagger

from jaxrs-analyzer.

Ok, that's interesting... Yes, I think your assumption with EnumTypeRepresentation goes into the right direction. I don't know if I can find the time to look into it within the next days, but please feel free to verify it using some tests in the mean time :) Have you seen the https://github.com/sdaschner/jaxrs-analyzer-st project? That helps in verifying local builds.

from jaxrs-analyzer.

changing visit(final TypeRepresentation.EnumTypeRepresentation representation) in SchemaBuilder has fixed the problem.
Handling in case if inCollection was not present like in visit(final TypeRepresentation.ConcreteTypeRepresentation representation) :-)

from jaxrs-analyzer.

@Override
public void visit(final TypeRepresentation.EnumTypeRepresentation representation) {
builder.add("type", "string");
if (!representation.getEnumValues().isEmpty()) {
final JsonArrayBuilder array = representation.getEnumValues().stream().sorted().collect(Json::createArrayBuilder, JsonArrayBuilder::add, JsonArrayBuilder::add);
//
if (inCollection) {
builder.add("items", Json.createObjectBuilder().add("type", "string").add("enum", array).build());
} else {
builder.add("enum", array);
}
//
}
}

from jaxrs-analyzer.

basically last if else fix it from code above

from jaxrs-analyzer.

#165

Successfully tested with quite complex swagger contract :-)

from jaxrs-analyzer.

from jaxrs-analyzer.