Change colcon.pkg dependencies to be test-dependencies.

[clalancette]

Description

The dependencies on ament_cmake, ament_cmake_gtest, and googletest-distribution are only needed when building for tests, so we should only inject them as test dependencies.

I've opened this against the 2.3.x branch (since that is what ROS 2 is currently using), but it probably should be backported to all branches.

@Mergifyio backport 2.2.x 1.0.x

Contributor Checklist

• Commit messages follow the project guidelines.
• The code follows the style guidelines of this project.
• [N/A] Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
• [N/A] Any new/modified methods have been properly documented using Doxygen.
• Changes are backport compatible: they do NOT break ABI nor change library core behavior.
• Changes are API compatible.
• [N/A] New feature has been added to the versions.md file (if applicable).
• Applicable backports have been included in the description.

Reviewer Checklist

• The PR has a milestone assigned.
• The title and description correctly express the PR's purpose.
• Check contributor checklist is correct.
• Check CI results: changes do not issue any warning.
• Check CI results: CI pass and failing tests are unrelated with the changes.

[MiguelCompany]

@Mergifyio backport 2.2.x 1.0.x

[mergify]

backport 2.2.x 1.0.x

✅ Backports have been created

Details

• #294 Change colcon.pkg dependencies to be test-dependencies. (backport #293) has been created for branch 2.2.x
• #295 Change colcon.pkg dependencies to be test-dependencies. (backport #293) has been created for branch 1.0.x but encountered conflicts

[MiguelCompany]

@clalancette Thank you!

[clalancette]

@clalancette Thank you!

Thanks for reviewing and merging :).