feat: schema in model is now optional (#23)

If you don't have any schema you can omit it (e.g. for binary files).

closes #19

README.md

@@ -88,11 +88,11 @@ need the path of the described resource and models need the name of the model.
 ### Define the models
 
 Models have additional information you have to define. Besides the model name, the description and
-the summary, you need to define the *content type* for this model in addition to the *schema* that describes
-the model. Both are mandatory:
+the summary, you need to define the *content type* for this model in addition to the *schema* that
+describes the model:
 
 * `contentType`: the content type of the described request/response (like `"application/json"` or
-`"application/xml"`).
+`"application/xml"`). This is mandatory.
 * `schema`: The JSON Schema that describes the model. In the examples below external files are
 imported, but you can also define the schema inline using YAML format.
 

src/index.spec.js

@@ -107,6 +107,13 @@ describe('ServerlessAWSDocumentation', function () {
     });
 
     it('should add models but not add them to http events', function () {
+      // also add a model with no schema
+      this.serverlessMock.variables.service.custom.documentation.models.push({
+        name: 'NoSchemaModel',
+        contentType: 'application/json',
+        description: 'the other test model schema',
+      });
+
       this.plugin.beforeDeploy();
       expect(this.serverlessMock.service.provider.compiledCloudFormationTemplate).toEqual({
         Resources: {
@@ -132,6 +139,17 @@ describe('ServerlessAWSDocumentation', function () {
               Schema: 'some even more complex schema',
             },
           },
+          NoSchemaModelModel: {
+            Type: 'AWS::ApiGateway::Model',
+            Properties: {
+              RestApiId: {
+                Ref: 'ApiGatewayRestApi',
+              },
+              ContentType: 'application/json',
+              Name: 'NoSchemaModel',
+              Schema: {},
+            },
+          },
           ExistingResource: {
             with: 'configuration',
           },

src/models.js

@@ -10,7 +10,7 @@ module.exports = {
         },
         ContentType: model.contentType,
         Name: model.name,
-        Schema: model.schema,
+        Schema: model.schema || {},
       },
     };
   },