Opened 6 years ago

Last modified 6 years ago

#1802 closed defect

wcst_import should report an error on invalid option — at Version 4

Reported by: Dimitar Misev Owned by: ahambasan
Priority: critical Milestone: 9.7
Component: wcst_import Version: development
Keywords: Cc: Vlad Merticariu
Complexity: Medium

Description (last modified by Dimitar Misev)

If a user specifies an option that is not recognized by wcst_import, then wcst_import should throw an error instead of ignoring the option.

Furthermore, validating options seems to be duplicated in each recipe now. For common options this should be refactored into methods in the BaseRecipe class.

The code to validate ingredients files exist in several recipes files, method validate() in applications/wcst_import/recipes/*/recipe.py

Change History (4)

comment:1 by Dimitar Misev, 6 years ago

Description: modified (diff)

comment:2 by Dimitar Misev, 6 years ago

Perhaps something like this would be useful for the validation: https://pypi.org/project/jsonschema/

comment:3 by Bang Pham Huu, 6 years ago

The code to validate ingredients files exist in several recipes files, check validate() in recipe.py:

applications/wcst_import/recipes/*/recipe.py

comment:4 by Dimitar Misev, 6 years ago

Description: modified (diff)
Owner: changed from Bang Pham Huu to ahambasan
Status: newassigned
Note: See TracTickets for help on using tickets.