Update initial_value parameters of generic_system tests

[kumar-sanjeeev]

• Add initial values for every state interface in the ros2_control.xacro.
• Update print_hint to false to suppress warnings when no initial value is provided.

[mergify]

This pull request is in conflict. Could you fix it @kumar-sanjeeev?

[mergify]

@kumar-sanjeeev, all pull requests must be targeted towards the master development branch.
Once merged into master, it is possible to backport to humble, but it must be in master
to have these changes reflected into new distributions.

[christophfroehlich]

Thanks for the PR! I changed the base branch to humble, it seems that you have forgotten to change the default.
When you commit another time, the checks will be triggered hopefully.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 62.89%. Comparing base (d897740) to head (1c0ceb8).
Report is 30 commits behind head on humble.

Additional details and impacted files

@@            Coverage Diff             @@
##           humble    #1943      +/-   ##
==========================================
+ Coverage   62.58%   62.89%   +0.31%     
==========================================
  Files         108      109       +1     
  Lines       12026    12494     +468     
  Branches     8133     8481     +348     
==========================================
+ Hits         7526     7858     +332     
+ Misses        880      852      -28     
- Partials     3620     3784     +164     

Flag	Coverage Δ
unittests	62.89% <ø> (+0.31%)	⬆️

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

Files with missing lines	Coverage Δ
...e_interface/src/mock_components/generic_system.cpp	77.37% <ø> (-2.63%)	⬇️
...rface/test/mock_components/test_generic_system.cpp	36.53% <ø> (ø)

... and 3 files with indirect coverage changes

[christophfroehlich]

LGTM, there are no more deprecation warnings in the logs!

I don't think that we would need

      <state_interface name="velocity">
        <param name="initial_value">0.0</param>
      </state_interface>
      <state_interface name="acceleration">
        <param name="initial_value">0.0</param>
      </state_interface>

everywhere, at least it isn't done in #1150
This will only make a difference if calculate_dynamics is true

[kumar-sanjeeev]

LGTM, there are no more deprecation warnings in the logs!

I don't think that we would need

      <state_interface name="velocity">
        <param name="initial_value">0.0</param>
      </state_interface>
      <state_interface name="acceleration">
        <param name="initial_value">0.0</param>
      </state_interface>

everywhere, at least it isn't done in #1150 This will only make a difference if calculate_dynamics is true

Okay to be consistence with #1150, I will also remove them from here.