Skip to content

Conversation

@urfeex
Copy link
Member

@urfeex urfeex commented Jan 27, 2025

This PR attempts running the integration tests also with the External Control URCap and not only using headless mode.

@urfeex urfeex force-pushed the urcap_based_tests branch from 2ad8ba7 to eecd657 Compare January 27, 2025 11:36
@codecov
Copy link

codecov bot commented Jan 27, 2025

Codecov Report

Attention: Patch coverage is 71.25000% with 23 lines in your changes missing coverage. Please review.

Project coverage is 73.45%. Comparing base (2c12bee) to head (032d162).
Report is 20 commits behind head on master.

Files with missing lines Patch % Lines
src/example_robot_wrapper.cpp 71.25% 9 Missing and 14 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #252      +/-   ##
==========================================
+ Coverage   71.71%   73.45%   +1.74%     
==========================================
  Files          71       76       +5     
  Lines        2786     2969     +183     
  Branches      353      379      +26     
==========================================
+ Hits         1998     2181     +183     
+ Misses        596      588       -8     
- Partials      192      200       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@urfeex urfeex marked this pull request as ready for review January 27, 2025 15:05
@urfeex urfeex requested a review from a team January 27, 2025 15:05
@urfeex urfeex force-pushed the urcap_based_tests branch from db66760 to 032d162 Compare January 29, 2025 10:49
@urfeex urfeex merged commit fd2fda3 into master Feb 3, 2025
22 of 23 checks passed
@urfeex urfeex deleted the urcap_based_tests branch February 3, 2025 10:08
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.

3 participants