Missing RViz dependency in MSA template

[DLu]

Description

Adds a missing dependency on RViz to the MSA template. If not, the package will fail roslaunch checks if they are added.

Checklist

• Required by CI: Code is auto formatted using clang-format
• Extend the tutorials / documentation reference
• Document API changes relevant to the user in the MIGRATION.md notes
• Create tests, which fail without this PR reference
• Include a screenshot if changing a GUI
• While waiting for someone to review your request, please help review another open pull request to support the maintainers

[codecov]

Codecov Report

Merging #2455 (00aca0c) into master (bdd4bde) will increase coverage by 0.10%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2455      +/-   ##
==========================================
+ Coverage   60.24%   60.34%   +0.10%     
==========================================
  Files         351      351              
  Lines       26435    26455      +20     
==========================================
+ Hits        15923    15961      +38     
+ Misses      10512    10494      -18     

Impacted Files	Coverage Δ
..._interface/src/detail/constrained_goal_sampler.cpp	72.50% <0.00%> (-10.00%)	⬇️
...bot_state/include/moveit/robot_state/robot_state.h	90.91% <0.00%> (ø)
...dl_kinematics_plugin/src/kdl_kinematics_plugin.cpp	76.55% <0.00%> (+0.45%)	⬆️
...on/pick_place/src/approach_and_translate_stage.cpp	72.96% <0.00%> (+0.82%)	⬆️
moveit_core/utils/src/robot_model_test_utils.cpp	73.15% <0.00%> (+1.05%)	⬆️
...kinematic_constraints/src/kinematic_constraint.cpp	74.63% <0.00%> (+1.24%)	⬆️
...ipulation/pick_place/src/manipulation_pipeline.cpp	74.47% <0.00%> (+2.13%)	⬆️
...e/src/parameterization/model_based_state_space.cpp	72.80% <0.00%> (+2.40%)	⬆️
moveit_core/robot_model/src/joint_model_group.cpp	65.44% <0.00%> (+2.47%)	⬆️
...eit_ros/manipulation/pick_place/src/pick_place.cpp	92.23% <0.00%> (+3.34%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bdd4bde...00aca0c. Read the comment docs.

[DLu]

I also think moveit_simple_controller_manager should have a dependency here. Can add it if this is approved.

[rhaschke]

I also think moveit_simple_controller_manager should have a dependency here. Can add it if this is approved.

Yes, this as well as moveit_fake_controller_manager is needed.

[DLu]

moveit_fake_controller_manager is already there

[DLu]

Good to merge then?