jyapayne/compose
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Improve error message for type constraints

Browse source 
It was missing a space between the different types, when there
were 3 possible type values.

Signed-off-by: Mazz Mosley <mazz@houseofmnowster.com>
This commit is contained in:
Mazz Mosley 2015-10-19 14:36:56 +01:00
commit ff83c459d0
1 changed files with 1 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

3
compose/config/validation.py
View file

@ -143,7 +143,7 @@ def process_errors(errors, service_name=None):
if len(validator) >= 2: if len(validator) >= 2:
first_type = anglicize_validator(validator[0]) first_type = anglicize_validator(validator[0])
last_type = anglicize_validator(validator[-1]) last_type = anglicize_validator(validator[-1])
types_from_validator = "{}{}".format(first_type, ", ".join(validator[1:-1])) types_from_validator = "{}".format(", ".join([first_type] + validator[1:-1]))
msg = "{} or {}".format( msg = "{} or {}".format(
types_from_validator, types_from_validator,
@ -163,7 +163,6 @@ def process_errors(errors, service_name=None):
Inspecting the context value of a ValidationError gives us information about Inspecting the context value of a ValidationError gives us information about
which sub schema failed and which kind of error it is. which sub schema failed and which kind of error it is.
""" """
required = [context for context in error.context if context.validator == 'required'] required = [context for context in error.context if context.validator == 'required']
if required: if required:
return required[0].message return required[0].message