Clean the namespace to get rid of double or trailing forward slashes …

[lucasw]

…#1094

This cleans the namespace provided to this_node, getting rid of excess slashes ('//' becomes '/').

[nuclearsandwich]

@ros-pull-request-builder retest this please.

[lucasw]

(How to build and run the tests:

sudo apt-get install python-ros-buildfarm
cd ~/lunar_catkin_ws
catkin_make_isolated_and_test.py --rosdistro-name lunar --workspace-root .

-but then on one of my systems this is failing all over:

FAILURE: test [paramtest_value_specific_correct] did not generate test results
  File "/usr/lib/python2.7/unittest/case.py", line 329, in run
    testMethod()
  File "/home/lucasw/lunar_catkin_ws/src/ros_comm/tools/rostest/src/rostest/runner.py", line 164, in fn
    self.assert_(os.path.isfile(test_file), "test [%s] did not generate test results"%test_name)
  File "/usr/lib/python2.7/unittest/case.py", line 422, in assertTrue
    raise self.failureException(msg)

Then how to run just the namespace test?

source devel_isolated/setup.bash
rostest src/ros_comm/test/test_roscpp/test/launch/namespaces.xml

)

[dirk-thomas]

The patch looks good. Thank you. Merging...

How to build and run the tests:

Please see the docs for the ros_buildfarm repo (https://github.com/ros-infrastructure/ros_buildfarm/blob/master/doc/jobs/devel_jobs.rst#run-the-devel-job-locally) for this. It only supports running the job within Docker though.