SingleThreadedExecutor::spin_once_until_future_complete continues to spin even if future is completed

[fujitatomoya]

Bug report

Required Info:

• Operating System:
  • Ubuntu 22.04
• Installation type:
  • source build
• Version or commit hash:
  • ros2/ros2@5a35507
• DDS implementation:
  • Any (Fast-RTPS, RTI Connext)
• Client library (if applicable):
  • rclpy

Expected behavior

SingleThreadedExecutor::spin_once_until_future_complete should return if the future is completed

Additional information

rclpy/rclpy/rclpy/executors.py

Lines 725 to 726 in 9a5d55e

	def spin_once_until_future_complete(self, future: Future, timeout_sec: float = None) -> None:
	self.spin_once(timeout_sec)

[130s]

I'm unsure if this is related but me and some others saw an issue with spin_once_until_future_complete in answers.ros.org#386260 (I do not say we confirmed it as a bug).