Populate TaskProfile in TaskSummary

[Yadunund]

Fix #80 by implementing select changes from #24

[codecov]

Codecov Report

Merging #81 (2e56c91) into main (f4cf7f7) will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
+ Coverage   22.02%   22.07%   +0.05%     
==========================================
  Files         410      410              
  Lines       32894    32858      -36     
  Branches    16064    16022      -42     
==========================================
+ Hits         7244     7255      +11     
+ Misses      17814    17795      -19     
+ Partials     7836     7808      -28     

Flag	Coverage Δ
tests	22.07% <ø> (+0.05%)	⬆️

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

Impacted Files	Coverage Δ
...f_fleet_adapter/services/detail/impl_Negotiate.hpp
...ter/src/rmf_fleet_adapter/agv/test/MockAdapter.cpp
...ter/src/rmf_fleet_adapter/agv/test/MockAdapter.cpp
...2/rmf_fleet_adapter_python/src/planner/planner.cpp
...leet_adapter/src/rmf_fleet_adapter/agv/Adapter.cpp
...ffic_ros2/test/mock_schedule_nodes/wrong_query.cpp
...mf_fleet_adapter/services/detail/impl_FindPath.hpp
...Cpp-4.1.0/Rx/v2/src/rxcpp/operators/rx-finally.hpp
...ter/include/rmf_fleet_adapter/agv/TrafficLight.hpp
...f_fleet_adapter/test/phases/MockAdapterFixture.cpp
... and 810 more

[youliangtan]

LGTM!! Also tested it, looks fine 👍

[Yadunund]

Hmm not sure why test_rmf_fleet_adapter is failing after merging in main 🤔

  The following tests FAILED:
  	  2 - test_rmf_fleet_adapter (Failed)
  Errors while running CTest
  Output from these tests are in: /home/runner/work/rmf_ros2/rmf_ros2/ros_ws/build/rmf_fleet_adapter/Testing/Temporary/LastTest.log
  Use "--rerun-failed --output-on-failure" to re-run the failed cases verbosely.
  ---

[mxgrey]

According to the test artifacts, a segfault happened in test_RequestLift. I don't see any particular explanation for that, and I haven't seen a crash happen there before. Let's keep a careful eye on that test moving forward, to see if it crashes regularly. For now, let's just rerun the test and not block this PR, since it's almost certainly an unrelated issue.

[Yadunund]

Re-ran the test and it seems to pass this time.

[youliangtan]

🎉