redskap / swagger-brake

Swagger contract checker for breaking API changes
Apache License 2.0
58 stars 16 forks source link

WIP: Parameter via $ref and allOf fix #32

Closed Vity01 closed 4 years ago

Vity01 commented 4 years ago

The PR fixes 2 problems: 1) NPE when using multiple same named parameters from different objects (integrated via allOf) 2) NPE when request parameter defined with $ref is not loaded

unit test attached describes the problems

Vity01 commented 4 years ago

I have added another unit test for which it fails. SB cannot handle empty (aka 204) responses without content/schema.