ros-controls / ros_control

Generic and simple controls framework for ROS
http://wiki.ros.org/ros_control
BSD 3-Clause "New" or "Revised" License
470 stars 307 forks source link

Remove irrelevant test case from transmission_interface #460

Closed bmagyar closed 3 years ago

bmagyar commented 4 years ago

We used to host an implementation of is_permutation which got replaced by the standard library alternative once it was out. The test for the old function however was simply refactored instead of removed.

https://github.com/ros-controls/ros_control/blob/noetic-devel/transmission_interface/test/transmission_interface_loader_test.cpp#L63

I suggest we don't test STL functions, let's remove it!