Skip to content

Implement type checking w/in the EFIToolsKBaseImpl.py and wrapper codes #42

@rbdavid

Description

@rbdavid

At the end of each EFIToolsKBase class method is this code:

# At some point might do deeper type checking...
if not isinstance(output, dict):
    raise ValueError('Method run_EFIToolsKBase return value output is not type dict as required.')

I feel like I read somewhere (struggling to find where though) that type checking of input and output is strongly suggested for KBase development. Currently these class methods do no checking of key input/outputs, so very few errors will be detected. Albeit, the code should not be producing incorrectly typed values, its still good practice to do some checking.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions