fix: linter tests

[wep21]

🦟 Bug fix

Fixes #

Summary

fix some linter tests

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• Consider updating Python bindings (if the library has them)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

[wep21]

@chapulina I appreciate it if you do bloom-release to rolling/humble distribution after merging this PR.

[chapulina]

Thanks for the PR! I pushed 94da01d with 2 changes:

• Restore testing repo for CI
• Instead of suppressing the ament warnings, fix the include headers. This is finally our chance to group the Gazebo and ROS C++ headers together 😄

I noticed there's a test failure both locally for me and on CI. I'm not sure if it's something we're doing wrong or some regression upstream. I'll merge with that failure for now, since this PR is about linters.

[4.654s] 1: [test_ros_subscriber-3] cannot publish data, at ./src/rmw_publish.cpp:62 during '__function__'
[4.655s] 1: [test_ros_subscriber-3] [ERROR] [1653006329.567666397] [ros_subscriber.rclcpp]: Error in destruction of rcl subscription handle: Failed to delete datareader, at ./src/subscription.cpp:52, at ./src/rcl/subscription.c:183
[4.655s] 1: [test_ros_subscriber-3] cannot publish data, at ./src/rmw_publish.cpp:62 during '__function__'
[4.655s] 1: [test_ros_subscriber-3] Fail in delete datareader, at ./src/rmw_service.cpp:103 during '__function__'