Added tests for the rtde interface clasess #125
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This adds tests for * control package pause * control package start * control package setup outputs * control package setup inputs * get urcontrol version * request protocol version * rtde writer Added more tests to * test_rtde_data_package * test_rtde_parser * test_rtde_client Added functionality to rtde_client * get_target_frequency function and warning when it is not possible to set target frequency * Read timeout in isRobotBooted is now based on target frequency * Added a wait on 5 seconds to receive answer from pause request. This will prevent the driver from crashing once in a while when pausing the rtde client Added functionality for rtde_writer * Added checks for values and pins before sending to the robot * Removed unnecessary sleep in desctructor Added pipeline changes, so that the test output is shown in the pipeline in case of failure
Codecov ReportBase: 51.21% // Head: 63.40% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #125 +/- ##
===========================================
+ Coverage 51.21% 63.40% +12.19%
===========================================
Files 82 89 +7
Lines 2466 2968 +502
Branches 279 310 +31
===========================================
+ Hits 1263 1882 +619
+ Misses 1072 925 -147
- Partials 131 161 +30
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
fmauch
approved these changes
Oct 25, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This adds tests for
Added more tests to
Added functionality to rtde_client
Added functionality for rtde_writer
Added pipeline changes, so that the test output is shown in the pipeline in case of failure