Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Better error raising in object unmarshalling and initialization #5

Open
4 tasks
sambarnes opened this issue Jul 12, 2019 · 0 comments
Open
4 tasks

Comments

@sambarnes
Copy link
Owner

sambarnes commented Jul 12, 2019

Tons of TODOs scattered around the models and their constructors. Raise errors for things that are illegal completely and always, not just invalid according to blockchain parsing rules.

  • investigate how other large python projects handle this (assertions vs raising ValueError vs ...)
  • determine error classes
  • type and basic value assertions in all constructors
  • create unit tests for these cases to catch
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant