-
Notifications
You must be signed in to change notification settings - Fork 388
Refactor test setup to use PathSubstitution #2781
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…s2_control_node_launch.py
|
@shrujanus, all pull requests must be targeted towards the |
christophfroehlich
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, but please Please open a PR against master branch.
| from ament_index_python.packages import get_package_share_directory, get_package_prefix | ||
| from launch import LaunchDescription | ||
| from launch_testing.actions import ReadyToTest | ||
| import launch_testing.markers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we still need this for @pytest.mark.launch_test?
| import pytest | ||
| import unittest | ||
| import os | ||
| import launch_testing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we use this?
Sorry about that! Should I close this PR and request another one to master? or is there a way to change that right here? |
|
As you like. you also can rebase your commit on top of master, force push, and change the base branch of this PR. |
The reason I used the Is it common for developers to create branches off their ROS distribution branch (e.g., |
ros2_control stack's rolling version is compatible with humble distro, see development version in the docs. |
Refactored the ./controller_manager/test/test_ros2_control_node_launch.py file to use PathSubstitution and removed the unnecessary variables in the import.