search

ros-industrial-consortium / packml

ROS packml (https://en.wikipedia.org/wiki/PackML) support package
Apache License 2.0
9 stars 14 forks source link

State timing is not covered in unit tests #37

Open shaun-edwards opened 6 years ago

shaun-edwards commented 6 years ago

A recent refactoring of the underlying state machine missed the state timing: https://github.com/ros-industrial-consortium/packml/pull/35#issuecomment-401534099

The regression was almost missed because unit tests do not exist to verify this functionality.