diff --git a/modules/swagger-compat-spec-parser/src/main/java/io/swagger/parser/SwaggerCompatConverter.java b/modules/swagger-compat-spec-parser/src/main/java/io/swagger/parser/SwaggerCompatConverter.java index b9969a5570..8eb9e87cbd 100644 --- a/modules/swagger-compat-spec-parser/src/main/java/io/swagger/parser/SwaggerCompatConverter.java +++ b/modules/swagger-compat-spec-parser/src/main/java/io/swagger/parser/SwaggerCompatConverter.java @@ -437,15 +437,13 @@ public Operation convertOperation(String tag, io.swagger.models.apideclaration.O // default response type Property responseProperty = propertyFromTypedObject(operation); - if (responseProperty != null) { - Response response = new Response() - .description("success") - .schema(responseProperty); - if (output.getResponses() == null) { - output.defaultResponse(response); - } else { - output.response(200, response); - } + Response response = new Response() + .description("success") + .schema(responseProperty); + if (output.getResponses() == null) { + output.defaultResponse(response); + } else if (responseProperty != null) { + output.response(200, response); } Map> auths = operation.getAuthorizations(); diff --git a/modules/swagger-compat-spec-parser/src/test/java/io/swagger/converter/LegacyConverterTest.java b/modules/swagger-compat-spec-parser/src/test/java/io/swagger/converter/LegacyConverterTest.java index 0a4b59424b..5b38d21015 100644 --- a/modules/swagger-compat-spec-parser/src/test/java/io/swagger/converter/LegacyConverterTest.java +++ b/modules/swagger-compat-spec-parser/src/test/java/io/swagger/converter/LegacyConverterTest.java @@ -130,7 +130,10 @@ public void testFixedProperties() throws IOException { for (Response item : path.getPost().getResponses().values()) { assertNull(item.getSchema()); } - assertNull(path.getDelete().getResponses()); + assertEquals(path.getDelete().getResponses().size(), 1); + assertEquals(path.getDelete().getResponses().containsKey("default"), true); + assertEquals(path.getDelete().getResponses().get("default").getDescription(), "success"); + final PathParameter id = (PathParameter) Iterables.find(path.getPatch().getParameters(), new Predicate() {