Merge pull request #373 from lbr-stack/fix/lbr-fri-ros2-stack-372/ament_cmake_pytest_on_test

depend on ament_cmake_pytest for tests only (#372)

Author: mhubii

CHANGELOG.rst

@@ -16,15 +16,15 @@ Jazzy v2.x (TBD)
   * Sets the ``k_velocity`` to fix the safety limits for effort client command modes.
   * Removes the controller-related parts from the ``*_description.xacro`` files, now named ``*.urdf.xacro``.
   * Fix tests.
-
+  * Depend on ``ament_cmake_pytest`` for tests only.
 * ``lbr_ros2_control``: Uses the joint limits as obtained from the joints inside the URDF, previously redundant in ``<command_interface>`` tags. 
 * Related pull requests:
 
   * Namespace argument: https://github.com/lbr-stack/lbr_fri_ros2_stack/pull/344, https://github.com/lbr-stack/lbr_fri_ros2_stack/pull/353
   * Effort limits: https://github.com/lbr-stack/lbr_fri_ros2_stack/pull/357
   * URDF de-coupling: https://github.com/lbr-stack/lbr_fri_ros2_stack/pull/359
   * Ament target dependencies: https://github.com/lbr-stack/lbr_fri_ros2_stack/pull/364
-  * Test fixes: https://github.com/lbr-stack/lbr_fri_ros2_stack/pull/367
+  * Test fixes: https://github.com/lbr-stack/lbr_fri_ros2_stack/pull/367, https://github.com/lbr-stack/lbr_fri_ros2_stack/pull/373
 
 Jazzy v2.4.3 (2025-12-09)
 --------------------------

lbr_description/CMakeLists.txt

@@ -7,7 +7,6 @@ endif()
 
 # find dependencies
 find_package(ament_cmake REQUIRED)
-find_package(ament_cmake_pytest REQUIRED)
 
 # install
 install(
@@ -18,6 +17,7 @@ install(
 ament_environment_hooks("${CMAKE_CURRENT_SOURCE_DIR}/${PROJECT_NAME}.dsv")
 
 if(BUILD_TESTING)
+  find_package(ament_cmake_pytest REQUIRED)
   ament_add_pytest_test(test_urdf ${CMAKE_CURRENT_SOURCE_DIR}/test/test_urdf.py)
 endif()
 

lbr_description/package.xml

@@ -8,7 +8,6 @@
   <license>Apache-2.0</license>
 
   <buildtool_depend>ament_cmake</buildtool_depend>
-  <buildtool_depend>ament_cmake_pytest</buildtool_depend>
 
   <exec_depend>gz_ros2_control</exec_depend>
   <exec_depend>joint_state_publisher_gui</exec_depend>
@@ -17,6 +16,7 @@
   <exec_depend>tf2_ros</exec_depend>
   <exec_depend>xacro</exec_depend>
 
+  <test_depend>ament_cmake_pytest</test_depend>
   <test_depend>python3-pytest</test_depend>
 
   <export>