Skip to content

rqt-jtc: Add launch test#2405

Open
christophfroehlich wants to merge 1 commit into
masterfrom
rqt-launch-test
Open

rqt-jtc: Add launch test#2405
christophfroehlich wants to merge 1 commit into
masterfrom
rqt-launch-test

Conversation

@christophfroehlich

Copy link
Copy Markdown
Member

Description

Similar to the test added with ros-controls/ros2_control#3367

Is this user-facing behavior change?

no

Did you use Generative AI?

GPT Codex 5.3

@christophfroehlich christophfroehlich added backport-humble Triggers PR backport to ROS 2 humble. backport-jazzy Triggers PR backport to ROS 2 jazzy. backport-kilted Triggers PR backport to ROS 2 kilted. labels Jun 9, 2026
@codecov

codecov Bot commented Jun 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.51%. Comparing base (9f162e1) to head (69c8aff).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2405      +/-   ##
==========================================
+ Coverage   85.60%   86.51%   +0.90%     
==========================================
  Files         148      149       +1     
  Lines       15827    15851      +24     
  Branches     1339     1339              
==========================================
+ Hits        13549    13713     +164     
+ Misses       1792     1651     -141     
- Partials      486      487       +1     
Flag Coverage Δ
unittests 86.51% <100.00%> (+0.90%) ⬆️

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

Files with missing lines Coverage Δ
...oint_trajectory_controller/test/test_run_launch.py 100.00% <100.00%> (ø)

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@christophfroehlich christophfroehlich marked this pull request as ready for review June 10, 2026 07:30
@christophfroehlich christophfroehlich moved this to Needs review in Review triage Jun 10, 2026
@christophfroehlich

Copy link
Copy Markdown
Member Author

@christianrauch do you also use rqt_joint_trajectory_controller? Is there anything we have to fix for Qt6?

@christianrauch

Copy link
Copy Markdown
Contributor

I don't use the rqt_joint_trajectory_controller, and it will take a while until I can test it with lyrical.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-humble Triggers PR backport to ROS 2 humble. backport-jazzy Triggers PR backport to ROS 2 jazzy. backport-kilted Triggers PR backport to ROS 2 kilted.

Projects

Status: Needs review

Development

Successfully merging this pull request may close these issues.

2 participants