search

OpenCyphal / pydsdl

Cyphal DSDL processing front end implemented in Python
https://opencyphal.org
MIT License
11 stars 9 forks source link

Adding better error messaging for port ID check #17

Closed thirtytwobits closed 5 years ago

thirtytwobits commented 5 years ago

Problem: There's not enough information in the InvalidFixedPortIDError to diagnose the real error.

Solution: Include if the ID was for a service or message type and include the full path. This allows the user to see which ID range they violated or if this is just a path error.

Testing: Fed invalid identifiers and verified that the message is more informative.