-
Notifications
You must be signed in to change notification settings - Fork 159
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
Skip CLI tests on Windows until we resolve the blocking/hanging isuse. #489
Conversation
We've been lacking on Windows test data for months due to the blocking or hung state that these tests can get into. I think the best thing to do is skip these tests on windows to allow other builds to complete while we continue investigating the problem. Signed-off-by: Steven! Ragnarök <steven@nuclearsandwich.com>
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.
SGTM
When we do our manually testing for Foxy, we can make sure to pick up the slack.
I'm pretty sure the test failures above were introduced by ros2/rclpy#536 (discussion). I am going to merge in order to improve the situation on Windows. |
Please ticket the follow up task to revert this change. |
Do you feel an issue separate from ros2/build_farmer#248 is warranted? |
I would be concerned that the task of reverting this change gets lost in the length and broader scope of that ticket. |
|
Other CLI tests are skipped on Windows since #489. To be reverted when ros2/build_farmer#248 is resolved. Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Other CLI tests are skipped on Windows since #489. To be reverted when ros2/build_farmer#248 is resolved. Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Fixes #480 The actual tests are the same, except with the use of launch_testing we ensure the CLI daemon is restarted between tests. This follows a similar pattern as the other ros2cli tests. In addition to converting to launch tests, this change also runs the tests for all RMW implementations. For now, we are skipping tests on Windows. Other CLI tests are skipped on Windows since #489. To be reverted when ros2/build_farmer#248 is resolved. Signed-off-by: Jacob Perron <jacob@openrobotics.org>
We've been lacking on Windows test data for months due to the blocking
or hung state that these tests can get into. I think the best thing to
do is skip these tests on windows to allow other builds to complete
while we continue investigating the problem.
I've opted for skipping entire test modules as I think the underlying issue is in the test apparatus rather than any specific test.