Replaces unnessary async methods with coroutine generating methods

[AsgeirNoM]

Methods have been stripped of async and now return Coroutine[Any, Any, Response] why?

• The async methods did not do anything requiring scheduling.

• Running them synchronously and returning the actually async parts as Coroutines means the loop now has to schedule and plan half the number of tasks, while the logic remains the same.

• Due to returning the Coroutines the following was changed:

  • await wait_hello_from_server now returns Literal[True] instead of None
  • await wait_for_idle_response didn't have a return type, but now returns Literal[True]

Test was altered, why?

• It also failed in the main branch, I didn't really investigate but it seams odd that it would be off by 2.