Add unit test to travis to examine pull request by travis - Githubissues

start-jsk / openrtm_aist_core

Metapackage that contains commonly used RT-Middleware-based modules. Althogh each of these packages by themselves are ROS-agnostic, you can use them over ROS together with the packages in rtmros_common.

1 stars 6 forks source link

Add unit test to travis to examine pull request by travis #47

Closed 130s closed 10 years ago

130s commented 10 years ago

I'm not sure if test_openrtm_aist.test is complete, but does it make sense to add it to travis?

130s commented 10 years ago

I think rostest is already added (like here). Closing.

Now the question is if the test file test_openrtm_aist.test is sufficient or not though.

k-okada commented 10 years ago

note: In the travis, we tested for both 'devel' and 'install', 1) as for devel/src directory, all 'add_rostest()' ( https://github.com/start-jsk/openrtm_aist_core/blob/master/openrtm_aist/catkin.cmake#L96) is tested in the travis https://github.com/start-jsk/openrtm_aist_core/blob/master/.travis.yml#L65, 2) after 'catkin_make install', travis removes devel and src directory and tested again(https://github.com/start-jsk/openrtm_aist_core/blob/master/.travis.yml#L70), at this time we only tested the file that has '.test' in the suffix. see http://answers.ros.org/question/131646/how-do-i-test-code-agenst-catkin-install-ed-directory/ for discussion.