fix OpenAPI resource generation

tangiel · tangiel · commit 16073efd6d7b · 2016-12-16T11:29:38.000-08:00
Resources must have names in the OpenAPI spec. This change names all
request resources "body" in the spec.
diff --git a/endpoints-framework/src/main/java/com/google/api/server/spi/swagger/SwaggerGenerator.java b/endpoints-framework/src/main/java/com/google/api/server/spi/swagger/SwaggerGenerator.java
@@ -258,6 +258,7 @@ private void writeApiMethod(ApiMethodConfig methodConfig, EndpointMethod endpoin
           TypeToken<?> requestType = parameterConfig.getSchemaBaseType();
           Schema schema = repo.getOrAdd(requestType, apiConfig);
           BodyParameter bodyParameter = new BodyParameter();
+          bodyParameter.setName("body");
           bodyParameter.setSchema(new RefModel(schema.name()));
           operation.addParameter(bodyParameter);
           break;
diff --git a/endpoints-framework/src/test/resources/com/google/api/server/spi/swagger/foo_endpoint.swagger b/endpoints-framework/src/test/resources/com/google/api/server/spi/swagger/foo_endpoint.swagger
@@ -109,6 +109,7 @@
             "type": "string"
           },
           {
+            "name": "body",
             "in": "body",
             "required": false,
             "schema": {
@@ -145,6 +146,7 @@
             "type": "string"
           },
           {
+            "name": "body",
             "in": "body",
             "required": false,
             "schema": {
diff --git a/endpoints-framework/src/test/resources/com/google/api/server/spi/swagger/foo_endpoint_default_context.swagger b/endpoints-framework/src/test/resources/com/google/api/server/spi/swagger/foo_endpoint_default_context.swagger
@@ -109,6 +109,7 @@
             "type": "string"
           },
           {
+            "name": "body",
             "in": "body",
             "required": false,
             "schema": {
@@ -145,6 +146,7 @@
             "type": "string"
           },
           {
+            "name": "body",
             "in": "body",
             "required": false,
             "schema": {
diff --git a/endpoints-framework/src/test/resources/com/google/api/server/spi/swagger/foo_endpoint_internal.swagger b/endpoints-framework/src/test/resources/com/google/api/server/spi/swagger/foo_endpoint_internal.swagger
@@ -157,6 +157,7 @@
             "type": "string"
           },
           {
+            "name": "body",
             "in": "body",
             "required": false,
             "schema": {
@@ -209,6 +210,7 @@
             "type": "string"
           },
           {
+            "name": "body",
             "in": "body",
             "required": false,
             "schema": {
diff --git a/endpoints-framework/src/test/resources/com/google/api/server/spi/swagger/foo_endpoint_localhost.swagger b/endpoints-framework/src/test/resources/com/google/api/server/spi/swagger/foo_endpoint_localhost.swagger
@@ -109,6 +109,7 @@
             "type": "string"
           },
           {
+            "name": "body",
             "in": "body",
             "required": false,
             "schema": {
@@ -145,6 +146,7 @@
             "type": "string"
           },
           {
+            "name": "body",
             "in": "body",
             "required": false,
             "schema": {
