Skip to content
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.

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

repl: fixed prompt_object_options return value type #30

Merged
merged 1 commit into from
Aug 4, 2017

Conversation

pablopalacios
Copy link
Contributor

Previously, repl.helpers.prompt_object_options was returning a dict
with string keys. This was making validation of object option
requirements fail inadvertently. Consequently, prompt of sub
options (eg. format options, install condition configuration, etc)
never happened because they depend on this validation.

A type checking was added into uhu.core.validate_option_requirements
to prevent this kind of bug. This modification made
repl.helpers.prompt_object_options tests fail as expected. Finally, a
proper fix was applied into prompt_object_options.

Signed-off-by: Pablo Palácios ppalacios992@gmail.com

Previously, repl.helpers.prompt_object_options was returning a dict
with string keys. This was making validation of object option
requirements fail inadvertently. Consequently, prompt of sub
options (eg. format options, install condition configuration, etc)
never happened because they depend on this validation.

A type checking was added into uhu.core.validate_option_requirements
to prevent this kind of bug. This modification made
repl.helpers.prompt_object_options tests fail as expected. Finally, a
proper fix was applied into prompt_object_options.

Signed-off-by: Pablo Palácios <ppalacios992@gmail.com>
@coveralls
Copy link

coveralls commented Aug 4, 2017

Coverage Status

Coverage increased (+0.01%) to 94.826% when pulling 5d37cee on pablopalacios:repl into d0a6d8d on UpdateHub:master.

@otavio otavio merged commit b5491c6 into UpdateHub:master Aug 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants