refactor(raw_vehicle_cmd_converter): remove duplicates and add unit test

[taikitanaka3]

Description

• refactor change

100% tests passed, 0 tests failed out of 5

Label Time Summary:
copyright     =   2.17 sec*proc (1 test)
cppcheck      =   0.70 sec*proc (1 test)
gtest         =   0.15 sec*proc (1 test)
lint_cmake    =   0.40 sec*proc (1 test)
linter        =   4.92 sec*proc (4 tests)
xmllint       =   1.64 sec*proc (1 test)

Total Test time (real) =   5.08 sec
Finished <<< raw_vehicle_cmd_converter [5.44s]          

Summary: 1 package finished [6.10s]

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

• I've confirmed the contribution guidelines.
• The PR follows the pull request guidelines.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

• The PR follows the pull request guidelines.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

• There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

[codecov]

Codecov Report

Base: 10.44% // Head: 10.41% // Decreases project coverage by -0.03% ⚠️

Coverage data is based on head (a3a334d) compared to base (94c248b).
Patch coverage: 35.46% of modified lines in pull request are covered.

❗ Current head a3a334d differs from pull request most recent head f2eec00. Consider uploading reports for the commit f2eec00 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2077      +/-   ##
==========================================
- Coverage   10.44%   10.41%   -0.04%     
==========================================
  Files        1171     1177       +6     
  Lines       77152    84162    +7010     
  Branches    19144    19665     +521     
==========================================
+ Hits         8061     8763     +702     
- Misses      59973    65850    +5877     
- Partials     9118     9549     +431     

Flag	Coverage Δ		*Carryforward flag
differential	6.76% <30.35%> (?)
total	10.34% <32.43%> (-0.11%)	⬇️	Carriedforward from 465713e

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files	Coverage Δ
...nverter/include/raw_vehicle_cmd_converter/node.hpp	0.00% <ø> (ø)
...onverter/include/raw_vehicle_cmd_converter/pid.hpp	0.00% <0.00%> (ø)
vehicle/raw_vehicle_cmd_converter/src/node.cpp	0.00% <0.00%> (ø)
vehicle/raw_vehicle_cmd_converter/src/pid.cpp	0.00% <0.00%> (ø)
...ehicle/raw_vehicle_cmd_converter/src/brake_map.cpp	27.27% <27.77%> (+12.27%)	⬆️
..._converter/test/test_raw_vehicle_cmd_converter.cpp	31.91% <33.33%> (+2.09%)	⬆️
...ehicle/raw_vehicle_cmd_converter/src/accel_map.cpp	31.91% <35.29%> (+15.24%)	⬆️
...ehicle/raw_vehicle_cmd_converter/src/steer_map.cpp	47.05% <47.05%> (ø)
common/osqp_interface/src/osqp_interface.cpp	34.35% <75.00%> (+0.60%)	⬆️
...hicle/raw_vehicle_cmd_converter/src/csv_loader.cpp	74.00% <85.00%> (+30.66%)	⬆️
... and 767 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[takayuki5168]

LGTM

[taikitanaka3]

TODO
rtemove ff_map_initialized_ and terminate when there is no ff map