robotology / robot-testing-framework

Robot Testing Framework (RTF)
http://robotology.github.io/robot-testing-framework/index.html
GNU Lesser General Public License v2.1
19 stars 11 forks source link

CTest integration v2 #57

Closed traversaro closed 6 years ago

traversaro commented 7 years ago

To simplify adding tests using testrunner to CTest using the add_test cmake command, the CMake infrastructure of RTF should provide a variable containing the absolute path to the testrunner binary.

Helper CMake functions to automatically add a CTest test from a suite would also be convenient.

cc @S-Dafarra

traversaro commented 7 years ago

Use case : https://github.com/S-Dafarra/Operational_Control_with_iCub/pull/1/files#diff-15547c54d3d4898a882b4ab7b3cee381R29 .

apaikan commented 7 years ago

I have added some simple tests which can be run using testrunner via make test (see testing branch) . @traversaro and @drdanz we can elaborate that!

S-Dafarra commented 7 years ago

I had an issue using make test: the simulator window appeared and disappeared immediately, making the test fail due to fixtures problems. This issue didn't come out using ctest though.

traversaro commented 7 years ago

cc @aerydna @drdanz

traversaro commented 7 years ago

Fixed by https://github.com/robotology/robot-testing/pull/88 .

apaikan commented 6 years ago

@traversaro is this issue fixed? :p

drdanz commented 6 years ago

We now have a rtf_add_suite CMake command, perhaps also an rtf_add_testcase could be useful? Otherwise I think this could be closed.

traversaro commented 6 years ago

I think the rtf_add_suite is enough unless we actually need a rtf_add_testcase in a real-world scenario.