Currently catkin run_tests will return 0 even if there are failing tests (for example see this travis build that had a test failure). This behavior is probably because run_tests is an alias for catkin_make run_tests. It would be nice if catkin run_tests could return a better error code.

In the meantime, I am calling catkin_test_results after catkin run_tests.