Skip to content

Add test to check JSB is not throwing when there is a boolean interface (backport #2115)#2131

Open
mergify[bot] wants to merge 1 commit intojazzyfrom
mergify/bp/jazzy/pr-2115
Open

Add test to check JSB is not throwing when there is a boolean interface (backport #2115)#2131
mergify[bot] wants to merge 1 commit intojazzyfrom
mergify/bp/jazzy/pr-2115

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Jan 29, 2026

Hi,

I was having some issue on the update method that was throwing an exception when using a boolean interface. Now I see that got fixed in ros-controls/ros2_control#2360 and I was using an older version of ros2_control.

Anyway, I've done this test for testing exactly that issue. I leave it here just in case you want to add it.


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

…ce (#2115)

(cherry picked from commit c440398)

# Conflicts:
#	joint_state_broadcaster/test/test_joint_state_broadcaster.hpp
@mergify
Copy link
Contributor Author

mergify bot commented Jan 29, 2026

Cherry-pick of c440398 has failed:

On branch mergify/bp/jazzy/pr-2115
Your branch is up to date with 'origin/jazzy'.

You are currently cherry-picking commit c440398.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   joint_state_broadcaster/src/joint_state_broadcaster.cpp
	modified:   joint_state_broadcaster/test/test_joint_state_broadcaster.cpp

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   joint_state_broadcaster/test/test_joint_state_broadcaster.hpp

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants