New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add missing validation message for property types #36

Merged

RobWin merged 1 commit into RobWin:master from marcelstoer:fix/missing-validation-message

Feb 23, 2018

Contributor

marcelstoer commented Feb 22, 2018

This fixes the following - rather useless - validation output:

2) 
Expecting:
 <io.swagger.models.properties.IntegerProperty@d7e9e510>
to be exactly an instance of:
 <io.swagger.models.properties.BaseIntegerProperty>
but was an instance of:
 <io.swagger.models.properties.IntegerProperty>

The fix will add the missing context message:

2) [Checking property '<actual-property-name>' of definition '<definition-name>'] 
Expecting:
 <io.swagger.models.properties.IntegerProperty@d7e9e510>
to be exactly an instance of:
 <io.swagger.models.properties.BaseIntegerProperty>
but was an instance of:
 <io.swagger.models.properties.IntegerProperty>


          Add missing validation message for property types

c53d65a

coveralls commented Feb 22, 2018

Coverage remained the same at 87.637% when pulling c53d65a on marcelstoer:fix/missing-validation-message into 6f2ea68 on RobWin:master.

RobWin merged commit dcaf45e into RobWin:master

marcelstoer deleted the fix/missing-validation-message branch

February 23, 2018 12:32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet