Skip to content

Commit

Permalink
chore: remove skipped test (#392)
Browse files Browse the repository at this point in the history
  • Loading branch information
rachwalk authored Jan 28, 2025
1 parent 1455379 commit 7ca87dd
Showing 1 changed file with 0 additions and 29 deletions.
29 changes: 0 additions & 29 deletions tests/communication/ros2/test_connectors.py
Original file line number Diff line number Diff line change
Expand Up @@ -163,32 +163,3 @@ def test_ros2ari_connector_send_goal_erronous_callback(
finally:
connector.shutdown()
shutdown_executors_and_threads(executors, threads)


@pytest.mark.skip(
reason="""
This test does not allow pytest to exit.
TODO: Decide whether still working callbacks should
be cancelled when shutting down"""
)
def test_ros2ari_connector_send_goal_blocking_callback(
ros_setup: None, request: pytest.FixtureRequest
):
action_name = f"{request.node.originalname}_action" # type: ignore
action_server = ActionServer(action_name)
executors, threads = multi_threaded_spinner([action_server])
connector = ROS2ARIConnector()
try:
message = ROS2ARIMessage(
payload={},
metadata={"msg_type": "nav2_msgs/action/NavigateToPose"},
)
handle = connector.start_action(
action_data=message,
target=action_name,
on_feedback=lambda feedback: time.sleep(100),
)
assert handle is not None
finally:
connector.shutdown()
shutdown_executors_and_threads(executors, threads)

0 comments on commit 7ca87dd

Please sign in to comment.