rock-core / tools-syskit

Rock's model-based component management layer
1 stars 14 forks source link

fix: greatly simplify test error messages #365

Closed doudou closed 1 year ago

doudou commented 1 year ago

Depends on:

On top of #366

The most dramatic change is 6866df6522f9a8f7a66c668942e0b9529d3e904c, which removes the bulk of the error messages, which were related to how syskit_deploy works and not the test. The other really good change is the addition of a better explanation as to why two tasks did not get merged. I made the change for the "device is attached to two tasks" errors, but not yet to other relevant errors (e.g. missing deployments).

Along with the Roby change, it turned an error that was pages (multiple, easily 4-5) long into a single page with a single backtrace and actually actionable information.