While working on examples I noticed that arrays are not handled correctly by default

[michaelgwelch]

As I work on making #82 more robust and after running the tests, I've observed that form arrays are not being handled properly. I'll add the links but the 3.0 specs says that if not specified then explode defaults to true for form parameters.

As part of #82, I will be trying to address form and simple style parameters for primitives, arrays, and objects and obeying the explode setting.

3 tests are passing that should be failing:

• Testing the case when an example is provided, use the provided example value

  This test is looking at this parameter: https://github.com/ErikWittern/openapi-snippet/blob/2e6c2c91859c56442f134ced22ef609e7b4b36cd/test/parameter_example_swagger.json#L29-L42

  Note that it is a form parameter with no value for explode. Per the spec explode defaults to true for form parameters. So this line of code:

  https://github.com/ErikWittern/openapi-snippet/blob/2e6c2c91859c56442f134ced22ef609e7b4b36cd/test/test.js#L198

  Should actually be looking for tags: ['dog', 'cat'] since there are an array of values. This will then be rendered in an actual request as tags=dog&tags=cat.

• Query Params Defined for all methods should be resolved

  this test https://github.com/ErikWittern/openapi-snippet/blob/2e6c2c91859c56442f134ced22ef609e7b4b36cd/test/test.js#L278-L289

  is looking at this parameter:

  https://github.com/ErikWittern/openapi-snippet/blob/2e6c2c91859c56442f134ced22ef609e7b4b36cd/test/parameter_example_swagger.json#L81-L94

  which again is a form parameter without a value for explode so explode should default to true.

• Query Params Defined for all methods are overriden by method definitions

  Same issue. Test is here:

  https://github.com/ErikWittern/openapi-snippet/blob/2e6c2c91859c56442f134ced22ef609e7b4b36cd/test/test.js#L291-L301

  Referencing this parameter

  https://github.com/ErikWittern/openapi-snippet/blob/2e6c2c91859c56442f134ced22ef609e7b4b36cd/test/parameter_example_swagger.json#L151-L162

I have fixes for this as part of #82 which I'm still working on.

[michaelgwelch]

A simple fix for the test cases is to just explicitly set explode to false for each of these parameters. Then #86 adds support for exploding objects and arrays and tests all the different variations.

Fixed by #86