ros2 / ros2/system_tests

update test for avoid_ros_namespace_conventions QoS option to use no_demangle

Open
#209 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
C++
Stars
45
Forks
57
Avg merge
9h 19m
Merged PRs (30d)
2

Description

Leftover from: https://github.com/ros2/ros2/issues/361

Again, like https://github.com/ros2/rcl/issues/146, I'm not sure how to test this exactly.

But if we can figure out how to test this in a portable way, the test here can probably be removed in favor of the test in https://github.com/ros2/rcl/issues/146.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Read the context in ros2/ros2#361 and ros2/rcl#146 first, since the issue does not identify a test file or entry point. Determine whether a portable test can cover the avoid_ros_namespace_conventions QoS option using no_demangle; done means either updating this system test accordingly or removing it in favor of the rcl test.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
testing-qa
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.