Skip to content

Fix the teardown of the diff_drive_controller test (backport #2183)#2186

Merged
christophfroehlich merged 1 commit intokiltedfrom
mergify/bp/kilted/pr-2183
Feb 25, 2026
Merged

Fix the teardown of the diff_drive_controller test (backport #2183)#2186
christophfroehlich merged 1 commit intokiltedfrom
mergify/bp/kilted/pr-2183

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Feb 25, 2026

Should probably fix ros-controls/ros2_control_ci#665


This is an automatic backport of pull request #2183 done by Mergify.

@christophfroehlich christophfroehlich merged commit edd5db3 into kilted Feb 25, 2026
13 checks passed
@christophfroehlich christophfroehlich deleted the mergify/bp/kilted/pr-2183 branch February 25, 2026 15:59
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.15%. Comparing base (5fdd616) to head (ea7e679).
⚠️ Report is 1 commits behind head on kilted.

Additional details and impacted files
@@           Coverage Diff           @@
##           kilted    #2186   +/-   ##
=======================================
  Coverage   85.15%   85.15%           
=======================================
  Files         151      151           
  Lines       14569    14573    +4     
  Branches     1260     1260           
=======================================
+ Hits        12406    12410    +4     
  Misses       1716     1716           
  Partials      447      447           
Flag Coverage Δ
unittests 85.15% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...ive_controller/test/test_diff_drive_controller.cpp 94.21% <100.00%> (+0.03%) ⬆️
...ntroller/test/test_omni_wheel_drive_controller.hpp 86.27% <100.00%> (+0.27%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

2 participants