-
Notifications
You must be signed in to change notification settings - Fork 324
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
[gripper] Add test for effort gripper controller #769
[gripper] Add test for effort gripper controller #769
Conversation
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #769 +/- ##
==========================================
- Coverage 35.78% 30.88% -4.90%
==========================================
Files 189 7 -182
Lines 17570 832 -16738
Branches 11592 505 -11087
==========================================
- Hits 6287 257 -6030
+ Misses 994 133 -861
+ Partials 10289 442 -9847
Flags with carried forward coverage won't be shown. Click here to find out more.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
I used gmock for many years but never came across typed tests, you learn something new every day! This is very elegant and clean, thanks!
(cherry picked from commit fb46a77)
(cherry picked from commit fb46a77)
Add test for effort gripper controller.
Fix #764
GripperController uses template parameters, so I used type-parameterized tests.
https://github.com/google/googletest/blob/main/docs/advanced.md#type-parameterized-tests
https://en.cppreference.com/w/cpp/types/integral_constant