Previously if open() was a coroutine and raised an error, the
connection would be left open.

Fixes #2570
backport of dc354c57 to 5.1

TestWebsocketHandler always requires close_future in this branch