Fix the node name overlapping.

[destogl]

Fixes #3004
The demo for testing is in ros-controls/ros2_control_demos#1028

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.40%. Comparing base (5667d80) to head (506361b).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #3006   +/-   ##
=======================================
  Coverage   89.40%   89.40%           
=======================================
  Files         157      157           
  Lines       18781    18783    +2     
  Branches     1510     1510           
=======================================
+ Hits        16791    16793    +2     
  Misses       1369     1369           
  Partials      621      621           

Flag	Coverage Δ
unittests	89.40% <100.00%> (+<0.01%)	⬆️

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

Files with missing lines	Coverage Δ
...are_interface/src/hardware_component_interface.cpp	81.46% <100.00%> (+0.14%)	⬆️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[christophfroehlich]

LGTM, I suppose this fixes the reported issue.

For reference, we have to work on passing node arguments (especially remappings) to the hardware nodes, see #2144

[saikishor]

This works like a charm. Tested it with the example_2 + tried to add a publisher to the HW component and also tried to remap it from the ros2_control_node and it is working quite good