Move test_cli_remapping into test_cli - #599
Conversation
|
Tick the box to add this pull request to the merge queue (same as
|
5a16452 to
7879b05
Compare
|
In this PR, I have resolved issue #279 by moving the test_cli_remapping package into the test_cli package.
I have tested the build locally and it compiles successfully. The tests are ready let me know if any changes are needed... |
fujitatomoya
left a comment
There was a problem hiding this comment.
lgtm with green CI.
|
Pulls: #599 |
Signed-off-by: Aaravanand <aaravanand@gmail.com>
7879b05 to
ca3b1fb
Compare
|
@fujitatomoya ptal |
|
@fujitatomoya all the tests passed |
Description
This PR consolidates the
test_cli_remappingpackage into thetest_clipackage to reduce the overhead of having too many granular test packages, keeping related CLI tests together.All source files for the remapping tests have been moved into
test_cli/test/and the build configurations (CMakeLists.txtandpackage.xml) have been merged accordingly. The oldtest_cli_remappingpackage has been entirely removed.Fixes #279
Is this user-facing behavior change?
No, this is purely an internal structural change to consolidate testing packages.
Did you use Generative AI?
I did use generative AI in some form in making the content of this
pull request.
Additional Information
test_clisuccessfully builds with the newly migrated remapping test (name_maker_rclcppexecutable andtest_cli_remapping.pylaunch test).launch_testing_ros,launch_testing_ament_cmake,test_msgs, etc.) have been moved over totest_cli.