nav2_behavior_tree: fix input port parsing error in AreErrorCodesPresent

[DylanDeCoeyer-Quimesis]

The getInput method does not support std::set<uint16_t> parsing. So, let's replace the type of the input port by std::vector which is supported, and convert the list to a std::set<uint16_t>.

This commit fixes issue #4985.

---

Basic Info

Info	Please fill out this column
Ticket(s) this addresses	(add tickets here #1)
Primary OS tested on	(Ubuntu, MacOS, Windows)
Robotic platform tested on	(Steve's Robot, gazebo simulation of Tally, hardware turtlebot)
Does this PR contain AI generated software?	(No; Yes and it is marked inline in the code)

---

Description of contribution in a few bullet points

Description of documentation updates required from your changes

Description of how this change was tested

---

Future work that may be required in bullet points

For Maintainers:

• Check that any new parameters added are updated in docs.nav2.org
• Check that any significant change is added to the migration guide
• Check that any new features OR changes to existing behaviors are reflected in the tuning guide
• Check that any new functions have Doxygen added
• Check that any new features have test coverage
• Check that any new plugins is added to the plugins page
• If BT Node, Additionally: add to BT's XML index of nodes for groot, BT package's readme table, and BT library lists

[SteveMacenski]

I'm pretty irritated that this wasn't tested thoroughly by the original developer, but I see looking at the unit tests for this node how this got based us. This was the base class of the derived Controller/Planner/Smoother variants that set the values in their constructor based on contextual knowledge, so I see how this may not have been appropriately tested as a stand alone node. That's my bad in quality control, sorry!

[SteveMacenski]

I think the unit test just needs the input type updated to vector:

unknown file
C++ exception with description "The creation of the tree failed because the port [error_codes_to_check] was initially created with type [std::set<unsigned short, std::less<unsigned short>, std::allocator<unsigned short> >] and, later type [std::vector<int, std::allocator<int> >] was used somewhere else." thrown in SetUp().

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines	Coverage Δ
...ns/condition/are_error_codes_present_condition.hpp	100.00% <100.00%> (ø)

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.