robofit / arcor

Augmented reality-based human-robot interaction.
GNU Lesser General Public License v2.1
8 stars 14 forks source link

Continuous integration (testing) #68

Closed ZdenekM closed 7 years ago

ZdenekM commented 7 years ago

Curently, travis can't check much as there is not much code to compile. It would be good to:

http://wiki.ros.org/unittest https://github.com/hcrlab/wiki/blob/master/software_engineering/unit_testing.md https://github.com/hcrlab/wiki/blob/master/software_engineering/continuous_integration.md http://wiki.ros.org/roslint http://wiki.ros.org/roslaunch (roslaunch_add_file_check)

ZdenekM commented 7 years ago

The roslint test was added to several packages (art_basic_control, art_pr2_grasping, art_projected_gui, art_interface_utils, art_db, art_brain). Others should follow. Moreover, it's not possible to merge PR before tests are finished. If tests (roslint) fails, one can see what's wrong in build log (after_script).

ZdenekM commented 7 years ago

Example unit test created for art_utils/ProgramHelper class (https://github.com/robofit/ar-table-itable/tree/master/art_utils).

ZdenekM commented 7 years ago

launch file testing: #78 + https://github.com/robofit/ar-table-pr2/pull/23

ZdenekM commented 7 years ago

Example rostest for art_db: https://github.com/robofit/ar-table-itable/blob/master/art_db/tests/art_db.test. It works well when launched as rostest art_db art_db.test but not so well when launched using catkin_make run_tests (see http://answers.ros.org/question/250056/rostest-fails-when-run-using-catkin_make-run_tests/) - so it's disabled in CMakeLists.txt for now.

ZdenekM commented 7 years ago

Launching of rostests resolved: https://github.com/ros/catkin/issues/841. Switched to catkin_make_isolated on travis.