Yelp / bravado-core

Other
109 stars 98 forks source link

Provide more useful error message for response header validation. #347

Closed brycedrennan closed 5 years ago

brycedrennan commented 5 years ago

Previously the header_name was not found in the error, which made it not very informative.

Example: Before: None is not of type 'string' After: None is not of type 'string' for header 'X-Request-Id'

coveralls commented 5 years ago

Coverage Status

Coverage increased (+0.1%) to 99.001% when pulling dc10a2f49ed216316671675b450120c36975516e on BlueOwlDev:header-validation into 4f765c95466cb0a357fa42614fbd855e61ea5d31 on Yelp:master.

brycedrennan commented 5 years ago

These failing tests were already failing in master but I fixed them in #348

brycedrennan commented 5 years ago

@macisamuele I've updated the test to ensure the header name is included in the error message.