Skip to content

Add tests isolation in rclcpp/test (backport #3081)#3127

Open
mergify[bot] wants to merge 1 commit intokiltedfrom
mergify/bp/kilted/pr-3081
Open

Add tests isolation in rclcpp/test (backport #3081)#3127
mergify[bot] wants to merge 1 commit intokiltedfrom
mergify/bp/kilted/pr-3081

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Apr 13, 2026

Description

As discussed in ros2/rmw_zenoh#881 (comment), this PR adds tests isolation to all tests initializing a Context or a Node, in order to start a Zenoh router in case of rmw_zenoh_cpp.

On my host before this fix:

  • 900 warning logs "Unable to connect to a Zenoh router"
  • Total Test time (real) = 770.40 sec

After this fix:

  • 0 warning logs "Unable to connect to a Zenoh router"
  • Total Test time (real) = 480.62 sec

Is this user-facing behavior change?

No

Did you use Generative AI?

No


This is an automatic backport of pull request #3081 done by [Mergify](https://mergify.com).

Signed-off-by: Julien Enoch <julien.e@zettascale.tech>
(cherry picked from commit 87be5fb)

# Conflicts:
#	rclcpp/test/rclcpp/CMakeLists.txt
@mergify mergify bot added the conflicts label Apr 13, 2026
@mergify
Copy link
Copy Markdown
Contributor Author

mergify bot commented Apr 13, 2026

Cherry-pick of 87be5fb has failed:

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

You are currently cherry-picking commit 87be5fb.
  (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)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   rclcpp/test/rclcpp/CMakeLists.txt

no changes added to commit (use "git add" and/or "git commit -a")

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.

1 participant