Skip to content

Make all packages use gmock, not gtest #2162

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Apr 3, 2025

Conversation

soham2560
Copy link
Contributor

@soham2560 soham2560 commented Apr 3, 2025

Brief

In this PR

How was this tested?

colcon build --symlink-install
source install/setup.bash
colcon test --packages-select controller_interface controller_manager controller_manager_msgs hardware_interface hardware_interface_testing joint_limits ros2_control ros2_control_test_assets ros2controlcli rqt_controller_manager transmission_interface --event-handlers console_direct+ 

Note: Do let me know if there is a better way to have tested this! just did what came first to my mind

Copy link
Contributor

@christophfroehlich christophfroehlich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks!

@christophfroehlich christophfroehlich enabled auto-merge (squash) April 3, 2025 21:39
@christophfroehlich christophfroehlich merged commit bb4af6d into ros-controls:master Apr 3, 2025
15 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Check that all packages use gmock, not gtest
3 participants