Skip to content

Commit

Permalink
Switch from ament_cmake_nose to ament_cmake_pytest (#628)
Browse files Browse the repository at this point in the history
Due to nose-devs/nose#1083, the XML output of nose is incompatible with the ROS build farm. Switching to pytest should resolve the issue. Thanks for the tip @cottsay!
  • Loading branch information
jtbandes authored Aug 21, 2021
1 parent 0a1a947 commit e52514a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions rosapi/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,6 @@ install(
)

if(BUILD_TESTING)
find_package(ament_cmake_nose REQUIRED)
ament_add_nose_test(${PROJECT_NAME}_test_stringify_field_types test/test_stringify_field_types.py)
find_package(ament_cmake_pytest REQUIRED)
ament_add_pytest_test(${PROJECT_NAME}_test_stringify_field_types test/test_stringify_field_types.py)
endif()
2 changes: 1 addition & 1 deletion rosapi/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
-->
<exec_depend>rosidl_default_runtime</exec_depend>

<test_depend>ament_cmake_nose</test_depend>
<test_depend>ament_cmake_pytest</test_depend>
<test_depend>sensor_msgs</test_depend>
<test_depend>shape_msgs</test_depend>
<test_depend>geometry_msgs</test_depend>
Expand Down

0 comments on commit e52514a

Please sign in to comment.