Use target_compile_definitions instead of installing test files

[saikishor]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.37%. Comparing base (d4139b7) to head (f967d58).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2009      +/-   ##
==========================================
- Coverage   89.38%   89.37%   -0.02%     
==========================================
  Files         132      132              
  Lines       14663    14649      -14     
  Branches     1258     1258              
==========================================
- Hits        13106    13092      -14     
- Misses       1090     1091       +1     
+ Partials      467      466       -1     

Flag	Coverage Δ
unittests	89.37% <100.00%> (-0.02%)	⬇️

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

Files with missing lines	Coverage Δ
...er_interface/test/test_controller_with_options.cpp	100.00% <100.00%> (ø)
controller_manager/test/test_spawner_unspawner.cpp	96.62% <100.00%> (-0.11%)	⬇️

... and 4 files with indirect coverage changes

[christophfroehlich]

LGTM, apart from removing ament_index_cpp from the deps in CMakeLists and package.xml

[christophfroehlich]

Thanks!